2002-01-22  Robert Collins  <rbtcollins@hotmail.com>

        * Makefile.in (OBJS): Remove supp.o - it appears accidental.
* choose.cc: Don't define alloca anymore, it's done in win32.h

2002-01-22  Robert Collins  <rbtcollins@hotmail.com>

        * PickPackageLine.cc: New file.
        * PickPackageLine.h: New file.
        * PickLine.cc: New file.
        * PickLine.h: New file.
        * PickCategoryLine.cc: New file.
        * PickCategoryLine.h: New file.
        * PickView.cc: New file.
        * PickView.h: New file.
        * Makefile.in: Add new objects to setup.exe.
        Backout -fno-exceptions option.
        * choose.cc: Include PickView.h.
        Remove view related defines and static variables.
        (_pkg_headers): Moved to PickView.cc.
        (_cat_headers): Ditto.
        (view::views::Unknown): Ditto.
        (view::views::PackageFull):
        (view::views::Package):
        (view::views::Category):
        (pkgtrustp): Moved to package_meta.cc.
        (add_required): Moved to package_meta.cc.
        (pick_category_line::empty): Moved to PickCategoryLine.cc
        (paint): Adjust for moved static variables.
        (view::scroll): Moved to PickView.cc.
        (list_vscroll): Adjust for moved static variables.
        (list_hscroll): Ditto.
        (list_click): Ditto.
        (note_width): Moved to PickView.cc.
        (view::view): Ditto.
        (view::set_view_mode): Ditto.
        (view::mode_caption): Ditto.
        (view::views::caption): Ditto.
        (view::set_headers): Ditto.
        (DoInsertItem): Ditto.
        (view::init_headers): Ditto.
        (view::insert_pkg): Ditto.
        (view::insert_category): Ditto.
        (view::clear_view): Ditto.
        (view::views::operator++): Ditto.
        (view::click): Ditto.
        (default_trust): Adjust for moved statics.
        (pick_pkg_line::paint): Moved to PickPackageLine.cc.
        (pick_pkg_line::click): Ditto.
        (pick_category_line::actiontext): Moved to PickCategoryLine.cc
        (pick_category_line::paint): Ditto.
        (pick_category_line::click): Ditto.
        (set_view_mode): Adjust for moved statics.
        (create_listview): Ditto.
        * choose.h: Remove Category and packagemeta forward defines - not needed.
        Don't include unneeded headers list and package_meta.
        (_header): Move to PickView.h.
        (pick_line): Move to PickLine.h.
        (pick_pkg_line): Move to PickPackageLine.h.
        (pick_category_line): Move to PickCategoryLine.h.
        (view): Move to PickView.h.
        * ini.h (trusts): Move to package_meta.h.
        * package_meta.h (trusts): New enum.
        (packagemeta::set_requirements): New helper method.
        (packagemeta::trustp): Ditto.

* choose.h (view): Move forward declaration of views into public area or g++ v3
will complain.

* Makefile.in (CXXFLAGS): Add -fno-exceptions.
* win32.h (alloca): Define as __builtin_alloca.
* io_stream_memory.cc: Add include file.

2002-01-21  Robert Collins  <rbtcollins@hotmail.com>

        * archive_tar.cc (archive_tar::~archive_tar): Mark the stream as destroyed.
        * archive_tar_file.cc (archive_tar_file::~archive_tar_file): Ditto.
        * compress.cc (compress::~compress): Be less verbose.
        * compress_bz.cc (compress_bz::~compress_bz): Ditto.
        Mark the stream as destroyed.
        * compress_gz.cc (compress_gz::~compress_gz): Mark the stream as destroyed.
        * io_stream.cc (io_stream::~io_stream): Only warn when the stream is not marked
        as destroyed.
        * io_stream_cygfile.cc (io_stream_cygfile::~io_stream_cygfile): Mark the
        stream as destroyed.
        * io_stream_file.cc (io_stream_file::~io_stream_file): Ditto.
        * io_stream_memory.cc (io_stream_memory::~io_stream_memory): Ditto.

2002-01-21  Robert Collins  <rbtcollins@hotmail.com>

        * Makefile.in (CFLAGS): Allow customisable malloc debug flags.
        (OBJS): Link in malloc objects.
        * category.cc (category): Use new char, not strdup.
        * choose.cc (pick_pkg_line::paint): Allow for the regionsize to shrink.
        (do_choose): Use new char, not malloc.
        * concat.cc (vconcat): Ditto.
        * cygpackage.cc (cygpackage): Use new char, not strdup.
        (set_canonical_version): Ditto.
        (cygpackage::destroy): Use delete[], not free.
        * desktop.cc (uexists): Ditto.
        * dialog.cc (egest): Use new char, not strdup.
        * dlmalloc.c: New file - Doug Lea's malloc 2.7.0
        * hash.cc (hash::hash): Use delete instead of free.
        (hash::add): Use new char, not strdup.
        * ini.cc (do_remote_ini): Use delete[] not free.
        * inilex.l: Use new char, not strdup.
        (ini_init): Ditto.
        * iniparse.y: Ditto.
        Use a local package_db rather than a static one.
        * install.cc (struct mallinfo): Workaround an apparent dlmalloc bug.
        (do_install_thread): Call mallinfo to force a consistency check. Appears
        to fix a 'inuse' error in -DDEBUG builds.
        Use delete[] for strings.
        * io_stream_cygfile.cc (io_stream_cygfile::io_stream_cygfile): Use new char,
        not strdup.
        (io_stream_cygfile::~io_stream_cygfile): Use delete[], not free.
        (io_stream_cygfile::remove): Use new char, not strdup.
        * io_stream_file.cc (io_stream_file::io_stream_file): Use new char, not strdup.
        (io_stream_file::io_stream_file): Use delete[], not free.
        (io_stream_file::remove): Use new char, not strdup.
        * io_stream_memory.cc (memblock::~memblock): Use delete[], not free.
        * io_stream_memory.h (memblock): Use new char, not malloc.
        * localdir.cc (LocalDirPage::OnInit): Use new char, not strdup.
        * main.cc (WinMain): Use new char, not strdup.
        * mount.cc (find2): Use new char, not malloc.
        (read_mounts): Ditto.
        (cygpath): Use new char, not strdup.
        * netio.cc (NetIO::~NetIO): Use delete[], not free.
        (NetIO::set_url): Use new char, not strdup.
        (NetIO::get_ftp_auth): Use delete[], not free.
        * nio-ftp.cc (NetIO_FTP::NetIO_FTP): Ditto.
        Use new char instead of strdup.
        * nio-http.cc (base64_encode): Use new char, not malloc.
        * package_meta.cc (hash::add_subdirs): Use new char instead of strdup.
        * package_source.h (~ packagesource): Use delete[] instead of delete.
        * port.h (strdup): Deprecate the use of strdup.
        * rfc1738.cc (rfc1738_do_escape): Use new char, not calloc.
        * script.cc (init_run_script): Us delete[], not free.
        (run_script): Ditto.
        * simpsock.cc (SimpleSocket::fill): Use new char, not malloc.
        (SimpleSocket::invalidate): Use delete[], not free.
        * site.cc (site_list_type::init): Use new char instead of strdup.
        * site.h (~site_list_type): Use delete[], not free.

and in zlib/

2002-01-21  Robert Collins  <rbtcollins@hotmail.com>

        * zutil.c (z_error): Const correctness change.
        * zutil.h (z_error): Ditto.

2002-01-20  Robert Collins  <rbtcollins@hotmail.com>

        * Makefile.in (OBJS): Add win32.o - win32 support functions.
        * README: Update todos.
        * archive.cc (extract_file): Support a suffix for appending to extracted files.
        * archive.h: Ditto.
        * choose.cc (set_view_mode): Use the new view::views class.
        (view::views::Unknown): New static for defaulting view::views variables.
        (view::views::PackageFull): New static for the current view.
        (view::views::Package): Ditto.
        (view::views::Category): Ditto.
        (topbucket::paint): Remove.
        (topbucket::empty): Rename to pick_category_line::empty.
        (topbucket::~topbucket): Remove.
        (paint): Use the new view::views class.
        Don't paint 'all' as a category when showing package categoies.
        (pick_category_line::actiontext): New method.
        (pick_category_line::paint): Make showing the category optional, and show the
        'category action'.
        (pick_category_line::click): Make showing the category optional, and
        differentiate between the name being clicked, and the action description.
        (view::view): Use the new view::views class.
        (view::set_view_mode): Ditto.
        (view::mode_caption): Ditto.
        (view::views::caption): New method.
        (view::set_headers): Use the new view::views class.
        (view::clear_view): Ditto.
        (viewsplusplus): Replaced by
        (view::views::operator++): New operator.
        (set_view_mode): Use the new view::views class.
        (create_listview): Ditto.
        (dialog_cmd): Ditto.
        * choose.h (actions): Remove.
        (views): Remove.
        (pick_line): Add a new convenience constructor.
        (top_bucket): Remove.
        (pick_category_line): Reparent under pick_line, and adsorb top_bucket.
        (view::views): New class.
        * ini.h: Don't include choose.h
        (is_download_action): Remove.
        (is_upgrade_action): Remove.
        (is_uninstall_action): Remove.
        (is_full_action): Remove.
        * install.cc (num_replacements): New static.
        (rebootneeded): Ditto.
        (replace_one): Handle upgrades separate from removal or new installs.
        (install_one_source): Handle in use files.
        (install_one): Don't repeat upgrades.
        (do_install_thread): Initialize new statics.
        Order actions as uninstall, replace, install.
        Warn about rebooting if needed.
        * res.rc (IDS_REBOOT_REQUIRED): New string.
        * resource.h (IDS_REBOOT_REQUIRED): Give value.
        * win32.h: Declare class Win32, and update copyright and intent message.
        * win32.cc: New file.

