Bump ChangeLog to increment version number.

This change to ChangeLog serves only to regenerate a new version number
on the release branch in preparation for a release.

* CHANGES: Revise to cover all recent changes/fixes.

2005-09-01  Brian Dessent  <brian@dessent.net>

* package_meta.cc (packagemeta::uninstall): Remove support for
undocumented and dangerous postremove script.

2005-09-01  Igor Pechtchanski  <pechtcha@cs.nyu.edu>

* script.cc (run): Change to return the exit code or negative error.
(Script::run): Ditto.
(try_run_script): Receive both filename and extension and run only
one script.  Also return the exit code.
* script.h (try_run_script): Change signature.
(Script::run): Ditto.
* postinstall.cc (RunScript::operator()): Change to return the exit
code or negative error.
* install.cc (Installer::preremoveOne): Pass extension to
try_run_script().

2005-09-01  Eric Blake  <ebb9@byu.net>

* script.cc (shells): Use /bin/bash, not sh, to allow upgrades to
/bin/sh.

Fix whitespace.

2005-09-01  Igor Pechtchanski  <pechtcha@cs.nyu.edu>

* PickView.h (PickView::size_col): New instance variable.
* PickView.cc (pkg_headers, cat_headers): Add size column.
(PickView::set_headers): Initialize size_col.
(PickView::init_headers): Include width of size column.
* PickPackageLine.cc (PickPackageLine::paint): Handle size_col.
* String++.cc (format_1000s): New function.
* String++.h (format_1000s): Declare new function.

2005-09-01  Brian Dessent  <brian@dessent.net>

* Makefile.am (BASECXXFLAGS): Add -Wno-uninitialized to work around
spurious warnings with gcc 3.4.4.

libgetopt++/
2005-09-01  Brian Dessent  <brian@dessent.net>

* Makefile.am (AM_CXXFLAGS): Add -Wno-uninitialized to work around
spurious warnings with gcc 3.4.4.

* CHANGES: First draft for new release.

* CONTRIBUTORS: Add Frank Richter.

* CONTRIBUTORS: Add myself, Igor Pechtchanski and Brian Dessent.

2005-05-22  Brian Dessent  <brian@dessent.net>

* res.rc (IDD_CHOOSE): Change description to include administrative
packages in addition to obsolete.

2005-05-21  Brian Dessent  <brian@dessent.net>