* io_stream.cc (io_stream::gets): Eliminate trailing new line chars.
* package_meta.cc (packagemeta::uninstall): Check for additional
Windows shortcut.  Unset R/O file attribute before trying to delete
file.

2002-01-09  Robert Collins  <rbtcollins@hotmail.com>

        * choose.cc (pkgtrustp): Fall back to installed if a package is not present in
        the current trust level. For prev, fall back via curr.
        (set_existence): Allow sources with no sites, but cached files to be
        kept.
        (pick_category_line::paint): Start drawing clickable categories.
        * fromcwd.cc (found_file): Reinstate local file scanning.

2002-01-07  Gary R. Van Sickle  <g.r.vansickle@worldnet.att.net>

        * fromcwd.cc: Run indent.
        (do_fromcwd): Reverse sense of comment.  Set next_dialog
        to IDD_CHOOSER instead of IDD_CHOOSE.
        * ini.cc (find_routine): Remove "/" from "/setup.ini".

2001-01-07  Robert Collins  <rbtcollins@hotmail.com>

        * category.cc (Categorycmp): Add a const safe version.
        * category.h (Categorycmp): Add a const safe version.
        (Category): Add operator ==.
        * choose.cc (fill_missing_category): Add every package to "All".
        (pick_category_line::paint): Add support for hidden labels, and a tree depth.
        (pick_category_line::click): Ditto.
        (view::view): Pass in a top level category to use.
        (view::insert_pkg): Special case - skip category "All".
        (view::insert_category): Ditto.
        (view::clear_view): Set the contents label state.
        (create_listview): Pass in a top level category to view().
        * choose.h (topbucket): Make bucket available to derived classes.
        (pick_category_line):  Add support for hidden labels, and a tree depth.
        Remove our second copy of bucket.
        (view): Pass in a top level category to the constructor.
        Use pick_category_line to allow three level display.

2001-01-06  Robert Collins  <rbtcollins@hotmail.com>

        * cygpackage.h (cygpackage): Make set_[s|l]desc virtual.
        * iniparse.y (add_correct_version): Copy descriptions across.
        * package_meta.cc (SDesc): Iterate through versions, return the first with a
        description.
        * package_version.h (packageversion): Add new virtuals set_[l|s]desc.

2001-01-04  Robert Collins  <rbtcollins@hotmail.com>

        * script.cc (run_script): Change cygpath:// to cygfile://.
        * install.cc (do_install_thread): Fix off-by-one errors.

2001-01-04  Robert Collins  <rbtcollins@hotmail.com>

        * package_db.cc (packagedb::flush): Fix an off-by-one error.

remove accidentally commited file

2002-01-04  Jan Nieuwenhuizen  <janneke@gnu.org>

        * Makefile.in (realclean): more clean.
        (OBJS): Add script.o.
        * postinstall.cc (run_script_in_etc_postinstall): New function.
        (do_postinstall): Split off new funtion init_run_script ().
        (init_run_script):
        (run):
        (run_script): Move to script.cc.  Replace some deprecated remove
        and move calls.
        * script.h:
        * script.cc: New file.
        * Forward port cygwin-20010707.jcn3.patch.
        * package_meta.cc (uninstall): Run pre- and postremove scripts.
        * install.cc: Typo fix.
        (do_install): Run script initialisation.

2001-01-04  Gary R. Van Sickle  <g.r.vansickle@worldnet.att.net>

        * cistring.cc: Run d2u.
        * cistring.h: Run d2u.

        * desktop.cc (etc_profile): Remove "test -f ./.bashrc && . ./.bashrc"
        from the generated /etc/profile.  Bash will source this file
        automatically, and having this here merely results in .bashrc being
        executed twice. Run d2u.
        * desktop.h: Run d2u.

        * propsheet.cc: Run d2u.
        (PropSheetProc): New function.  Add minimize box here instead of in
        PropertyPage::DialogProc.
        (PropSheet::Create): Change to use creation callback PropSheetProc.
        (DLGTEMPLATEEX): Add 'hidden' Windows struct definition.

        * propsheet.h: Run indent, d2u.
        * proppage.h: Run indent, d2u.

        * proppage.cc: Run d2u.
        (PropertyPage::DialogProc): Remove minimize-box-adding
        functionality.  Remove commented-out "PropSheet_SetWizButtons" calls.
        Add support for calling virtual OnMessageCmd.  Add setting of fonts in
        WM_INITDIALOG handler.
        (resource.h): New include for resource IDs.

        * res.rc: Resize and rearrange property page dialog templates
        to bring them in line with "Microsoft's Backward Compatible Wizard 97"
        specification.  Run d2u.
        (IDD_SITE): Add an edit control and an "Add" button in order to
        combine the IDD_SITE and IDD_OTHER_URL functionality onto one page.
        (IDD_OTHER_URL): Remove dialog template.
        (IDD_DLSTATUS): Remove dialog template.

        * site.cc: Run d2u.
        (SitePage::OnBack): Remove NEXT() macro invocation.
        (SitePage::OnActivate): New member function.
        (load_dialog): Remove.  Functionality subsumed into
        SitePage::OnActivate.
        (save_dialog): Change to support both list and user URLs.  Remove
        OTHER_IDX and mirror_idx logic.
        (SitePage::PopulateListBox): New member function.
        (SitePage::CheckControlsAndDisableAccordingly): New member function.
        (SitePage::OnMessageCmd): New override.
        (check_if_enable_next): Remove.
        (dialog_cmd): Remove.
        (do_download_site_info_thread): Remove calls to NEXT() macro.
        (SitePage::Create): Call the single-param PropertyPage::Create
        overload.
        (other_url): New static taken from other.cc.
        (SitePage::OnNext): Remove mirror_idx logic.
        (SitePage::OnInit): Remove "Other URL" entry from list box.  Remove
        list box populating code, now handled in SitePage::PopulateListBox.
        (mirror_idx, NO_IDX, OTHER_IDX): Remove.
        (save_site_url): Fix potential buffer overflow problem.  Switched to
        TCHAR in grossly premature preparation for multilingual support.
        * site.h: Run d2u.
        (SitePage::OnActivate): New member function.
        (SitePage::CheckControlsAndDisableAccordingly) New member.
        (SitePage::OnMessageCmd): New override.
        (do_download_site_info_thread): Add MessageBox call on failure to
        download site list.

        * splash.cc: Run d2u.
        (SplashPage::OnInit): Set the font for the title.
        * splash.h: Run d2u.

        * threebar.cc: Run d2u.
        * threebar.h: Run d2u.

        * window.h: Run d2u.
        (Window::IsButtonChecked): New member function declaration.
        (Window::OnMessageCmd): New member function.
        (Window::SetDlgItemFont): New member function declaration.
        (Window::MAXFONTS, Window::Fonts, Window::FontCounter): New data
        members.
        * window.cc: Run d2u.
        (Window::IsButtonChecked): New member function definition.
        (Window::SetDlgItemFont): New member function definition.
        (Window::Window): Add initialization for FontCounter.
        (Window::~Window): Delete any fonts we created.

        * geturl.cc: Run d2u.
        (progress): Remove the "3" field width from the "%3d"
        percent-complete format indicator.  Causes line to not start at
        beginning of text box, and does little to help with "jumping", since
        the "bytes downloaded so far" field is variable-width anyway.  Change
        kb/s format field to "%03.1" to 0-pad the kb/s number in the event of
        painfully slow connections, or temporary slowdowns in faster
        connections should such more-instantaneous functionality become
        available.

        * net.h: Run d2u.
        (NetPage::OnMessageCmd): New member function declaration.
        (NetPage::CheckIfEnableNext): New member function declaration.
        * net.cc: Run d2u.
        (NetPage::OnMessageCmd): New member function definition.
        (dialog_cmd): Remove, subsumed into NetPage::OnMessageCmd.
        (check_if_enable_next): Remove.
        (NetPage::CheckIfEnableNext): New member function, subsumes
        check_if_enable_next.
        (propsheet.h): Add include.
        (NetPage::Init): Add call to CheckIfEnableNext.
        (load_dialog): Remove call to check_if_enable_next.
        (NetPage::Create): Call single-template-ID-parameter overload of
        PropertyPage::Create instead of three-parameter one.

        * Makefile.in (OBJS): Remove other.o.
        * other.cc: Remove file.

2002-01-01  Robert Collins  <rbtcollins@hotmail.com>

        * README: Update TODO's.
        * choose.cc (set_existence): Delete non installed packages with no
        mirrors.
        (fill_missing_category): Fix an off-by-one error.
        (default_trust): Ditto. Also delete any unused categories.
        (view::init_headers): Fix an off-by-one error.
        (set_view_mode): Ditto.
        (set_view_mode): Ditto.
        (create_listview): Ditto.
        (dialog_cmd): Ditto.
        (do_choose): Ditto.
        * io_stream.cc: Indent.
        * io_stream.h: Ditto.
        * package_db.cc: Ditto.
        * package_meta.h: Const correctness for SDesc ();
        * proppage.cc: Run d2u and indent.
        * propsheet.cc: Run d2u and indent.
        * window.cc: Run d2u and indent.
        * window.h: Run d2u and indent.

fix typo

* Makefile.in (iniparse.o): Accomodate newer bisons.