* prereq.cc: New file.
* prereq.h: Ditto.
* tree-minus.bmp: Ditto.
* tree-plus.bmp: Ditto.
* Makefile.am: Add prereq.cc and prereq.h to build.
* PickCategoryLine.cc (PickCategoryLine::paint): Add new parameter and
refactor.  Use bitmap for '+' and '-' indicators.  Draw package lines
column-wise for better drawing efficiency.
(PickCategoryLine::click): Use previously calculated x value.
* PickCategoryLine.h (PickCategoryLine::paint): Update prototype.  Add
'spin_x' member.
* PickLine.h (PickLine:paint): Update prototype.
* PickPackageLine.cc (PickPackageLine::DrawCheck): Remove.
(PickPackageLine::DrawIcon): New function.
(PickPackageLine::paint): Add parameter.  Refactor completely for more
efficient painting.
* PickPackageLine.h (PickPackageLine::DrawIcon): Add prototype.
(PickPackageLine::paint): Update prototype.
(PickPackageLine::DrawCheck): Remove.
* PickView.cc (pkg_headers): Remove unused 'slen' field, add new
'needs_clip' field to signify if clipping is necessary for this column.
(PickView::note_width): Fold long line.  Refactor slightly for
simplicity.
(PickView::set_view_mode): Remove.
(PickView::cycleViewMode): New function.
(PickView::setViewMode): Refactor completely.  Incorporate functionality
of clear_view() here instead.  Simplify package selection into a single
'for' loop rather than repeating code for each type.
(isObsolete): New function, with second overloaded version.
(PickView::setObsolete): New function.
(PickView::insert_pkg): Do not show obsolete packages.  Move
declaration of 'db' lower.  Fix whitespace.
(PickView::insert_category): Do not show obsolete packages.
(PickView::clear_view): Remove.  Moved logic into 'setViewMode'.
(PickView::scroll): Add additional parameter 'howmany' with default
value of 1.  Scroll by 'howmany' lines for SB_LINEDOWN and SB_LINEUP.
(NUM_CATEGORY_COL_WIDTH): Define.
(PickView::init_headers): Fix whitespace.  Add comments.  Ignore
obsolete packages in width calculations.  Add functionality to size
'Category' column so that at least NUM_CATEGORY_COL_WIDTH categories
are displayed for every package.  Ensure that 'new_col' takes into
account all possible labels.
(PickView::PickView): Initialize showObsolete to false.
(PickView::init): Remove unused bitmap bm_rtarrow.  Add new bitmaps
bm_treeplus and bm_treeminus.  Use a macro for better readability.
Fix whitespace formatting.  Remove header computation from here, call
refresh() instead.
(PickView::registerWindowClass): Remove unnecessary background brush.
(PickView::WindowProc): Add support for WM_MOUSEWHEEL message
processing.  Fix indentation.  Reformat for readability.
(PickView::paint): Get the update region before calling BeginPaint.
Fill the update region with the default colors.  Do not do clipping
here.  Pass update region to the PickLine paint function instead.
Free the update region when finished.
(PickView::Create): Fix indentation.
(PickView::defaultTrust): Ditto.
(PickView::refresh): Add column recalculation logic here.
* PickView.h: Fix indentation.
(RTARROW_WIDTH): Remove unused define.
(NEW_COL_SIZE_SLOP): Ditto.
(TREE_INDENT): Define.
(PickView::get_view_mode): Remove.
(PickView::set_view_mode): Rename to setViewMode for consistency.
(PickView::cycleViewMode): Add new function.
(PickView::setObsolete): Ditto.
(PickView::clear_view): Remove.
(PickView::bm_treeplus): Define new member.
(PickView::bm_treeminus): Ditto.
(PickView::scroll): Add parameter.
(PickView::Header::slen): Remove unused member.
(PickView::Header::needs_clip): Define new member.
(PickView::showObsolete): Ditto.
(isObsolete): Add prototype, and for overloaded version as well.
* choose.cc: Add include.
(ChooserControlsInfo): Allow new checkbox to be resized.
(ChooserPage::OnInit): Start dialog with 'Hide obsolete' checked.
(ChooserPage::OnNext): Call the dependency checker, act accordingly.
(ChooserPage::changeTrust): Notify dependency checker of current trust
level when it changes.
(ChooserPage::OnMessageCmd): Just call cycleViewMode instead when
changing the view.  Update the chooser when the user toggles the
'hide obsolete' checkbox.
(ChooserPage::OnMouseWheel): New function.  Pass mouse wheel message
on to chooser.
* choose.h (OnMouseWheel): Add prototype.
* main.cc: Add include.
(main): Define and initialize the PrereqPage object.  Add it into
the property sheet list.
* PropPage.cc (PropertyPage::DialogProc): Add support for WM_MOUSEWHEEL
message processing in derived classes.
(PropertyPage::OnMouseWheel): New function.
* PropPage.h (PropertyPage::OnMouseWheel): Add prototype.
* res.rc (IDD_LOCAL_DIR): Shorten edit box so that pushbutton is not
crowded.
(IDD_SITE): Provide static text with an ID so that it can be resized
properly.  Move pushbutton slightly to right so that it doesn't crowd
edit box.
(IDD_SPLASH): Change copyright displayed on spash screen to be generic
since the actual list of contributors is long.
(IDD_CHOOSE): Add checkbox.  Resize other controls to accomodate.
(IDD_PREREQ): New dialog.
(SPIN): Delete duplicated bitmap resource.
(IDB_RTARROW): Remove unused bitmap resource.
(IDB_TREE_PLUS): Add.
(IDB_TREE_MINUS): Ditto.
(IDS_TRUSTPREV_TOOLTIP): Revise wording to be more concise.
(IDS_TRUSTCURR_TOOLTIP): Ditto.
(IDS_TRUSTEXP_TOOLTIP): Ditto.
(IDS_VIEWBUTTON_TOOLTIP): Ditto.
* resource.h (IDD_PREREQ): Define.
(IDB_RTARROW): Remove.
(IDB_TREE_PLUS): Add, and renumber.
(IDB_TREE_MINUS): Ditto.
(IDC_SITE_USERURL): Define.
(IDC_CHOOSE_HIDE): Ditto.
(IDC_PREREQ_TEXT): Ditto.
(IDC_PREREQ_EDIT): Ditto.
(IDC_PREREQ_CHECK): Ditto.
* site.cc (SiteControlsInfo): Ensure that 'User URL' static text is
properly resized.

2005-05-14  Max Bowsher  <maxb@ukf.net>

"Type" field of packageversions was not even being filled in. Fix.

* cygpackage.h, cygpackage.cc (cygpackage::createInstance):
Make "type" a mandatory parameter of the constructor.
* IniDBBuilderPackage.cc: Adjust callers.

2005-05-14  Bas van Gompel  <patch-cygsup.buzz@bavag.tmfweb.nl>

Fix behaviour of hardlinks in source packages.

* archive.cc (archive::extract_file):
Use prefixPath for link target on hardlinks.

2005-05-14  Max Bowsher  <maxb@ukf.net>

Hopefully, solve the "preremove DLL dependency" bug, by running all
preremove scripts before actually removing any files.

* install.cc
(Installer::preremoveOne): New method. Run preremove scripts here.
(Installer::replaceOne): Remove method.
(num_replacements): Remove variable.
(do_install_thread): Significantly rewrite.
* package_meta.cc (packagemeta::uninstall):
Do not run preremove script here.

2005-05-14  Max Bowsher  <maxb@ukf.net>

* download.cc (do_download_thread):
Do not use packageversion::changeRequested().
* package_version.cc (packageversion::Type):
Implement declared but previously unimplemented method.
(packageversion::changeRequested, _packageversion::changeRequested):
Remove.
* package_version.h
(packageversion::changeRequested, _packageversion::changeRequested):
Remove.
* install.cc (Installer::replaceOne): Simplify, return void, not int.
(installOneSource): Rename to...
(installOne): ...this. Change argumentsi and return type. Refactor.
(install_one): Remove.
(do_install_thread): Tidying up.

2005-05-07  Max Bowsher  <maxb@ukf.net>

* download.cc (validateCachedPackage): Remove all MD5 checking code.
It is not acceptable to check MD5 sums as part of simply answering
whether a package exists in the local cache or not. It is too slow.
MD5 sums always have been - and will continue to be - checked at
install time.
(NoMD5Option): Remove this option, as the code it controls has been
removed.

2005-05-07  Max Bowsher  <maxb@ukf.net>

* install.cc (md5_one): Make the MD5 checking done at install time
display a progess GUI, as the MD5 checking done at other times does.

2005-05-07  Max Bowsher  <maxb@ukf.net>

* resource.h, res.rc: Followup to recent de-MSVC-ification of
resource.h, by removing a lot from res.rc, and the last few bits
from resource.h.

Fix typo in ChangeLog.

2005-05-05  Brian Dessent  <brian@dessent.net>

* choose.cc (ChooserPage::OnInit): Initialize tooltips for this
dialog.

* proppage.h (PropertyPage::makeClickable): Update prototype to
reflect same argument name as function.

* proppage.cc (PropertyPage::DialogProc): Remove redundant 'break'
statements that follow 'return's.  Add missing braces to case
statements.  Call handler for TTN_GETDISPINFO.

* res.rc (IDS_TRUSTKEEP_TOOLTIP): Add new STRINGTABLE entry for
tooltip text.
(IDS_TRUSTPREV_TOOLTIP): Ditto.
(IDS_TRUSTCURR_TOOLTIP): Ditto.
(IDS_TRUSTEXP_TOOLTIP): Ditto.
(IDS_TRUSTPREV_TOOLTIP): Ditto.
(IDS_VIEWBUTTON_TOOLTIP): Ditto.

* resource.h: Likewise, add identifiers for tooltip resources.

* window.h: Add includes.
(Window::TooltipHandle): Add new private data structure.
(Window::TooltipStrings): Ditto.
(Window::ActivateTooltips): Define new function to support tooltip
functionality.
(Window::SetTooltipState): Ditto.
(Window::AddTooltip): Ditto.
(TooltipNotificationHandler): Ditto.

* window.cc (Window::Window): Initialize handle to NULL.
(Window::~Window): Call DestroyWindow on tooltip control if it was
activated.
(Window::ActivateTooltips): New function to activate the tooltip
control.
(Window::SetTooltipState): New function to allow toggling of
tooltips.
(Window::AddTooltip): New function that adds a tooltip to a given
control in a given window.  Also implement overloaded versions that
provide simpler interface and the ability to use a string resource.
(Window::TooltipNotificationHandler): New function to handle
TTN_GETDISPINFO notification messages.

2005-05-05  Brian Dessent  <brian@dessent.net>

* resource.h: Reorganize.  Group by type, and give each its own
range of IDs.  Regenerate IDs accordingly.
(IDD_CHOOSER): Remove obsolete and unused symbol.

2005-05-05  Brian Dessent  <brian@dessent.net>

* root.h (class RootPage): Add missing virtual to OnMessageCmd.

2005-05-05  Brian Dessent  <brian@dessent.net>

* root.cc (dialog_cmd): Remove.  Use OnMessageCmd instead.
(RootPage::Create): Accomodate above change.

* root.h (class RootPage): Add prototype for OnMessageCmd.

* res.rc (IDD_ROOT): Fix broken button groups.  Correct the order of
the controls and use of WS_GROUP so that the radio buttons act
correctly.

2005-05-05  Max Bowsher  <maxb@ukf.net>

Bulk rename of String++ member cstr_oneuse () to c_str () for
similarity to std::string.

* Exception.cc
* IniDBBuilderPackage.cc
* IniParseFindVisitor.cc
* LogFile.cc
* PickCategoryLine.cc
* PickPackageLine.cc
* PickView.cc
* String++.cc
* String++.h
* archive.cc
* cygpackage.cc
* desktop.cc
* dialog.cc
* diskfull.cc
* download.cc
* filemanip.cc
* find.cc
* geturl.cc
* ini.cc
* install.cc
* io_stream.cc
* io_stream_cygfile.cc
* io_stream_file.cc
* localdir.cc
* mount.cc
* nio-http.cc
* package_db.cc
* package_meta.cc
* package_source.h
* postinstall.cc
* proppage.cc
* root.cc
* script.cc
* site.cc
* splash.cc
* threebar.cc
* window.cc

Correct formatting of recent ChangeLog entries.

2005-05-05  Max Bowsher  <maxb@ukf.net>

Some String++ refactoring, setting out in the general direction of
eventually just using the STL std::string, not our own class.

* String++.cc (String::cstr, String::cstr () const): Remove.
(new_cstr_char_array): New function.
* String++.h: Sync with above.
Remove irrelevant "class io_stream;" declaration.

* cygpackage.h: Add "class io_stream;" declaration, now no longer
being supplied by side-effect.

* filemanip.cc: Remove unneeded includes, add "using namespace std;".
(parse_filename): Exchange some awkward pointer manipulation for
std::string usage. Use new_cstr_char_array ().
* filemanip.h: Sync with above.
(fileparse): Improve style of declaration.

* io_stream_cygfile.cc (io_stream_cygfile::normalise):
Use new_cstr_char_array ().
* site.cc (site_list_type::init, get_site_list): Ditto.

2005-05-05  Brian Dessent  <brian@dessent.net>

* ControlAdjuster.h: (enum ControlPosition): Declare new
resizing methods CP_STRETCH_LEFTHALF and CP_STRETCH_RIGHTHALF, and
their TOP/BOTTOM aliases.

* ControlAdjuster.cc: (ControlAdjuster::ControlInfo::adjust):
Implement CP_STRETCH_LEFTHALF and CP_STRETCH_RIGHTHALF, which
are designed to resize elements in a two-column layout.

* localdir.h: (class LocalDirPage): Remove constructor
implementation from here...