2001-12-22  Gary R. Van Sickle  <g.r.vansickle@worldnet.att.net>

* window.h (Window): New file, new class.
* window.cc (Window): New file, new class.

* threebar.h (ThreeBarProgressPage): New file, new class.
* threebar.cc (ThreeBarProgressPage): New file, new class.

* splash.h (SplashPage): New file, new class.
* splash.cc (SplashPage): Replace file with implementation of new
class.

* source.h (SourcePage): New file, new class.
* source.cc: Run indent.
(SourcePage): Add class implementation to this file.
(do_source): Remove, functionality subsumed by SourcePage::Create()
and SourcePage::OnDeactivate().
(dialog_proc): Remove, functionality subsumed by
SourcePage::OnActivate().

* site.h: Run indent.
(SitePage): Add class declaration.
* site.cc: Run indent.
(SitePage): Add class implementation.
(do_download_site_info_thread): New function.
(context): New var.  Context info for do_download_site_info_thread().
(do_download_site_info): New function.
(SitePage::Create, SitePage::OnInit SitePage::OnNext)
(SitePage::OnBack): SitePage class implementation..
(dialog_cmd): Remove IDOK, ID_BACK, and IDCANCEL handlers, now handled
by SitePage members.
(dialog_proc): Remove, now handled by SitePage::OnInit and base class
functionality.
(do_site): Remove, now handled by do_download_site_info_thread() and
SitePage::OnNext functionality.
(get_site_list): Remove dismiss_url_status_dialog() call.

* root.h (RootPage): New file, new class.
* root.cc: Run indent.
(dialog_cmd): Pass parent HWND parameter to note(), yesno().
(dialog_proc): Remove.
(do_root): Remove
(dialog_cmd): Remove IDOK, ID_BACK, and IDCANCEL clauses from switch.
(RootPage::OnInit): New member function.  Move get_root_dir() logic to
here from do_root().
(RootPage::Create): New member function.
(RootPage::OnNext): New member function.  Move IDOK logic from
dialog_cmd() to here.
(RootPage::OnBack): New member function.

* propsheet.h (PropSheet): New file, new class.
* propsheet.cc (PropSheet): New file, new class.

* proppage.h (PropertyPage): New file, new class.
* proppage.cc (PropertyPage): New file, new class.

* postinstall.cc (do_postinstall): Add owner parameter.

* other.cc (do_other): Add owner parameter.

* nio-file.cc (NetIO::Purl): Pass NULL parent param to note().
* nio-ftp.cc (NetIO_FTP::NetIO_FTP): Pass NULL parent param to
get_ftp_auth().
* nio-http.cc (retry_get): Pass NULL parent param to get_auth(),
get_proxy_auth(),
and get_ftp_auth().
* nio-ie5.cc (NetIO::_url): Pass NULL parent param to note(),
get_auth(), get_proxy_auth().

* netio.h (get_auth, get_proxy_auth, get_ftp_auth): Add owner param.
* netio.cc (auth_common, NetIO::get_auth, NetIO::get_proxy_auth,
NetIO::get_ftp_auth)
(auth_common): Add owner param.  Pass owner param to DialogBox.
(NetIO::get_auth, NetIO::get_proxy_auth, NetIO::get_ftp_auth): Pass
owner param to auth_common().

* net.h (NetPage): New file, new class.
* net.cc: Run indent.
(net.h, threebar.h): Add includes.
(Progress): Add extern.
(dialog_cmd): Remove IDOK, IDC_BACK, and IDCANCEL cases, now handled by
property sheet logic.  Saved temporarily for reference.
(NetPage::Create, NetPage::OnInit, NetPage::OnDeactivate,
NetPage::OnNext, NetPage::OnBack):
New implementation of NetPage class members.
(do_net): Remove.
(dialog_proc): Remove.

* msg.h (fatal, note, yesno): Add owner param.
* msg.c (fatal, note, yesno, mbox): Add owner param.
(mbox): Remove MB_TOPMOST from MessageBox call.  Unnecessary and wrong
now that we have a parent.

* main.cc: Run indent.
(commctrl.h, proppage.h, propsheet.h, splash.h, source.h)
(localdir.h, net.h, site.h, choose.h, threebar.h, desktop.h): Include
headers.
(root_dialog_proc): extern into this file.
(Progress): Progress dialog defined here, used in several other files.
(WinMain): Instantiate and create Splash, Source, Root, LocalDir, Net,
Site, Chooser, Desktop pages and MainWindow sheet.  Call
InitCommonControls() to make sure Windows is set up for our use of
property sheets. Add pages to sheet.
Call MainWindow.Create() to "DoModal".  Remove main loop, that logic is
now handled by the PropSheet class.
(root.h): Add include.

* log.cc (log_save): Pass NULL parent to fatal().
(exit_setup): Pass NULL parent to note().

* localdir.h (LocalDirPage): New file, new class.
* localdir.cc:  Run indent.
(localdir.h): New include.
(threebar.h): New include.
(Progress): extern into this file.
(LocalDirPage::Create, LocalDirPage::OnInit, LocalDirPage::OnActivate)
(LocalDirPage::OnNext, LocalDirPage::OnBack): Implementation of
LocalDirPage.
(LocalDirPage::OnNext): Move log() call from do_local_dir() to here.
(dialog_cmd): Remove IDOK, ID_BACK, and IDCANCEL handlers.
(do_local_dir): Remove.
(dialog_proc): Remove.
(cwd): Remove, not used by anything.

* install.cc: Run indent.
(process.h, threebar.h): New includes.
(Progress): extern into this file.
(ins_dialog, ins_action, ins_pkgname, ins_filename, ins_pprogress)
(ins_iprogress, ins_diskfull, init_event): Remove, now handled by
ThreeBarProgressPage.
(dialog_cmd, dialog_proc, dialog): Removed, handled in
ThreeBarProgressPage.
(init_dialog): Remove all mention of the above ins_* handles.  Now
handled in ThreeBarProgressPage.  Altered SetWindowText()s to call
ThreeBarProgressPage instance Progress directly.
(progress): Alter bar update logic to call ThreeBarProgressPage
instance Progress directly.
(uninstall_one): Alter SetWindowText()s to call ThreeBarProgressPage
instance Progress directly.
(install_one_source): Alter SetWindowText()s to call
ThreeBarProgressPage
instance Progress directly. Pass NULL parent to note().
(do_install_thread): Rename from do_install(), added owner param.
Alter SetWindowText()s to call ThreeBarProgressPage instance Progress
directly.
Remove dismiss_url_status_dialog() call, no longer necessary.  Remove
ShowWindow(<hide>) call, also unnecessary now.  Pass owner handle to
fatal().
(do_install_reflector): New function.
(do_install): New function.

* ini.cc (process.h, threebar.h): New includes.
(Progress): externed into this file.
(find_routine): Pass NULL parent to note().
(do_local_ini): Add owner param.
(do_remote_ini): Add owner param.  Pass owner to get_url_to_membuf()
and note().  Remove call to dismiss_url_status_dialog().
(do_ini_thread): Rename from do_ini().  Add owner param.  Pass owner to
do_local_ini(), do_remote_ini(), yesno(), and note().  Set next_dialog
to IDD_CHOOSER on exit.
(do_ini_thread_reflector): New function.
(context): New var.  Context for do_ini_thread.
(do_ini): New function.

* geturl.h (get_url_to_membuf, get_url_to_string, get_url_to_file): Add
owner param.
* geturl.cc (gw_dialog, gw_url, gw_rate, gw_progress, gw_pprogress,
gw_iprogress)
(gw_progress_text, gw_pprogress_text, gw_iprogress_text, init_event):
Removed.
(threebar.h): New include.
(Progress): externed into this file.
(dialog_cmd, dialog_proc, dialog): Removed, handled by
ThreeBarProgressPage now.
(init_dialog): Remove "if (gw_dialog == 0)" clause.  Alter
SetWindowText()s
and bar setting SendMessage()s to call ThreeBarProgressPage instance
Progress directly.  Remove "one bar only" logic, this is now handled
explicitly in the ThreeBarProgressPage class.
(progress): Altered bar and text update logic to call
ThreeBarProgressPage instance Progress directly.  Changed kbps
calculation to floating point and now print out a single decimal
place.
(get_url_to_membuf): Add owner param.  Pass it to init_dialog.
(get_url_to_string): Add owner param.  Pass it to get_url_to_membuf.
(get_url_to_file): Add owner param.  Pass it to init_dialog. Alter bar
update logic to call ThreeBarProgressPage instance Progress directly.
(dismiss_url_status_dialog):  Remove.

* fromcwd.cc (do_fromcwd): Add owner param.  Initialize found_ini to
false, was true.

* download.cc: Run indent.
(process.h, threebar.h): New includes.
(Progress): externed into this file.
(download_one): Add owner param.  Pass it to get_url_to_file().
(do_download_thread): Renamed from do_download.  Add owner param. When
calculating total_download_bytes, take binpicked and srcpicked into
account.  Remove call to dismiss_url_status_dialog(), no longer needed.
Pass owner handle to download_one() and yesno().
(do_download_reflector, do_download): New functions.
(context): New var.  Context for do_download_thread().

* dialog.h (D(x)): Add owner param.

* desktop.h (DesktopSetupPage): New file, new class.
* desktop.cc: Run indent.
(desktop.h): Add include.
(dialog_proc, do_desktop): Remove, now handled in
DesktopSetupPage::OnInit().
(dialog_cmd): Remove IDOK, IDC_BACK, and IDCANCEL cases, handled in
DesktopSetupPage::OnFinish(), DesktopSetupPage::OnBack(), and PropSheet
resp.
(DesktopSetupPage::Create, DesktopSetupPage::OnInit)
(DesktopSetupPage::OnBack, DesktopSetupPage::OnFinish): Implementation
of DesktopSetupPage.