* localdir.cc: (LocalDirPage::LocalDirPage): ...to here, and call
AddControlsInfo on initialization.
(static LocaldirControlsInfo): Add.  Define sizing rules for dialog
elements.

* proppage.h: Add includes.  (PropertyPage::ClickableURL): Declare new
storage type for internal use by class. (PropertyPage::urls):
Likewise.  (PropertyPage::urlWinProc): Add declaration.
(PropertyPage::makeClickable): Likewise.

* proppage.cc: Add includes.  (PropertyPage::DialogProc): Implement
handler for WM_CTLCOLORSTATIC message.  Supports static controls that
have been turned into clickable hyperlinks. (static
PropertyPage::urls): Declare.  (PropertyPage::makeClickable):
Implement new member function that provides user interface for making
text into clickable URLs.  (PropertyPage::urlWinProc): Implement new
private member function used as a windowproc for subclassing a static
control.

* resource.h: (IDC_SPLASH_URL, IDC_FILEMODES_LINK, IDC_ROOTDIR_GRP,
IDC_INSTALLFOR_GRP, IDC_ALLUSERS_TEXT, IDC_JUSTME_TEXT, IDC_MODE_GRP,
IDC_MODE_TEXT, IDC_MODE_BIN, IDC_LOCALDIR_GRP, IDC_SPLASH_TEXT,
IDC_SPLASH_ICON, IDC_SPLASH_COPYR): Add. Replaces anonymous
IDC_STATIC ids, so that controls can be referenced by the resizer.

* res.rc: (IDD_LOCAL_DIR): Rearrange controls to match top-down order
in the dialog.  Fold long lines.  Replace anonymous IDC_STATIC ids.
(IDD_ROOT): Reorder in same manner.  Replace anonymous IDC_STATIC
ids.  Add descriptive text of each option, and hyperlink for more
information.  Mark suggested choice as "(RECOMMENDED)".  Swap order
of 'unix'/'dos' radio buttons so that recommended choice is first in
both columns.  (IDD_SPLASH): Reorder elements to match top-down
order of dialog, fold long lines. Add descriptive paragraph, remove
anonymous ids.  Update copyright text, replace sources.redhat.com
with cygwin.com.

* root.h: (class RootPage): Remove constructor implementation from
here...

* root.cc: ...to here, and call AddControlInfo.  (static
RootControlsInfo): Add.  Populate with resizing data.
(RootPage::OnInit): Make text a clickable hyperlink.

* splash.h: (class SplashPage): Likewise, move constructor from
here...

* splash.cc: ...to here.  Call AddControlInfo.  (static
SplashControlsInfo): Add and populate.  (SplashPage::Create): Make
link on splash page a clickable hyperlink.

* window.h: Add include.  (class Window): Store allocated fonts with a
vector instead of fixed array.  Change indentation of setParent to be
consistent.

* window.cc: (Window::Window): Remove FontCounter.
(Window::~Window): Likewise.  Iterate over vector of fonts
instead. (Window::SetDlgItemFont): Use vector.

2005-05-05  Max Bowsher  <maxb@ukf.net>

* Makefile.am (EXTRA_DIST, inilint_SOURCES):
Sync with several recent changes.

Fix wrong date in ChangeLog.

Remove spurious CR characters from ChangeLog.

2005-04-05  Max Bowsher  <maxb@ukf.net>

* Makefile.am (EXTRA_DIST): Add bootstrap.sh.

2005-05-05  Brian Dessent  <brian@dessent.net>

* res.rc: (CREATEPROCESS_MANIFEST_RESOURCE_ID): Include section
comment.

2005-05-05  Brian Dessent  <brian@dessent.net>

* res.rc: (IDC_CHOOSE): Widen 'view' button by 6px horizontally and
4px vertically.  Adjust other controls to accomodate.
(CREATEPROCESS_MANIFEST_RESOURCE_ID): Define so that manifest is
included in the binary.

* setup.exe.manifest: New file.  Results in common controls with
themed appearance on modern versions of Windows.

2005-05-05  Brian Dessent  <brian@dessent.net>

* main.cc (WinMain): Change conditional to avoid spurious error
message on success.  Include reason for failure in log message.

2005-05-04  Max Bowsher  <maxb@ukf.net>

* state.cc, state.h (net_method, net_proxy_host, net_proxy_port,)
(net_user, net_passwd, net_proxy_user, net_proxy_passwd,)
Inet_ftp_user, net_ftp_passwd):
Move these global variables to be static variables of the NetIO class.

* netio.h, netio.cc (net_method, net_proxy_host, net_proxy_port,)
(net_user, net_passwd, net_proxy_user, net_proxy_passwd,)
net_ftp_user, net_ftp_passwd):
As above.

* ConnectionSetting.cc, net.cc:
Change include, refer to variables in new location.

2005-05-04  Max Bowsher  <maxb@ukf.net>

* PickLine.cc: Remove file - redundant.
* Makefile.am (setup_SOURCES): Adjust.

2005-05-04  Max Bowsher  <maxb@ukf.net>

* package_version.h (packageversion::set_requirements):
Remove deftrust argument default value.

2005-05-03  Max Bowsher  <maxb@ukf.net>

* package_version.cc (DependencyProcessor): Remove trivial class.
(select, processOneDependency): Do not use DependencyProcessor, just
pass parameters normally.

2005-05-04  Brian Dessent  <brian@dessent.net>

Fix bug #820: Clicking Curr, etc., causes some dependencies to not
be installed.

* choose.cc (ChooserPage::changeTrust):
Add a missing "packagedb.markUnVisited ()".

Commit missed ChangeLog entry.

2005-05-03  Max Bowsher  <maxb@ukf.net>

* PickCategoryLine.cc (PickCategoryLine::click):
* PickPackageLine.cc (PickPackageLine::click):
Abandon use of trivial packagemeta::PrepareForVisit() wrapper.

* package_meta.h (packagemeta::PrepareForVisit): Remove.
* package_meta.cc (packagemeta::PrepareForVisit): Remove.

2005-05-03  Max Bowsher  <maxb@ukf.net>

* choose.h (ChooserPage::ifChecked): Remove.
(ChooserPage::changeTrust): Convert from template to ordinary function.
* choose.cc (ChooserPage::changeTrust): As above, plus avoid the use
of custom class SetRequirement, when bind2nd(mem_fun(...),...) can
achieve the same effect.
* package_meta.h (SetRequirement): Remove, now unused.

2005-05-03  Max Bowsher  <maxb@ukf.net>

* package_meta.h (packagemeta::set_requirements):
Remove default argument which seems inappropriate.
* PickPackageLine.cc (PickPackageLine::click):
Make an argument explicit, and not that it seems like the wrong value.

* README: Describe setup's use of branches.

2005-05-03  Brian Dessent  <brian@dessent.net>

Fix part of bug #853 - proxy port not remembered.

* net.cc (doing_loading): New static boolean.
(load_dialog): Use the boolean to prevent save_dialog() from taking
action due to field changes due to initialization.
(save_dialog): Check the boolean and respond appropriately.

2005-04-19  Max Bowsher  <maxb@ukf.net>

* postinstall.cc (RunFindVisitor::visitFile): Fix crash on
encountering a postinstall script with a name of 4 characters or less.
Bug introduced on 2004-12-25.

2005-04-05  Max Bowsher  <maxb@ukf.net>

The Big Cleanup: Phase "rfc1738"

* rfc1738.cc, rfc1738.h: Delete.
* csu_util/rfc1738.cc: Reimport latest version from upstream source.
Make thread-safe. Rewrite C++ wrapper in terms of std::string.
Document the divergences from the upstream source.
* csu_util/rfc1738.h: Recreate for the std::string version of the
C++ wrapper.

* Makefile.am (setup_SOURCES): Sync with added/removed files.
* IniParseFindVisitor.cc: Modify #includes.
(IniParseFindVisitor::visitFile): Change function name, as the new
C++ wrapper does not contain an irregularity that the old one did.
* download.cc: Modify #includes.
* ini.cc: Modify #includes.

2005-04-05  Max Bowsher  <maxb@ukf.net>

Remove setup's internal compression libraries, instead relying on the
mingw-foo packages which have been part of the Cygwin net distribution
for some time now.  At the same time, reorganize the README file to
inform about the change, and make some additional tidy-ups there whilst
working there.

* GUIDELINES: Remove, contents transferred to README.
* README: Add build instructions. Delete unconfirmed and fixed bugs
  sections. Incorporate the former contents of GUIDELINES.

* bz2lib/: Remove.
* zlib/: Remove.
* Makefile.am (DIST_SUBDIRS, SUBDIRS):
* configure.in: Remove AC_CONFIG_SUBDIRS for removed subdirs.
* compress_bz.h: Remove complex _WIN32 preprocessor conditional.
* compress_gz.h: Fix #include file name.
* archive_tar_file.cc: Remove unused #include.
* install.cc: Remove unused #include.

* CHANGES: Update.

* cfgaux/.direxists: Create, to work around obscure CVS bug.

2005-01-27  Max Bowsher  <maxb@ukf.net>

Set CYGWIN=nontsec, for postinstall scripts.

* main.cc (main): putenv("CYGWIN=nontsec").

2005-01-27  Bas van Gompel  <patch-cygsup.buzz@bavag.tmfweb.nl>

Fix corrupt display of ini parse percentage in log file.

* ini.cc (GuiParseFeedback): Use a sensible data type for lastpct.

2004-12-29  Max Bowsher  <maxb@ukf.net>

* site.cc (get_site_list): Filter the incoming mirrors list to contain
only URL schemes that we can read.

* CHANGES: Update in preparation for new snapshot.

2004-12-28  Max Bowsher  <maxb@ukf.net>

The Big Cleanup: Phase "hash"

* hash.cc, hash.h: Delete.
* Makefile.am (setup_SOURCES): Sync with deleted files.

* package_mets: Modify includes.
(standard_dirs): Remove.
(hash::add_subdirs): Remove.
(packagemeta::uninstall): Re-implement using STL set instead of
homebrew hash.

2004-12-28  Max Bowsher  <maxb@ukf.net>

* String++.h (operator std::string): Take care to never do
std::string((char*)NULL) - it causes a hang!

Fix error in ChangeLog.

2004-12-27  Max Bowsher  <maxb@ukf.net>

The Big Cleanup: Phase "log"

* log.cc, log.h: Delete old C-style compatibility wrapper around the
logging subsystem.
* Makefile.am (setup_SOURCES): Sync with deleted files.

* AntiVirus.cc, archive_tar_file.cc, ini.cc, install.cc,
* io_stream_cygfile.cc, localdir.cc, msg.cc, net.cc, netio.cc,
* netio.h, nio-ftp.cc, nio-ftp.h, root.cc, source.cc:
Throughout, refine includes, and update code using the old
compatibility wrapper to C++ streams style code.