* cistring.h: New file, new class.
* cistring.cc: New file, new class.

* choose.h: Run indent.
(Chooser): New class declaration.
* choose.cc: Run indent.
(do_choose): Add owner param.  Pass it to DialogBox() and
fatal().
(Chooser): New class implementation.

* res.rc (IDS_VERSION_INFO): New string.
(IDD_SOURCE, IDD_LOCAL_DIR, IDD_ROOT, IDD_SITE, IDD_OTHER_URL)
(IDD_DLSTATUS, IDD_INSTATUS, IDD_SPLASH, IDD_CHOOSE, IDD_DESKTOP):
Change WS_POPUP to WS_CHILD.  Numerous positioning/size changes
throughout.
(IDD_CHOOSE): Give template the WS_EX_CONTROLPARENT style to enable
TAB control navigation etc.  Give the controls a reasonable tab order.
Grouped radio buttons.  Something's still not right, can't tab away
from back/next/cancel group reliably.
* resource.h (IDD_CHOOSER): New dialog ID.

* Makefile.in (OBJS): Add cistring.o, proppage.o, propsheet.o,
threebar.o, and window.o.

2001-12-21  Robert Collins  <rbtcollins@hotmail.com>

        * choose.cc (listview_proc): Update the scrollbar when the headers are dragged.
        (pick_pkg_line::paint): Clip columns to prevent overlap.

2001-12-20  Robert Collins  <rbtcollins@hotmail.com>

        * Makefile.in: Add libcomctl32.a for setup.exe.
        * archive.h (archive): Prevent non-child direct creation of the class.
        * archive_tar.h (archive_tar): Prevent inaccurate synthetic copy and assignment.
        * choose.cc: Include commctrl for the choose header.
        (NEW_COL_SIZE_SLOP): Define without magic numbers.
        (pkgtrustp): Convert to function.
        (set_action): Remove.
        (add_required, fill_missing_category,default_trust): Use list syntax for
        package list.
        Use new set_action.
        (choose_caption): Remove.
        (topbucket::paint): New method.
        (topbucket::empty): Ditto.
        (topbucket::click): Ditto.
        (topbucket::~topbucket): Ditto.
        (paint): Remove i,ii and p as they are obsolete.
        Remove header drawing code.
        Use new chooser method to paint the chooser.
        Use new chooser itemcount method.
        (scroll_common): Rename to view::scroll.
        Scroll the header window separately.
        (list_vscroll, list_hscroll, list_click): Update for new chooser methods.
        (listview_proc): Look for header messages.
        (note_width): Accept stringless calls.
        (pick_line::set_line): Remove.
        (pick_line::paint): Remove.
        (pick_pkg_line::paint): Paint this one line, using the header column widths.
        (pick_category_line::paint): Paint this category, and anything in the bucket.
        (pick_line::click): Remove.
        (pick_pkg_line::click): Handle a click.
        (pick_category_line::click): Handle a click.
        (_view): Rename all to view.
        (view::view): Initialise the control contents.
        (view::set_headers): Work with the header control.
        (view::init_headers): Use list syntax for package list.
        Rearrange the width logic a little.
        (view::insert_pkg): Update to use the hierarchy.
        (view::insert_category): Ditto.
        (view::insert_at, view::insert_under): Remove.
        (view::clear_view): Update to use the hierarchy.
        (view::click): Update to use the hierarchy.
        (set_view_mode): Use list syntax for package list.
        Update for new chooser methods.
        (DoInsertItem): New helper function. (Needs a home somewhere).
        (create_listview): Remove choose control code and use the constructor instead.
        (dialog_cmd,scan2,do_choose): Use list syntax for package list.
        * choose.h: Include newly required headers list and package_meta.
        (_voew): Rename to view.
        (pick_line): Convert to an ABC.
        (pick_pkg_line): New class.
        (pick_catgory_line): New class.
        (topbucket): New class.
        (view::scroll): New method created from listview_scroll function.
        * desktop.cc (make_passwd_group): Use list syntax for package list.
        * download.cc (do_download): Use list syntax for package list.
        * fromcwd.cc (found_file): Use list syntax for package list.
        * iniparse.y (pacakges): Use list syntax for package list.
        * install.cc (uninstall_one): Use list syntax for package list.
        (do_install): Use list syntax for package list.
        * io_stream.h (io_stream): Prevent non-child direct creation of the class.
        Prevent inaccurate synthetic copy and assignment.
        * list.h: Include required header stdlib.
        Work on const correctness.
        * package_db.cc (packagedb::getpackagebyname): Remove.
        (packagedb::packagedb): Remove custom list code, and make db a local.
        (packagedb::getfirstpackage): Remove.
        (packagedb::getnextpackage): Remove.
        (packagedb::addpackage): Remove.
        (flush): Remove custom list code.
        (packagedb::registerpackage): Remove.
        (packagedb::packages): New static member.
        (packagedb::task): New static member.
        * package_db.h (PackageDBActions): New enum.
        (packagedb): Remove getfirstpackage, getnextpackage, getpackagebyname,
        addpackage, registerpackage, packages, packagecount, curr_package, db.
        Add packages, task.
        * pacakge_meta.cc: Include package_db.h.
        (packagemeta::action_caption): New method.
        (packagemeta::set_action): New method.
        * package_meta.h (CategoryPackage): Initialise key.
        (packagemeta): Initialise key.
        Add set_action and action_caption methods.
        Prevent inaccurate synthetic copy and assignment.
        * resource.h (IDC_CHOOSE_LISTHEADER): New ID.
        * source.cc: Include package_db for db manipulation.
        (save_dialog): Set the db task.
        * win32.h: Tell the w32 include headers what IE version to expect/require.

2001-12-06  Robert Collins  <rbtcollins@hotmail.com>

        * README: Wishlist updates.

2001-12-04  Robert Collins  <rbtcollins@hotmail.com>

        * package_db.cc (getnextpackage): Don't overrun the array.

2001-12-04  Robert Collins  <rbtcollins@hotmail.com>

        * Makefile.in (OBJS): Add setup_version.o.
        (version.c): Rename to setup_version.
        * choose.cc (set_action): Prevent NULL pointer dereference.
        (package_sort): Remove.
        * fromcwd.cc (canonicalize_version): Moved to version.cc.
        (check_ini): New function for use with find to see if there is a cached setup.ini.
        (do_fromcwd): Check for a cached setup.ini from any mirror site.
        * ini.cc (find_routine): New function, for finding cached setup.ini's.
        (do_local_ini): New function, drives parsing of local ini's.
        (do_remote_ini): New function, drives parsing and caching of remote ini's.
        (do_ini): Move guts to do_remote_ini and do_local_ini.
        * list.h (removebyindex): New method.
        * other.cc (save_dialog): Use new site_list and all_site_list to add the
        new site to the site picklist.
        (dialog_cmd): Only save the dialog when exiting the dialog.
        * res.rc (IDD_SITE): Make IDC_URL_LIST a multiple-selection control.
        * site.cc (site_list_type): Move to site.h
        (site_list): New global, sites chosen by the user.
        (all_site_list): New global, all potential sites.
        (site_list_type::init): New method, parses a mirror.lst URL string into a
        site_list_type object.
        (site_list_type::site_list_type): New method.
        (check_if_enable_next): Examine the control, not a local variable.
        (load_dialog): Select all the user chosen sites by default.
        (save_dialog): Adjust for multiple selection semantics.
        (save_site_url): Save all user chosen sites.
        (dialog_cmd): Don't save the dialog when the listbox recieves a message -
        thats not needed.
        Remove obsolete other_url reference.
        (dialog_proc): Use all_site_list instead of site_list.
        (site_sort): Remove.
        (get_site_list): Remove malloc use - use list template instead.
        Move site_list_type initialisation to site_list_type::init().
        (get_initial_list_idx): Renamed to ...
        (get_saved_sites): this. Also read in multiple sites to the site_list.
        (do_site): Use all_site_list not site_list.
        Log all chosen sites.
        * site.h: New file, defines site related class, functions and variables.
        * state.h: Add comment about use of state.h
        (mirror_site): Remove.
        (other_url): Remove.
        (MIRROR_SITE): Remove.
        * version.cc: New file, for version-handling and related functions.