2004-12-27  Max Bowsher  <maxb@ukf.net>

* compress.cc: Refine includes. Remove placeholder function
definitions. Make destructor an empty function.
* compress.h: Refine includes. Make placeholder functions pure
virtual. Add missing const on buffer argument to write function,
to match derived classes.
* compress_bz.cc, compress_gz.cc: Refine includes. Throw exception,
rather than logging and returning an error, when an unimplemented
function is called.
* compress_bz.h, compress_gz.h: Refine includes.

2004-12-26  Max Bowsher  <maxb@ukf.net>

The Big Cleanup: Phase "version"

* String++.h (operator std::string): Add.
* desktop.cc, fromcwd.cc: Remove irrelevant include.

* version.cc, version.h: Delete, replacing all functionality in new
files.

* setup_version.h: Add, taking over declaration of generated version
symbol functionality.

* Makefile.am (setup_SOURCES): Sync with new/deleted files.
(setup_version.c): Rename the symbol 'version' to 'setup_version'.
* main.cc: Modify includes.
(main): Adapt to setup_version rename.
* splash.cc: Modify includes.
(SplashPage::OnInit): Adapt to setup_version rename.

* csu_util/version_compare.cc, csu_util/version_compare.h: Add,
containing a completely rewritten replacement for canonical_version().

* IniDBBuilderPackage.cc: Modify includes.
(IniDBBuilderPackage::buildVersion): Use new version_compare API.
* ini.cc: Modify includes.
(setup_version): Rename...
(ini_setup_version): ...to this.
(do_local_ini, do_remote_ini): Adapt to above change.
(do_ini_thread): Adapt to above change. Use new version_compare API.

2004-12-25  Max Bowsher  <maxb@ukf.net>

* install.cc (Installer::installOneSource): Remove a stray "break;"
left over from the switch-case -> if-else conversion in the previous
change.

Fix typo in comment.

2004-12-25  Max Bowsher  <maxb@ukf.net>

The Big Cleanup: Phase "win32"

* win32.h: Major overhaul. Document why we use _WIN32_IE. Move out
fragment dealing with alloca() to the single file in which it is used.
Remove fragment dealing with _MAX_PATH, and convert all uses of it
elsewhere to MAX_PATH. Remove random include of <stdarg.h>. Remove
NOCOMATTRIBUTE, as setup contains no code which it affects. Remove
includes of <wininet.h> and <windowsx.h> - see later in this log.
(IsWindowsNT): New inline function, a much simpler implementation of
Win32::OS().

* win32.cc: Delete.

* Makefile.am (setup_SOURCES): Remove "win32.cc".

* AntiVirus.cc (detect), PickPackageLine.cc (PickPackageLine::paint),
* install.c (Installer::installOneSource):
Use IsWindowsNT(), not Win32::OS().
* main.cc (main): Use IsWindowsNT(), not locally defined iswinnt.

* PickView.cc (PickView::WindowProc), netio.cc (auth_proc),
* proppage.cc (PropertyPage::DialogProc):
Avoid using obfuscating macros from <windowsx.h>.

* archive.cc: Include or #define alloca() in the only file in which
it is used.

* nio-ie5.h: Include <wininet.h> here, where it is actually needed,
rather than in "win32.h" which is included almost everywhere.

* archive_tar.cc, archive_tar.h, cygpackage.cc, cygpackage.h,
* desktop.cc, find.cc, install.cc, main.cc, mklink2.c, mount.cc,
* script.cc: Use MAX_PATH, not _MAX_PATH.

2004-12-25  Max Bowsher  <maxb@ukf.net>

Remove some overcomplications.

* Makefile.am (setup_SOURCES): Remove FilterVisitor.{cc,h}.
* FilterVisitor.cc, FilterVisitor.h: Delete.
* String++.cc (strmatch, String::matches): Remove.
* String++.h (String::matches): Remove.
* postinstall.cc: Do not include "FilterVisitor.h".
(RunFindVisitor): Exclude items with the extension ".done".
(do_postinstall_thread): Remove overcomplicated exclusion of items
with the extension ".done".

2004-12-24  Max Bowsher  <maxb@ukf.net>

* String++.cc, String++.h: Remove unused and incomplete code.
* LogFile.cc (LogFile::log_save): Don't leak memory.
* filemanip.cc, geturl.cc, root.cc, version.cc:
Don't pointlessly create temporary copies strings.

* CHANGES: New file.

2004-12-23  Max Bowsher  <maxb@ukf.net>

* choose.cc (ChooserPage::OnNext): Relegate some overly verbose
logging to an #ifdef DEBUG section.
* geturl.h (dismiss_url_status_dialog): Remove stale function
declaration, for which there is no definition.
* io_stream_cygfile.cc (io_stream_cygfile::io_stream_cygfile):
Motivated by the recent unexplained hang bug, add some extra logging.

2004-11-22  Max Bowsher  <maxb@ukf.net>

* MD5++.cc, MD5++.h: Remove this class.
* csu_util/MD5Sum.cc, csu_util/MD5Sum.h: New more complete MD5 class.
* Makefile.am (setup_SOURCES): Adjust to new files.
* download.cc (validateCachedPackage): Simplify, using the new class.
* install.cc (md5_one): Ditto.
* package_source.h (packagesource::md5): Change to be an object of the
new class.

2004-11-22  Max Bowsher  <maxb@ukf.net>

* main.cc (main): Fix bug in one of the 2004-11-19 commits.
Avoid referring to the theLog pointer before it is set.

2004-11-20  Max Bowsher  <maxb@ukf.net>

* res.rc (IDD_SOURCE): Clarify the radio-button option descriptions.
Re-layout the radio buttons.

2004-11-20  Max Bowsher  <maxb@ukf.net>

Repair the broken detection of wrongly-sized package files.

* download.cc (validateCachedPackage): Make static, since not used
elsewhere. Improve logging. Migrate file-size tests to here, alongside
the MD5 tests.
(check_for_cached): Remove broken testing of file-size, now covered by
the above. Code cleanups.

2004-11-19  Max Bowsher  <maxb@ukf.net>

* package_version.cc (dumpAndList): Make dependencies much more
readable in full log output.

2004-11-19  Max Bowsher  <maxb@ukf.net>

* proppage.cc (PropertyPage::DialogProc): Wrap in a TOPLEVEL_CATCH
block, since allowing exceptions to unwind into the guts of Windows is
quite disasterous.

2004-11-19  Max Bowsher  <maxb@ukf.net>

* main.cc (main): Move option parsing code earlier - it must execute
before any significant object creations, which might test options in
their initialization.

2004-11-19  Max Bowsher  <maxb@ukf.net>

* main.cc (main): Remove some junk code - either commented out, or
contained within unreachable preprocessor conditional sections.

2004-11-19  Max Bowsher  <maxb@ukf.net>

* install.cc (Installer::installOneSource): Avoid leaking the open
file handles of every tarball that is installed.

2004-11-19  Max Bowsher  <maxb@ukf.net>

* package_source.h (packagesource::Canonical, packagesource::Base)
(packagesource::Filename): Add "const" specification to accessor
methods.

2004-11-18  Max Bowsher  <maxb@ukf.net>

* archive.h (archive::~archive): Define a pure virtual destructor to
enforce the abstract nature of this class.
* archive.cc (archive::~archive): Pure virtual destructors require an
implementation! Add one.
* io_stream.h (io_stream::destroyed): Remove this severely
brain-damaged implementation of a weakly-enforced abstract class.
The pure virtual destructor was strongly enforcing the abstract nature
anyway.
* io_stream.cc (io_stream::~io_stream): Empty out horribly pointless
code from the pure virtual destructor.
* archive_tar.cc (archive_tar::~archive_tar): Remove all
reference to the removed "destroyed" member.
* archive_tar_file.cc (archive_tar_file::~archive_tar_file): Ditto.
* compress_bz.cc (compress_bz::~compress_bz): Ditto.
* compress_gz.cc (compress_gz::~compress_gz): Ditto.
* io_stream_cygfile.cc (io_stream_cygfile::~io_stream_cygfile): Ditto.
* io_stream_file.cc (io_stream_file::~io_stream_file): Ditto.
* io_stream_memory.cc (io_stream_memory::~io_stream_memory): Ditto.

Commit missed ChangeLog entry

2004-11-17  Max Bowsher  <maxb@ukf.net>

* mount.cc (create_mount): Fix operator precedence error introduced in
first 2004-11-11 commit - warned of by gcc 3.3.3 but not 3.4.1 !

2004-11-13  Max Bowsher  <maxb@ukf.net>

Miscellaneous cleanup.

* geturl.cc (getUrlToStream): Make static.
* geturl.h (getUrlToStream): Remove declaration.

* geturl.cc, geturl.h, netio.cc, netio.h, nio-ftp.cc, nio-ftp.h,
nio-http.cc, nio-http.h:
Remove unused "allow_ftp_auth" flag throughout.

2004-11-11  Max Bowsher  <maxb@ukf.net>

* libmd5-rfc: New directory, to hold code imported from the libmd5-rfc
project in a clearly delineated fashion.
* libmd5-rfc/README: New file.
* md5.c, md5.h: Delete...
* libmd5-rfc/md5.c, libmd5-rfc/md5.h: ...and replace with pristine
versions from upstream (only change is restoration of upstream CVS
keyword expansions).
* libmd5-rfc/md5main.c: New file.
* Makefile.ami (setup_SOURCES): Edit file paths.
* download.cc, install.cc: Edit include paths.

2004-11-11  Max Bowsher  <maxb@ukf.net>

* dialog.cc (fatal): Make capable of handling a passed-in error code,
as an alternative to calling GetLastError().
* dialog.h (fatal): Adjust prototype as above.
* mount.cc (create_mount): Use the above to correctly report errors.

2004-10-28  Max Bowsher  <maxb@ukf.net>

* LogFile.cc (LogFile::getFileName), splash.cc (SplashPage::Create),
threebar.cc (ThreeBarProgressPage::SetBar2):
Migrate from cistring and string resources
to String and string literals.
* localdir.cc: Remove unneeded #include of "cistring.h".
* cistring.cc, cistring.h: Remove this now-redundant class.
* Makefile.am: Remove mention of the above removed files.
* res.rc, resource.h: Remove obsolete strings and IDs, both those
obseleted by and obselete before the above changes.
* res.rc: Merge the 2 adjacent STRINGTABLEs.

2004-10-27  Max Bowsher  <maxb@ukf.net>

* Makefile.am (setup_SOURCES): Add "RECTWrapper.h". It was missing.

2004-10-25  Bas van Gompel  <cygsup-patch.buzz@bavag.tmfweb.nl>

* package_source.cc (packagesource::set_canonical):
Fix one-off in 'base' when ini is in same dir as package.