2001-12-02  Robert Collins  <rbtcollins@hotmail.com>

        * Makefile.in: Add filemanip.o to setup.
        * archive.cc (archive::extract_file): Use new io_stream method 'copy'.
        * archive_tar.h (archive_tar_file): Implement get_size virtual.
        (archive_tar): Ditto.
        * choose.cc (set_action): Use [] operator instead of getnth - its more readable.
        (paint): Ditto.
        (_view::init_headers): Ditto.
        (_view::insert_pkg): Ditto.
        (set_view_mode): Ditto.
        (scan2): Ditto.
        (do_choose): Ditto.
        (base): Moved to filemanip.cc.
        (find_tar_ext): Ditto.
        (parse_filename): Ditto.
        (_Info::_Info): Remove.
        * compress_bz.h (compress_bz): Implement get_size virtual.
        * compress_gz.h (compress_gz): Implement get_size virtual.
        * cygpackage.cc (cygpackage::destroy): Use array delete for char * objects.
        * download.cc (get_file_size): Move to filemanip.cc.
        (check_for_cached): Use [] operator instead of getnth - its more readable.
        (download_one): Ditto.
        * filemanip.cc: New file.
        * geturl.cc (init_dialog): Make url a pointer to const as it is not modified.
        (get_url_to_membuf): New function - contains get_url_to_string worker code.
        (get_url_to_string): Becomes a trivial wrapper to get_url_to_membuf.
        * geturl.h: Declare new prototype.
        * ini.cc (do_ini): Use get_url_to_membuf and pass ini_init an io_stream.
        Tidy up the error code a little.
        * ini.h: Remove __cplusplus protection for ini_init.
        (_Info): Remove.
        * inilex.l: Remove old globals.
        (ini_init): Use an io_stream for getting the characters.
        (ini_getchar): Ditto.
        * iniparse.y: Declare yylex as a C++ function.
        (add_correct_version): Use [] operator instead of getnth - its more readable.
        * io_stream.cc (io_stream::move_copy): Use the copy method.
        (io_stream::copy): New method to simply copy from one stream to another.
        * io_stream.h (io_stream): New static method copy, and virtual method get_size.
        * io_stream_cygfile.cc (io_stream_cygfile::get_size): Implement this.
        * io_stream_cygfile.h (io_stream_cygfile::get_size): Declare this.
        * io_stream_file.cc (io_stream_file::get_size): Implement this.
        * io_stream_file.h (io_stream_file::get_size): Declare this.
        * io_stream_memory.h: Include errno as it's neded to parse the header.
        (io_stream_memory::get_size): Implement this.
        (io_stream_memory::st_size): Delete this.
        * list.h: Change getnth into the [] operator.
        * netio.cc (NetIO::NetIO): Change prototype to reflect Purl not being modified.
        (NetIO::set_url): Ditto.
        (NetIO::open): Ditto.
        * netio.h: As for netio.cc.
        * nio-file.cc (NetIO_File::NetIO_File): Change prototype to reflect Purl not
        being modified.
        * nio-file.h: Ditto.
        * nio-ftp.cc (NetIO_FTP::NetIO_FTP): Ditto.
        * nio-ftp.h (NetIO_FTP::NetIO_FTP): Ditto.
        * nio-http.cc (NetIO_HTTP::NetIO_HTTP): Ditto.
        * nio-http.h (NetIO_HTTP::NetIO_HTTP): Ditto.
        * nio-i5.cc (NetIO_IE5::NetIO_IE5): Ditto.
        * nio-i5.h (NetIO_IE5::NetIO_IE5): Ditto.
        * package_meta.cc (packagemeta::~packagemeta): Bugfix: use array delete.
        * package_source.cc (packagesource::set_canonical): Ditto.
        (packagesource::set_cached): Ditto.
        * package_source.h (site::~site): Ditto.
        * site.cc: Change site_list_type to a class, and search and replace sort_key to
        key globally.

2001-12-01  Robert Collins  <rbtcollins@hotmail.com>

        * package_source.cc (packagesource::set_canonical): Filename was out by one.

2001-11-30  Robert Collins  <rbtcollins@hotmail.com>

        * Makefile.in (OBJS): Remove category_list - it's not needed.
        * README: More wishlist updates.
        * category.cc (Category::Category): Initialise new members.
        (Categorycmp): New function.
        * category.h (Category): New member key for template use.
        * category_list.cc: Remove.
        * category_list.h: Remove.
        (Categorycmp): New function prototype.
        * choose.cc: Remove category_list - not needed.
        (set_action): Only show the 'source' option when the source is actually
        available.
        (add_required): Fix recursion limiter to 5.
        (fill_missing_category): Use list access.
        (default_trust): Ditto.
        (pick_line::paint): Ditto.
        (_view::init_headers): Ditto.
        (_view::insert_pkg): Ditto.
        (_view::insert_category): Ditto.
        (_view::click): Ditto.
        (set_view_mode): Ditto.
        (do_choose): Ditto.
        * cygpackage.h: Remove category_list - not needed.
        * geturl.cc (get_url_to_string): Fix null byte insertion.
        * inipatse.y: Remove category_list - not needed.
        (categories): Use list access.
        * package_db.cc: Ditto.
        * package_db.h: Remove category_list - not needed.
        (packagedb): Use the list template for categories.
        * package_meta.cc (add_category): Ditto.
        * package_meta.h (CategoryPackage): Link to the category as well.
        (packagemeta): Use a list of CategoryPackages instead of Categories
        for more memory efficient cross-referencing.

2001-11-30  Robert Collins  <rbtcollins@hotmail.com>

        * package_db.cc (packagedb::flush): Write a canonical version for all packages
        irrespective of the media they were installed from.

2001-11-29  Robert Collins  <rbtcollins@hotmail.com>

        * Makefile.in (OBJS): Add package_source and rfc1738.
        * README: Wishlist update.
        * category_list.cc (register_category): Use references.
        * category_list.h (CategoryList): Ditto.
        * choose.cc (isinstalled): Remove.
        (pkgtrustp): New macro.
        (set_action): Use packagemeta.
        (add_required): Ditto.
        (choose_caption): Example package to choose caption, rather than state flags.
        (check_existence): Remove.
        (set_existence): Remove old code.
        (fill_missing_category): Use packagemeta.
        (keep_or_skip): Remove.
        (default_trust): Use packagemeta and set trust values directly.
        (pick_line::set_line): Use packagemeta.
        (_view::insert_pkg): Ditto.
        (_view::insert_category): Ditto.
        (_view::click): Ditto.
        (set_view_mode): Ditto. Also remove "exclude" code.
        (create_listview): Use packagemeta.
        (dialog_cmd): Ditto.
        (getpkgbyname): Remove
        (scan2): Use packagemeta. Temporarily comment out the holefinding code.
        (read_installed_db): Remove.
        (package_sort): Use packagemeta.
        (do_choose): Use packagemeta. Don't sort the packages - the db does that.
        * choose.h: Use packagemeta.
        * cygpackage.cc (cygpackage::cygpackage): Initialise all variables.
        (cygpackage::cygpackage): Ditto.
        (cygpackage::set_canonical_version): Copy the passed string.
        (cygpackage::~cygpackage): Move cleanup code to destroy().
        (cygpackage::destroy): New function.
        (cygpackage::getfirstfile): Use a separate buffer.
        (cygpackage::getnextfile): Ditto.
        (cygpackage::Canonical_version): New function.
        (cygpackage::set_sdesc): Ditto.
        (cygpackage::set_ldesc): Ditto.
        * cygpackage.h (cygpackage::Canonical_version): New method.
        (cygpackage::set_sdesc): New method.
        (cygpackage::set_ldesc): New method.
        (cygpackage::SDesc): New method.
        (cygpackage::LDesc): New method.
        (cygpackage::set_canonical_version): New method.
        * desktop.cc: Include packagedb headers.
        (make_password_group): Use packagemeta.
        * download.cc: Include packagedb headers.
        (check_for_cached): New method to find any cached file and fill out the database
        info.
        (download_one): Take a packagesource, support multiple mirrors.
        (do_download): Use packagemeta.
        * from cwd.cc: Include packagedb headers.
        (found_file): Use packagemeta.
        * ini.cc (do_ini): Beginning of multiple-mirror support.
        * ini.h (CategoryPackage): Remove.
        (Dependency): Remove.
        (Package): Remove.
        (new_package,getpkgbyname,getpackagecategorybyname,add_category): Remove.
        * inilex.l: Support multiple mirrors.
        * iniparse.y: Include packagedb headers.
        (parser): Use packagemeta.
        (new_package): Remove.
        (add_correct_version): New function.
        (new_requirement): Remove.
        (add_category): Remove.
        * install.cc (uninstall_one): Use packagemeta.
        (install_one_source): New function.
        (install_one): Use packagemeta.
        (do_install): Use packagemeta.
        * io_stream.cc (io_stream::move): Bugfix on move/copy case.
        * io_stream_cygfile.cc (io-stream_cygfile::set_mtime): Do not reopen the file
        after setting the mtime, it makes no sense, and results in 0 length files.
        * io_stream_file.cc (io-stream_file::set_mtime): Ditto.
        * list.h: New file, template class for a form of list.
        * package_db.cc (packagedb::registerpackage): New function.
        * package_db.h (packagedb::npackages ()): New method.
        * package_meta.cc (packagemeta::add_version): Use list template.
        (packagemeta::set_installed): Ditto.
        (packagemeta::add_category): New method.
        * package_meta.h: Include necessary headers to parse the class.
        (CategoryPackage): New class.
        (packagemeta): Initialise all members.
        (packagemeta::SDesc): New method.
        (packagemeta::Categories): New method.
        (packagemeta::add_category): Ditto.
        (packagemeta::versions): Change to be a list instance.
        * package_source.cc (packagesource::set_canonical): New method.
        (packagesource::set_cached): New method.
        * package_source.h: Include necessary headers to parse the class.
        (site): New class.
        (packagesource::packagesource): Initialize all members.
        (packagesource): Make this class non-abstract.
        * package_version.cc (packageversion::packageversion): New method.
        * package_version.h (Dependency): New class.
        (packageversion::Canonical_vesion): New method.
        (packageversion::SDesc): New method.
        (packageversion::LDesc): New method.
        (packageversion::new_requirement): Ditto.
        (packageversion::srcpicked, binpicked): What to install.
        (packageversion::src, bin): Installation source data.
        * rfc1738.cc: New file. Escapes special characters in URL's.
        * rfc1738.h: Ditto.

Backout accidental commit