2004-10-25  Max Bowsher  <maxb@ukf.net>

* main.cc (next_dialog): Remove this legacy global variable.
* download.cc (do_download_thread): Return a value instead of setting
a global variable, then returning void.
(do_download_reflector): Look for the next dialog code as a return
value instead of a global variable.

2004-10-25  Max Bowsher  <maxb@ukf.net>

* port.h: Remove this redundant file - all contained definitions are
duplicated in "win32.h".
* Makefile.am, UserSetting.cc, UserSettings.cc, archive.cc,
archive_tar.cc, archive_tar_file.cc, choose.cc, compress.cc,
compress_bz.cc, compress_gz.cc, desktop.cc, find.cc, fromcwd.cc,
iniparse.y, install.cc, io_stream_cygfile.cc, io_stream_file.cc,
main.cc, netio.cc, site.cc, version.cc:
Remove all references to "port.h".

2004-10-25  Max Bowsher  <maxb@ukf.net>

Fix compilation with w32api 3.1 and later.

* ControlAdjuster.h, RECTWrapper.h, cistring.h, propsheet.h,
window.cc, window.h: Include "win32.h" not <windows.h> directly.
* win32.h (WINBASEAPI): Define to empty string.

2004-09-13  Igor Pechtchanski  <pechtcha@cs.nyu.edu>

* script.cc (run): Kill stdin when running scripts.

2004-09-01  Max Bowsher  <maxb@ukf.net>

* Makefile.am: Oops. Unwanted keyword expansion ate the code of the
previous commit. Restore the code, escaped to not look like a keyword.

2004-09-01  Max Bowsher  <maxb@ukf.net>

* Makefile.am: Simplify the shell incantation for getting the revision
of ChangeLog from grep-and-sed to just sed.

2004-08-31  Max Bowsher  <maxb@ukf.net>

Attain GCC 3.4 compatibility.

* PackageSpecification.h (PackageSpecification): Make the
unnecessarily private copy-constructor public.

2004-08-30  Max Bowsher  <maxb@ukf.net>

Rework the implementation of not including -Werror for specific files,
to a way that does not require hardcoding modified automake output back
into its input.

* Makefile.am (AM_CXXFLAGS, BASECXXFLAGS): Seperate out -Werror.
(autoload.o): Drop this rule, -finline-functions is no longer required
and -Werror is acceptable here now we have __attribute__ ((used)).
(WARNONLY_CFLAGS, inilex.o): Drop these.
(EXTRA_PROGRAMS): Add a fake "warnonly" program.
(warnonly_CXXFLAGS): Set to the $(BASECXXFLAGS).
(warnonly_SOURCES): Set to "inilex.cc".
(EXTRA_setup_SOURCES): Set to $(warnonly_SOURCES).
(setup_SOURCES): Remove "inilex.cc".
(setup_LDADD): Add "warnonly-inilex.o".

2004-08-30  Max Bowsher  <maxb@ukf.net>

Move closer to GCC 3.4 compatibility.

* IOStreamProvider.h, io_stream_cygfile.cc, io_stream_cygfile.h,
io_stream_file.cc: Do not incorrectly use "enum" after it has been
hidden inside a typedef.
* archive.h: In "virtual func() = 0;", the 0 is a language construct,
not some symbol that might evaluate to 0, and gcc 3.4 enforces this.
* autoload.c (DllInfo.name): Silence warning by making const.
(DLL, autoload_common): __attribute__ ((used)) is now required to
prevent gcc being over-eager in it's optimization, and removing these
seemingly-unused (used only from assembly) functions.
Silences some warnings, too.
* Makefile.am (AM_CXXFLAGS, AM_CFLAGS): The options
-Wstrict-prototypes -Wmissing-prototypes do not apply to C++ and
g++ 3.4 complains about them.

2004-08-29  Max Bowsher  <maxb@ukf.net>

* Exception.h: Reformatting, #include more, remove incorrect and
less-than-useful comments.
(TOPLEVEL_CATCH): New preprocessor macro.

* LogFile.cc (LogFile::operator()): Throw an exception object, not a
char* string.

Wrap the top-level of each thread (including the main thread) in a
try...catch block, in order to display a useful error message if an
exception is not caught.
* download.cc (do_download_reflector): Here ...
* ini.cc (do_ini_thread_reflector): ... and here ...
* install.cc (do_install_reflector: ... and here ...
* main.cc (main): ... and here ...
* postinstall.cc (do_postinstall_thread): ... and here ...
* site.cc (do_download_site_info_thread): ... and here!

* msg.h: Include a missing header requirement.
* res.rc (IDS_UNCAUGHT_EXCEPTION, IDS_UNCAUGHT_EXCEPTION_WITH_ERRNO):
New strings.
* resource.h (IDS_UNCAUGHT_EXCEPTION)
(IDS_UNCAUGHT_EXCEPTION_WITH_ERRNO): New IDs.

Add missing ChangeLog entry.

2004-08-28  Max Bowsher  <maxb@ukf.net>

        * res.rc (IDS_WINDOW_INIT_BADMSG): New string.
        * resource.h (IDS_WINDOW_INIT_BADMSG): New string resource ID.
        * window.cc (Window::FirstWindowProcReflector): Convert an abort()
        into a descriptive error.

2004-08-28  Max Bowsher  <maxb@ukf.net>

* Makefile.in, aclocal.m4, configure: Autoreconf. Fixes "make dist"
breakage with srcdir != objdir builds.