2001-11-26  Gary R. Van Sickle  <g.r.vansickle@worldnet.att.net>

        * nio-http.cc (NetIO_HTTP::NetIO_HTTP): Stop header parsing when
        SimpleSocket::gets() returns a zero-length string, so that we
        don't end up eating the entire stream thinking it's all header info.

2001-11-23  Robert Collins  <rbtcollins@hotmail.com>

        * install.cc (badrename): Remove.
        (do_install): Use packagedb flush method to write /etc/setup/installed.db.
        * io_stream.cc (io_stream::move_copy): New method.
        (io_stream::move): New method.
        * io_stream.h (io_stream): New methods move and move_copy.
        * io_stream_cygfile.cc (io_stream_cygfile::move): New method.
        * io_stream_cygfile.h (io_stream_cygfile): New method move.
        * io_stream_file.cc (io_stream_file::move): New method.
        * io_stream_file.h (io_stream_file): New method move.
        * package_db.cc (packagedb::flush): New method.
        * package_db.h (packagedb): New method flush.

2001-11-23  Robert Collins  <rbtcollins@hotmail.com>

        * Makefile.in (OBJS): Add package_version.
        * ategory.cc (Category::Category): Update constructors.
        * category_list.cc (CategoryList::register_category): Insert correctly.
        * choose.cc: Include package_version.h, not package.h.
        * cygpackage.cc: Ditto.
        (cygpackage::uninstall): New method.
        * cygpackage.h (cygpackage): Rename genericpackage to packageversion.
        New method uninstall.
        * hash.cc (add, has, enumerate): Change prototype to be accurate.
        * hash.h (hash): Change prototypes to be accurate.
        * ini.h (_Info): New parameterless constructor.
        (CategoryPackage): New constructor.
        (Package): Ditto.
        * install.cc: Don't include unneeded headers.
        Include package database headers.
        (hash::add_subdirs): Remove.
        (uninstall_one): Use packagemeta->uninstall to perform the uninstallation.
        Don't uninstall source packages.
        (do_install): Just can uninstall_one once as we ignore source packages now.
        Don't copy the installed.old database accross, instead iterate through the
        packagedb packages list and write the isntalled packages out.
        * package_db.cc: Include package_version.h, not package.h.
        (packagedb::packagedb): Never let db be undefined.
        Record the installation filename, so we can reconstruct installed.db.
        (packagedb::addpackage): Insert packages in alpha sorted order.
        (packagedb::categories): Initialise correctly.
        * package_meta.cc: Include win32 and mount headers as a workaround to
        current io_stream limitations. Include a TODO about that.
        (standard_dirs): New, from install.cc.
        (hash::add_subdirs): New, from install.cc.
        (packagemeta::set_installed): Rename genericpackage to packageversion.
        (packagemeta::uninstall): New method.
        * package_meta.h: Rename genericpackage to packageversion.
        (packagemeta): Update constructor.
        (add_version, set_installed): Rename genericpackage to packageversion.
        (uninstall): New method.
        (installed_from): New member.
        * package_source.cc: New file.
        * package_source.h: New file.
        * package_version.cc: New file.
        * package_version.h: New file.
        * package.cc: Removed.
        * package.h: Removed.

2001-11-21  Robert Collins  <rbtcollins@hotmail.com>

        * Makefile.in (OBJS): Add category.o and category_list.o.
        * category.h: New file.
        * category.cc: New file.
        * category_list.h: New file.
        * category_list.cc: New file.
        * choose.cc: Include category headers.
        (fill_missing_category): Use package_db to access global categories.
        (_view::insert_pkg): Ditto.
        (_view::insert_category): Ditto.
        (set_view_mode): Ditto.
        (getcategorybyname): Remove.
        * cygpackage.h: Include category_list.h to parse this correctly.
        (cygpackage): New virtual &Categories.
        * ini.h (_CategoryPackage): More C to C++ conversion.
        (Category): Remove.
        (getcategorybyname): Remove.
        (register_category): Remove.
        * iniparse.y: Include package db and category headers.
        (categories): Use package_db to access global categories.
        (category): Remove.
        (ncategories): Remove.
        (register_category): Remove.
        * package.h: New virtual &Categories.
        * package_db.cc (packagedb::packagedb): Don't read the installed db twice.
        (packagedb::categories): Declare.
        * package_db.h (packagedb): New member categories for global categories list.

2001-11-21  Robert Collins  <rbtcollins@hotmail.com>

        * choose.h: Don't include ini.h
* ini.h: Allow pure c++ in the header file.

2001-11-21  Robert Collins  <rbtcollins@hotmail.com>

        * install.cc (do_install): Write a version 2 installed.db file.
        * package_db.cc (packagedb::packagedb): Read version 1 and 2 installed.db files.

2001-11-21  Robert Collins  <rbtcollins@hotmail.com>

        * Makefile.in (OBJS): Add package abstraction objects.
        * choose.cc: Include package abstraction headers.
        (read_installed_db): Use packagedb class and ignore installed sources.
        * cygpackage.cc: New file.
        * cygpackage.h: New file.
        * filemanip.h (get_file_size): Use unixy types.
        * package.h: New file.
        * package.cc: New file.
        * package_db.cc: New file.
        * package_db.h: New file.
        * package_meta.cc: New file.
        * package_meta.h:

2001-11-21  Robert Collins  <rbtcollins@hotmail.com>

        * REAME: Updates to wishlist.

2001-11-16  Robert Collins  <rbtcollins@hotmail.com>

        * README: Updated and added my list of wishlist and todo items.

2001-11-14  Robert Collins  <rbtcollins@hotmail.com>

        * Makefile.in (OBJS): Include io_stream_memory in setup.exe.
        * archive_tar.cc (archive_tar::write): New prototype.
        (archive_tar::seek): New method.
        * archive.h (archive_tar_file): Update write() and add seek().
        (archive_tar): Ditto.
        * archive_tar_file.cc (archive_tar_file::write): New prototype.
        (archive_tar_file::seek): New method.
        * compress_bz.cc (compress_bz::write): New prototype.
        (compress_bz::seek): New function.
        * compress_bz.h (compress_bz): Update write() and add seek().
        * compress_gz.cc (compress_gz::write): New prototype.
        (compress_gz::seek): New function.
        * compress_gz.h (compress_gz): Update write() and add seek().
        * geturl.cc: Include io_stream headers.
        (GUBuf): Remove.
        (get_url_to_string): Use a io_stream_memory buffer instead of GUBuf.
        * io_stream.h (io_stream_seek_t): New enum.
        (io_stream): Update write() and add seek().
        * io_stream_cygfile.cc (io_stream_cygfile::write): New prototype.
        (io_stream_cygfile::seek): New function.
        * io_stream_cygfile.h (io_stream_cygfile): Update write() and add seek().
        * io_stream_file.cc (io_stream_file::write): New prototype.
        (io_stream_file::seek): New function.
        * io_stream_file.h (io_stream_file): Update write() and add seek().

2001-11-13  Brian Keener  <bkeener@thesoftwaresource.com>

       * localdir.cc: Add headers unistd.h and port.h.
       (save_local_dir): Expand search logic to include local directory for
       location of file last-cache.
       (dialog_cmd): Skip IDD_ROOT when downloading and use Back button.
       (do_local_dir): Expand search logic to include local directory for file
       last-cache.
       * res.rc (IDD_CHOOSE_DIALOG): Add hotkey to View button.
       * Source.cc (dialog_cmd): Skip IDD_ROOT if select Download from
       Internet.
       (do_source): Default to Install from Internet on first entry or
       previous selection if backing up from IDD_ROOT or IDD_LOCAL_DIR.

2001-11-13  Robert Collins  <rbtcollins@hotmail.com>

        * choose.cc (find_tar_ext): Only match at the end of the string.
        (getpkgbyname): Prevent NULL pointer dereference.

2001-11-13  Robert Collins  <rbtcollins@hotmail.com>

        * Makefile.in (CFLAGS): Add warnings and -Werr.
        (WARNONLY_FLAGS): For sources that produce (harmless) errors no matter what.
        (OBJS): Add io_stream class' objects.
        (autoload.o): Use WARNONLY_FLAGS.
        (inilex.o): Ditto.
        (iniparse.o): Ditto.
        * archive.cc: New file.
        * archive.h: New file.
        * archive_tar.cc: New file.
        * archive_tar.h: New file.
        * archive_tar_file.cc: New file.
        * autoload.c: Fix compiler warnings (unused, deprecated conversion).
        * choose.cc: Use io_stream.h, not mount.h.
        Run indent.
        (find_tar_ext): Rewritten to use strstr, and to accept .tar as an extension.
        (read_installed_db): Use io_streams.
        * choose.h: Run indent.
        * compress.cc: New file.
        * compress.h: New file.
        * compress_bz.cc: New file.
        * compress_bz.h: New file.
        * compress_gz.cc: New file.
        * compress_gz.h: New file.
        * concat.cc: Run indent.
        (cvsid): Fix compiler warnings (unused, deprecated conversion).
        * concat.h: Run indent.
        * desktop.cc: Run indent.
        Include mklink2.h rather than redefining by hand.
        (cvsid): Fix compiler warnings (unused, deprecated conversion).
        (etc_profile): Fix compiler warning (deprecated conversion).
        (make_link): Fix compiler warning (deprecated conversion).
        Tidy up use of args variable - consolidate into argbuf.
        (start_menu): Fix compiler warning (deprecated conversion).
        (desktop_icon): Fix compiler warning (deprecated conversion).
        (make_etc_profile): Fix compiler warning (deprecated conversion).
        (uexists): Fix compiler warning (deprecated conversion).
        (check_desktop): Fix compiler warning (deprecated conversion).
        (check_startmenu): Fix compiler warning (deprecated conversion).
        * dialog.cc: Run indent.
        (cvsid): Fix compiler warnings (unused, deprecated conversion).
        (fatal): Fix compiler warning (deprecated conversion).
        * dialog.h: Run indent.
        * diskfull.cc: Run indent.
        (cvsid): Fix compiler warnings (unused, deprecated conversion).
        * download.cc: Run indent.
        (cvsid): Fix compiler warnings (unused, deprecated conversion).
        (download_one): Fix compiler warning (signed vs unsigned comparison).
        * filemanip.h: Run indent.
        * find.cc: Run indent.
        (cvsid): Fix compiler warnings (unused, deprecated conversion).
        (find): Fix compiler warning (deprecated conversion).
        * find.h: Run indent.
        * fromcwd.cc: Run indent.
        * geturl.cc: Run indent.
        (cvsid): Fix compiler warnings (unused, deprecated conversion).
        (dialog_cmd): Return a value.
        (dialog_proc): Remove unused variables.
        Return a value.
        (get_url_to_file): Fix compiler warning (deprecated conversion).
        * geturl.h: Run indent.
        * hash.cc: Run indent.
        (cvsid): Fix compiler warnings (unused, deprecated conversion).
        * hash.h: Run indent.
        * ini.cc: Run indent.
        Use io_streams.
        (cvsid): Fix compiler warnings (unused, deprecated conversion).
        (do_ini): Use io_streams for local io.
        * ini.h: Run indent.
        * inilex.l (ignore_line): Fix compiler warning (is not a prototype).
        Fix compiler warning (brackets for truth assignment).
        * install.cc: Run indent.
        Fix compiler warning (/* in comment).
        Use io_streams.
        (cvsid): Fix compiler warnings (unused, deprecated conversion).
        (dialog_cmd): Return a value.
        (dialog_proc): Return a value.
        (start_tics): Deleted - was unused.
        (badrename): Fix compiler warnings (unused, deprecated conversion).
        (standard_dirs): Fix compiler warning (deprecated conversion).
        (exists): Remove.
        (uninstall_one): Convert to io_streams.
        (install_one): Convert to io_streams.
        (do_install): Convert to io_streans.
        Fix compiler warning (deprecated conversion).
        * io_stream.cc: New file.
        * io_stream.h: New file.
        * io_stream_cygfile.cc: New file.
        * io_stream_cygfile.h: New file.
        * io_stream_file.cc: New file.
        * io_stream_file.h: New file.
        * localdir.cc: Run indent.
        Use io_streams.
        (cvsid): Fix compiler warnings (unused, deprecated conversion).
        (get_root_dir_now): Remove.
        (save_local_dir): Convert to io_streams.
        (dialog_cmd): Return a value.
        (do_local_dir): Convert to io_streams.
        (log.cc): Run indent.
        (cvsid): Fix compiler warnings (unused, deprecated conversion).
        (log): Fix compiler warning (deprecated conversion).
        (log_save): Fix compiler warning (deprecated conversion).
        * log.h: Run indent.
        (log): Fix compiler warning (deprecated conversion).
        (log_save): Fix compiler warning (deprecated conversion).
        * main.cc: Run indent.
        Don't include "netio.h".
        (cvsid): Fix compiler warnings (unused, deprecated conversion).
        (netio_test): Remove.
        (sid_auth): Fix compiler warning (partially bracketed initialiser).
        (WinMain): Return a value.
        * mkdir.cc: Run indent.
        (cvsid): Fix compiler warnings (unused, deprecated conversion).
        * mklink2.c: Run indent.
        (cvsid): Fix compiler warnings (unused, deprecated conversion).
        (mkcygsymlink): New function.
        * mklink2.h: New file.
        * mount.cc: Run indent.
        (cvsid): Fix compiler warnings (unused, deprecated conversion).
        (create_mount): Remove unused variables.
        (cygpath): Fix compiler warning (signed vs unsigned comparison).
        * mount.h: Run indent.
        * msg.cc: Run indent.
        (cvsid): Fix compiler warnings (unused, deprecated conversion).
        (msg): Fix compiler warning (deprecated conversion).
        (mbox): Fix compiler warning (deprecated conversion).
        * msg.h: Run indent.
        (msg): Fix compiler warning (deprecated conversion).
        * net.cc: Run indent.
        (cvsid): Fix compiler warnings (unused, deprecated conversion).
        (dialog_cmd): Return a value.
        * netio.cc: Run indent.
        (cvsid): Fix compiler warnings (unused, deprecated conversion).
        (auth_cmd): Return a value.
        * netio.h: Run indent.
        * nio-file.cc: Run indent.
        (cvsid): Fix compiler warnings (unused, deprecated conversion).
        (NetIO_File::NetIO_File): Fix compiler warning (deprecated conversion).
        * nio-file.h: Run indent.
        * nio-ftp.cc: Run indent.
        (cvsid): Fix compiler warnings (unused, deprecated conversion).
        * nio-ftp.h: Run indent.
        * nio-http.cc: Run indent.
        (cvsid): Fix compiler warnings (unused, deprecated conversion).
        * nio-http.h: Run indent.
        * nio-ie5.cc: Run indent.
        (cvsid): Fix compiler warnings (unused, deprecated conversion).
        * nio-ie5.h: Run indent.
        * other.cc: Run indent.
        (cvsid): Fix compiler warnings (unused, deprecated conversion).
        (dialog_cmd): Return a value.
        * port.h: Run indent.
        * postinstall.cc: Run indent.
        (cvsid): Fix compiler warnings (unused, deprecated conversion).
        (cmd): Fix compiler warning (deprecated conversion).
        (eun): Fix compiler warning (deprecated conversion).
        * root.cc: Run indent.
        (cvsid): Fix compiler warnings (unused, deprecated conversion).
        * simpsock.cc: Run indent.
        (cvsid): Fix compiler warnings (unused, deprecated conversion).
        * simpsock.h: Run indent.
        * site.cc: Run indent.
        Use io_streams.
        (cvsid): Fix compiler warnings (unused, deprecated conversion).
        (get_root_dir_now): Remove.
        (save_site_url): Convert to io_streams.
        (dialog_cmd): Return 0.
        (get_initial_list_idx): Convert to io_streans.
        * source.cc: Run indent.
        (cvsid): Fix compiler warnings (unused, deprecated conversion).
        (check_if_enable_next): Remove.
        (load_dialog): Remove unused variables.
        (save_dialog): Ditto.
        (dialog_cmd): Return a value.
        * splash.cc: Return a value.
        (cvsid): Fix compiler warnings (unused, deprecated conversion).
        (dialog_cmd): Return a value.
        * state.cc: Run indent.
        (cvsid): Fix compiler warnings (unused, deprecated conversion).
        * state.h: Run indent.
* tar.cc: Remove.
* tar.h: Remove.

* desktop.cc (start_menu): Change start menu name to "Cygwin".
(check_startmenu): Ditto.

Revert botched checkin.

* choose.cc (parse_filename): Finish reorganization by correctly decrementing p
in memmove.

* choose.cc (parse_filename): Reorganize code to work around gcc optimization
problem.

2001-11-09  Gareth Pearce  <tilps@hotmail.com>

        * choose.cc (set_action): Allow skip as an option for installed
        packages that have version number not listed in setup.ini.

2001-11-10  Robert Collins  <rbtcollins@hotmail.com>

        * choose.cc (set_cation): Backout last change.
        (choose_caption): Ditto.

2001-11-10  Robert Collins  <rbtcollins@hotmail.com>

        * ini.h (actions): Remove - transferred to choose.h.
        (views): Ditto.
        * choose.h: Protect class's from "C" compilers.
         (actions): New enum, from ini.h. This may not be the 'right' place
        but that place doesn't exist yet.
        (_views): New enum, from ini.h.
        * choose.cc (set_action): Handle installed packages with a version not in
        setup.ini.
        (choose_caption): Ditto.
        * iniparse.y: Rearrange header order to handle new ini.h.

2001-11-10  Robert Collins  <rbtcollins@hotmail.com>

        * ini.h (_Info): Fix compiler warnings (signed vs unsigned comparison).
        (registry_category): Fix compiler warning (deprecated conversion).
        * iniparse.y (registry_category): Ditto.
        * choose.cc (cvsid): Fix compiler warnings (unused, deprecated conversion).
        (add_requried): Remove unused variable.
        (choose_caption): Handle all enumeration values via a default.
        (paint): Fix compiler warnings (unused, deprecated conversion).
        (scroll_common): Fix compiler warnings (unused, signed vs unsigned comparison).
        (list_clicK): Fix compiler warnings (signed vs unsigned comparison).
        (note_width): Fix compiler warnings (deprecated conversion).
        (keep_or_skip): Fix compiler warnings (unused).
        (_view::mode_caption): Fix compiler warnings (deprecated conversion).
        (set_view_mode): Fix compiler warnings (unused).
        (create_listview): Fix compiler warnings (unused, incorrect format, reaches
        end of function).
        (dialog_proc): Fix compiler warnings (unused).
        (read_installed_db): Fix compiler warnings (unused, deprecated conversion).
        * choose.h (_header): Fix compiler warnings (deprecated conversion).
        (_view): Ditto.

2001-11-08  Pavel Tsekov  <ptsekov@syntrex.com>

        * simpsock.h (SimpleSocket::invalidate): Declare new method.
        * simpsock.cc (SimpleSocket::invalidate): Implement new method.
        (SimpleSocket::SimpleSocket): Initialize buf to zero. Do not allocate
        memory for buf in the constructor.
        (SimpleSocket::~SimpleSocket): Use SimpleSocket::invalidate().
        (SimpleSocket::printf): Use SimpleSocket::write() instead of send().
        (SimpleSocket::write): Check object consistency - return -1 on error.
        invalidate() the object on socket write error.
        (SimpleSocket::fill): Check object consistency - return -1 on error.
        invalidate() the object if socket read error is encountered and there
        is no more data available in the internal read buffer.
        Allocate memory for the internal read buffer.
        (SimpleSocket::gets): Return zero (NULL pointer) if error is encountered
        during fill() and no more data is available in the internal read buffer.
        (SimpleSocket::read): Check object consistency - return -1 on error.
        invalidate() the object if socket read error is encountered.
        * nio-ftp.cc (NetIO_FTP:NetIO_FTP): Allow 125 as valid response code to
        the RETR command (fix for MS IIS ftp server 5 - possibly others too).
        (NetIO_FTP::ok): Check if the SimpleSocket object is ok().
        (NetIO_FTP::read): Use NetIO_FTP::ok().
        * nio-http.cc: Check for valid return value of SimpleSocket::gets().
        (NetIO_HTTP::ok): Check if the SimpleSocket object is ok().

2001-11-09  Robert Collins  <rbtcollins@hotmail.com>

        * simpsock.cc (cvsid): Avoid compiler warnings (not used, deprecated conversion).
        (SimpleSocket::SimpleSocket): Avoid compiler warnings (deprecated conversion).
        (SimpleSocket::printf): Ditto.
        (SimpleSocket::write): Ditto.
        * simpsock.h (SimpleSocket): Avoid compiler warnings for constructor,
        printf and write.
        * nio-ftp.cc (cvsid): Avoid compiler warnings (not used, deprecated conversion).
        (NetIO_FTP::NetIO_FTP): Remove unused variable done.
        * nio-http.cc (cvsid): Avoid compiler warnings (not used, deprecated conversion).

2001-11-07  Charles Wilson  <cwilson@ece.gatech.edu>

        * geturl.cc (get_url_to_string): clean up memory leak
(get_url_to_file): ditto

2001-11-06  Robert Collins  <rbtcollins@hotmail.com>

* localdir.cc (dialog_cmd): Go back to IDD_ROOT in every case.
* source.cc (dialog_cmd): Go forward to IDD_ROOT in every case.

2001-11-06  Pavel Tsekov  <ptsekov@syntrex.com>

        * fromcwd.cc (found_file): Free() memory containing the version and
        filepath information for a given package, before replacing the pointers which
        reference this memory.

2001-11-03  Robert Collins  <rbtcollins@hotmail.com>

        * fromcwd: Fix compiler warnings for cvsid.
        (canonicalize_version): Fix compilter warnings.
        (do_fromcwd): Ditto.
        * version.h: Fix compiler warnings for canonicalize_version.

2001-11-03  Robert Collins  <rbtcollins@hotmail.com>

        * res.rc (IDD_CHOOSE): Move the view caption to the right hand side.

2001-11-02  Robert Collins  <rbtcollins@hotmail.com>

        * res.rc (IDD_CHOOSE): Tweak for use on a 640x480 screen.

2001-11-02  Robert Collins  <rbtcollins@hotmail.com>

        * choose.cc (set_action): Correctly allow skip for any uninstalled package.

2001-11-02  Robert Collins  <rbtcollins@hotmail.com>

        * choose.cc (default_trust): Change the "Required" forced install category to "Base".

2001-11-02  Robert Collins  <rbtcollins@hotmail.com>

        * choose.cc: Ran indent.
        (set_action): When the requested trust level is missing and the level was not
        user requested, set the action to ACTION_SAME_CURR.

* Makefile.in (mingw_getopt.o): Fix typo in non-verbose compilation.

2001-11-02  Robert Collins  <rbtcollins@hotmail.com>

        * choose.cc (add_required): Allow SKIP and SRC_ONLY for currently installed
        packages.

2001-11-02  Robert Collins  <rbtcollins@hotmail.com>

        * choose.cc (default_trust): Autoset currently installed packages per the user's
        selection of prev/curr/exp.

2001-11-02  Robert Collins  <rbtcollins@hotmail.com>

        * splash.cc (dialog_proc): Return TRUE as specified in the WIN32 API.

2001-11-02  Pavel Tsekov  <ptsekov@syntrex.com>

        * geturl.cc (is_showing): Remove.
        (is_local_install): New static variable. Controls whether the
        progress dialog and the supporting thread will be created.
        (dialog_proc): Return TRUE to WM_INITDIALOG as specified by the WIN32 API.
        (init_dialog): Remove usage of SetForegroundWindow and is_showing.
        This prevents the progress dialog to gain control over the
        application and fixes the "setup.exe going to background" problem.
        (progress): Use is_local_install.
        (get_url_to_string): Set is_local_install.
        (dismiss_url_status_dialog): Use is_local_install.
        * install.cc (dialog_proc): Return TRUE as specified in WIN32 API.
        (init_dialog): Remove usage of SetForegroundWindow.
        This prevents the progress dialog to gain control over the
        application and fixes the "setup.exe going to background" problem.

2001-11-02  Robert Collins  <rbtcollins@hotmail.com>

        * choose.cc (list_click): Change r to row for clarity.
        When we shrink the list below 1 page, scroll to the top of the list.
        (_view::click): Return the actual change for contractions.
        (dialog_proc): Return TRUE to WM_INITDIALOG as specified by the Win32 API.

2001-11-02  Robert Collins  <rbtcollins@hotmail.com>

        * choose.cc (dialog_cmd): Swap cygwin root selection and local dir selection order.
* localdir.cc: Ran indent.
        (get_root_dir_now): New function.
        (save_local_dir): New function.
        (dialog_cmd): Use it and swap cygwin root selection and local dir selection order.
        (do_local_dir): If there is a saved local dir, read it in.
        * net.cc (dialog_cmd): Swap cygwin root selection and local dir selection order.
        * root.cc (dialog_cmd): Ditto.
        * source.cc (dialog_cmd): Ditto.

2001-11-01  Robert Collins  <rbtcollins@hotmail.com>

        * choose.cc(list_vscroll): Return 0 as required by win32.
        (list_hscroll): Ditto.
        (list_click): Ditto.

2001-11-01  Robert Collins  <rbtcollins@hotmail.com>

        * argv.cc: Remove.
        * argv.h: Remove.
        * main.cc(WinMain): Use __argv to retrieve command line parameters.
        * Makefile,in: Remove argv.o.

2001-11-01  Robert Collins  <rbtcollins@hotmail.com>

        * argv.cc: Remove.
        * argv.h: Remove.
        * main.cc(WinMain): Use __argv to retrieve command line parameters.
        * Makefile,in: Remove argv.o.

2001-11-01  Matt Hargett  <matt@use.net>

        * geturl.cc (progress): Change int to unsigned int to eliminate
        lint warnings.
        * (get_url_to_string): Check for NULL pointer.
        * mount.cc (find2): Use free instead of delete for malloc'd
        memory.
        * nio-http.cc (NetIO_HTTP::NetIO_HTTP): Fix memory leak.

2001-10-31  Robert Collins  <rbtcollins@hotmail.com>

        * choose.cc(_view::insert_under): Handle linen > nlines and lines[linen] with neither
        a package nor a category. The lines[linen] bug was reported by Matt (matt@usa.net).
        (read_installed_db): Bugfix for testing whether a package has a source tarball.
        Reported by Matt (matt@usa.net).
        * geturl.cc(dialog): Rename gw_dialog to local_gw_dialog to avoid programmer
        confusion and lint errors. Reported by Matt (matt@usa.net).

2001-10-31  Robert Collins  <rbtcollins@hotmail.com>

* argv.cc: New file, converts arguments for -mwindows programs. (Simplified derivation
from dcrt0.cc.)
* argv.h: New file.
* Makefile,in: Add argv.o and mingw_getopt.o to setup.exe object list.
Add mingw_getopt.o build rule.
* main.cc(WinMain): Initialise argv and argc for use by getopt.

* res.rc: Rearrange info on top line slightly.

* desktop.cc (make_passwd_group): Move file creation earlier in the process to
ensure that passwd-grp.bat reflects desired operation.
* postinstall.cc (each): Remove "*.done" file else rename will fail.
Copyrights.

* Makefile.in: Only build testsuite directory on first 'make check'.
* configure.in: Remove testsuite from SUBDIRS.
* configure: Regenerate.

* res.rc: Oops.  Take 640x480 screen size into account.

Remove CRs.

* res.rc: Enlarge chooser screen.

* Makefile.in: Accomodate changes to w32api_include macro.

Use correct flags for windres.

* Makefile.in (CFLAGS): Remove -nostdinc from CFLAGS.

* choose.cc (pick_line::paint): Reorganize after botched patch.

* choose.cc (pick_line::paint): Add missing declaration.

deal with spaces.

adjust for ncurses package-splitup

* choose.cc (pick_line::paint): Add package name to sdesc for display.

more updates

update-setup version 2

update

updates for cwilson packages

Wed Sep 26 21:39:00  Robert Collins rbtcollins@hotmail.com

        * choose.cc (_view::insert_pkg): Correctly exit loop.
        * iniparse.y (register_category): Always insert new categories.

* inilex.l (yylex): For comments, eat all characters up to, but not including
newline.

update

Temporary checkin

* fromcwd.cc (found_file): Use correct package name when adding an unknown
package rather than random stack garbage.