Add double-click for a 'default action' to ListView

Add support for a double-click in the ListView to invoke a 'default
action'.

Because we receive both NM_CLICK and then NM_DBCLK, reduce the area of a
pop-up column which is sensitive to a click to the drop-down button
(which opens a focus-stealing pop-up menu), so the rest of the area can
receive a click (which does nothing) or a double-click (which does the
default action).

For a package, this default action is to toggle between the currently
installed version (or uninstalled, if not installed), and the highest
non-test version.

There is no keyboard acceleration for this action currently.

Don't propagate actions down category tree into obsolete categories

Choosing the 'Install' action on the 'All' category shouldn't propagate
down into the '_obsolete' category, because that will just result in
dependency conflicts due to trying to install both obsolete packages and
their replacements.

Don't show FTP 550 'file not found' errors in a MessageBox

Don't show FTP 550 'file not found' errors in a MessageBox. Also log
InternetGetLastResponseInfo() when fetching a URL.

WinInet documentation seems to indicate this kind of extended error
information only exists when WinInet has an FTP error code to report.

Log this error (and the associated URL) rather than just showing a
MessageBox. Don't bother showing a MessageBox for 550, since we can
tolerate certain missing files (e.g. setup.zst), and we'll go on to
report that couldn't fetch needed files in a more generic way.

Use 'kill -W' when killing processes

We only have access to the Windows PID, which (since Cygwin 3.0.0) are
completely decoupled from Cygwin PIDs.

Use the new '-W' flag to indicate to kill that we are providing a
Windows PID.

(If the Cygwin install is too old to support kill -W, kill will fail,
and we'll fall back to using TerminateProcess())

Only dump transaction list when it's finalized

Dumping the transaction list after 1) trust level is set, 2) solving for
dependencies, and 3) adding source packages is a bit much.

Simple keyboard accelerators for Listview

Wire-up certain actions to keyboard accelerators.

To permit selecting a subitem apparently needs us to custom-draw everything,
so for simplicity, just make the accelerators per row.

Getting the enter keypress to be delivered to the listview is tricky (the
propsheet really want to hold onto it), so for the moment we just use the
Windows menu key and space bar as accelerators.

Indicate test packages in action menu

Add listview to tab order on chooser page

Use selected row background colour for a checkbox column

Allow for width of dropdown button in popup columns

Currently, the dropdown button in the 'New' column can overlap the version,
because  'New' columns is just sized to hold the widest version.

Allow for the width of the dropdown button as well in the 'New' column width

Merge branch 'topic/listview'

Ensure that an installed packageversion has an ldesc if possible

In packagedb::read(), copy the ldesc from setup.ini to the
packageversion read from installed.db.  Otherwise, an installed
package with only one version will not have an ldesc to use as a
tooltip.

v2: If the installed version is no longer available, copy the ldesc
from the current version of the package.

Add ldesc tooltips to sdesc column of listview

Listview's built-in tooltip support doesn't support subitems, so we have to
build our own

v2: Use string cache

Restore packagemeta::LDesc()

v2:
Update for removed 'usiing namespace std'

Add LDesc() accessor method to SolvableVersion

Use indents in category view

We keep around an empty imagelist for 1x1 images, to reset the indent when
not in tree view mode

(This isn't quite right as the listview will allocate a 1-pixel space before
column 0 for those images, but this seems the best we can do as 0x0
imagelists aren't allowed...)

Use an icon to represent expanded/collapsed state

Use a listview icon to represent expanded/collapsed state, rather than
puttting '[+]'/'[-]' in front of the category name

Just use the item icon directly to represent expanded/collapsed state,
rather than using a state icon as otherwise we have empty space where the
item icon would be, when it's size is being used for indenting.

The icons were made by tracing the previously used .bmp in Inkscape to
produce a .svg, then converting that to an icon with ImageMagick using
'convert -filter point -background transparent -define icon:auto-resize'

v2:
Drop execute mode (Achim Gratz)

Show the count of packages in a category

Custom draw popup menus in ListView control

Construct a menu containing the actions from the action list for the package
or category, and if one is selected, apply it.

This lets us remove packagemeta::set_action() which implements the strange
UX of cycling around actions without showing what the possibilities are.

This somewhat emulates a BS_SPLITBUTTON style control.  The 'popup' cell has
a visual hint that clicking on it opens a menu (a 'combox scrollbar'), and
the popup menu is located at the position of the click.

v2:
Factor out popup_menu, for future use by keyboard accelerators

Add methods for listing possible actions on, and applying one to, a package

The 'action id' is a value of the _actions enum, if positive, otherwise it's
absolute value is a 0-based index into the 'versions' collection to identify
a specific version to install.

v2:
Fix select_action() to consider deftrust

v3:
Fix select_action() for keep

v4:
Update for removed 'using namespace std'

Custom draw checkboxes in ListView control

Future work: What does this look like when a theme is in used? Does the row
size need to be slightly adjusted to ensure it accomodates checkbox height?

v2:
erase to background colour before drawing checkbox

Use a ListView common control rather than a hand-built grid

Each line is implemented by an object of a subclass of ListViewLine, either
PickPackageLine, or (in catgeory view) PickCategoryLine

v2:
Defer constructing category tree until packagedb is available
Also handle an empty category tree
Remove unused bitmaps
Scoping of strings returned to LVN_GETDISPINFO
Avoid doubled 'all' category
Update some number of lines, rather than just the current one

v3:
Speed up calculating column widths, by caching the DC used
Don't bother recalculating column widths when obsolete packages are
shown/hidden, just allow for obsolete packages always (which makes hardly
any difference)
Don't bother calculating column widths separately for category view since
they are now always the same.

v4:
Store current category action in CategoryTree
Recurse category action onto packages

This slightly changes the behaviour: previously, a category action only
effected packages which matched the name search filter.  Now all packages in
contained by the category are effected.

v5:
When updating, turn off redraw before emptying listview
Make headers a parameter to init
Don't leak contents
Only resize columns once
Remove ListView OnMessage as it has nothing to do
Never use a column width less than minimum
Use LVS_SINGLESEL
Factor out string cache for re-use
Preserve focused row over ListView::setContents()

Add OnNotify virtual function to class Window for WM_NOTIFY notifications

Add OnNotify virtual function to class Window for WM_NOTIFY notifications
Note that the result is returned via DWLP_MSGRESULT

Change packagemeta::_actions to an enum

Add zstd package to .appveyor.yml

Reformat so we can escape newlines to avoid an absurdly long line

Add zstd packages to build instructions

Also adjust for 'if building from git' is the only way, since we don't make
source releases.

Add support for ZStandard compression

Add support for ZStandard compression for packages and .ini files

Remove misleading copy&paste comments

Avoid stringop-overflow warning with gcc8

desktop.cc: In function 'void start_menu(const string&, const string&, const string&, const string&)':
desktop.cc:110:11: error: 'char* strncat(char*, const char*, size_t)' specified bound 260 equals destination size [-Werror=stringop-overflow=]

I think strlcat() was meant here, which MinGW doesn't have.  In it's
absence, open-code it's equivalent.

(SHGetSpecialFolderLocation() returns a pathname of length at most MAX_PATH,
and make_link() is limited to accepting a pathname of length MAX_PATH, so we
want to append our folder name, while truncating the result to MAX_PATH.)

excise "using namespace std;" and explicitly use a "std::" prefix throughout instead

Avoid potential null deref in check_for_cached()

Packages for which we don't know a filename can't possibly be cached.

See https://cygwin.com/ml/cygwin/2018-10/msg00033.html

Allow host machine to be specified to bootstrap

Allow host machine to be specified to bootstrap, rather than it picking the
first one it finds a compiler for

Improve libgcrypt detection

Set ACLOCAL_PATH so libgcrypt.m4 from cross-package is found, if it is the
only one installed.

Accomodate older libgcrypt.m4, which doesn't check SYSROOT

Use pkg-config for dependency checking where possible

Modernise depedency checking and use pkg-config where possible

Fix message spam when package vendor is something other than "cygwin"

Installed packages are getting coerced to vendor "cygwin", so there will be
warnings if the vendor string of the single repo is something other than
that.

Remove unused IDI_SPIN

Added in 713bbe5f, but never actually used

Remove unused choose-rtarrow.bmp

Usused since replaced by tree-plus.bmp/tree-minus.bmp in 82306ac2

Properly identify Win32 systems which support IsWow64Process2()

Properly identify Win32 systems which support IsWow64Process2() (Windows 10
version 1511 (TH2) or later) in the default user-agent string.

Teach bitness reporting in user-agent string about WoW64 on ARM64

Remove uninitialized NetIO::ftp_auth flag

This appears to be left over since f5d45c3b.

NetIO::get_ftp_auth() doesn't appear to be used at all, but perhaps should
by the NetIO_IE5 constructor for FTP auth.

Initialize file_size in NetIO constructor

At the moment, this is not initialized anywhere.  If it's value happens to
be greater than zero, that will be used as the size to report download
progress against.

Don't write dependency problems to log with CRLF line terminators

Only transform line endings in the dependency problems string used in a
Windows dialog text box, not in what's written to the log as well.

https://cygwin.com/ml/cygwin/2018-03/msg00266.html

Keep track of all known sites for a given version of a package

When adding a packageversion, extend it's packagesource::sites vector to
include the sites from any similar packageversions previously processed.

Also remove those packageversions from the libsolv pool so that libsolv will
always find the one that lists all the sites.

This is needed for:

- Correct handling of local installs where required packages are spread
between cache directories for more than one mirror

- Correctly falling back to a second site when multiple mirrors are used and
a problem occurs connecting to the first mirror tried.

Add Vendor() accessor method to SolvableVersion

Internalize the libsolv repo attribute data after each setup.ini

Call SolverPool::internalize() in the IniDBBuilderPackage destructor. This
makes attribute data from all previously processed setup.ini files available
when the next setup.ini is being processed.

Remove the now unneeded call to SolverPool::internalize() at the beginning
of packagedb::read().

Propagate exit code of elevated process

Propagate the exit code of elevated process, when --wait is used.

This addresses part of the problem about exit code noted in [1], although we
also need to audit that a non-zero exit code is reported in all error
situations.

[1] https://cygwin.com/ml/cygwin/2018-07/msg00020.html

Also report if starting the elevated child process failed.

If setup-minimum-version: was checked, don't check setup-version:

Don't check setup-version: to warn about potential setup.ini parsing
problems if a setup-minimum-version: is specified.

Add --no-version-check option

This suppresses the warning that a newer version of setup is available.

This is intended to be used by shortcuts and scripts that run setup in a
'setup' package.

This would also suppress any future auto-update feature.

Fix SolvableVersion::deplist() to handle version-relations correctly

This was using completely the wrong Id for the version-relation, so it was
always returning unversioned dependencies

Finish providing support for provides: and conflicts:

Introduce member functions SolvableVersion::provides() and
SolvableVersion::conflicts().  This enables packagedb::read() to access
provides and conflicts lists from setup.ini.

README: Put marker for footnote about copr next to the correct package

Signed-off-by: Jon Turney <jon.turney@dronecode.org.uk>

netio.cc: fix a bug in string handling

commit ed6137e5f94f45e50a12994aeae310278b2bf758 back in 2003(!)
introduced a bug in creating an empty password string which,
embarrassingly, only now has been catched by gcc.

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>

replace http links with official https URLs

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>

Fix detection of automatically added packages

In the "confirm" dialog, report a package as automatically added for install
if the version being installed is different from the version requested by
the user.  This can happen if the global update mode is "Keep" but a package
needs to be updated because of a versioned dependency.

Improve the preparation of the package database

Introduce a new function packagedb::noChanges() to set the database to a "no
changes requested" state.  Call it the first time the chooser page is
activated, before calling ChooserPage::applyCommandLinePackageSelection().

Also use it to simplify the code in two places.

Remove code from ChooserPage::changeTrust() that was making changes to the
database after ChooserPage::applyCommandLinePackageSelection() had already
been called.

Allow command-line uninstallation of specific orphaned packages

Also change the implementation of the "--delete-orphans" option to use
the same definition of "orphaned" that's used elsewhere in the code.

Make sure that the IniDBBuilderPackage destructor is called when needed

The IniDBBuilderPackage destructor contains code that is intended to be run
after each setup.ini file is processed.  But the IniDBBuilderPackage
variables in do_local_ini() and do_remote_ini were declared outside the loop
that processed the files.  Move the declaration inside the loop so that the
destructor is called after each iteration.

Handle test-only packages better

Don't create a current version with an empty version string when parsing a
package which doesn't have a current version.

Sort the packages listed in the "confirm" dialog

Give a fatal error on a checksum failure during install

This only affects local installs, where the hash of an archive is not
checked until we reach do_install_thread().  At this point it seems
too late to recover safely.

Link with -lws2_32 again

libgpg-error 1.28 includes new APIs which use socket functions, which
on Win32 are provided by WinSock2.

https://cygwin.com/ml/cygwin/2018-03/msg00306.html

Signed-off-by: Yaakov Selkowitz <yselkowi@redhat.com>

Migrate configured connection type from the removed 'direct(legacy)' to 'direct'

Make SolvableVersion::accessible() useful for local installs

For local installs, we need SolvableVersion::accessible() indicate whether
or not there is an archive available in the local cache.

This is used in packagemeta::set_action() to prevent the chooser from
offering unavailable versions for re-installation.

Remove packages not found by scan from solver

Remove not-installed packages not found by scan from solver, as well as from
packagemeta, to avoid solutions including them from being proposed.

Add a --version option

Add a --version option, and make some related tweaks to output

Add conflicts: support

Add provides: support

Always give the fake root category the name 'All'

After we rearranged things in 0c539f7f, it's now too early to tell if we
have any packages or not.

The only thing about this category that is ever used is it's name, so we
don't actually need to use the real 'All' category here.

Saying 'No packages found' was never particularly helpful here, so just use
a fake category with the fixed name 'All'.

Fix packagemeta::ScanDownloadedFiles

packagemeta::scan clears the site list if the package was not found, and
packagemeta::ScanDownloadedFiles uses packageversion::accessible() to check
that.

Instead communicate via a return value

v2:
empty packages were always inaccessible, even though we returned early from
scan() without clearing the sites list, so return false in that case

Update .gitignore

Add setup-minimum-version: to setup.ini

This allows setup.ini to require a certain setup version, rather than advise
a newer version when one is available.

Unfortunately, versions of setup prior to this one don't implement this, but
at least we have this going forward.

When we want to start using this, we can break backwards compatibility with
even older setup in a less clean way, simply by using setup.ini grammar that
they can't parse.

Improve file:// URL handling

As a repo URL, we're accepting:

* raw windows paths (with both \ and /), e.g.

  c:\cygwin repo
  \\machine\share\cygwin repo

* proper file: protocol URLs, e.g.

  file:///c:/cygwin%20repo
  file://machine/share/cygwin%20repo

Most non-standard URLs accepted by the old code should work, too.
Paths longer than 260 characters are not supported anymore.

NetIO: Remove unused URL parsing code

Remove simpsock.{cc,h} and drop linkage with ws2_32

Remove NetIO_HTTP

Add Proxy class to encapsulate proxy configuration for NetIO_IE5

Drop Proxy::bypass method (jturney)

Remove NetIO_File

Convert raw paths into file:// form URLs so WinInet can handle them

This maybe changes the set of malformed file:// URLs we handle (which is ok,
although it would be nice to know the details),

Rename IDC_NET_IE5 to IDC_NET_PRECONFIG

Remove direct(legacy) connection type

If setup.rc contains net-method:Legacy, it will be silently converted to
net-method:IE

Avoid a crash if we try to uninstall an unknown package

This can happen if the solver decides it wants to uninstall 'base'.

A package provides what it obsoletes, as well as itself.

This is needed to allow a package which obsoletes an installed package to
get installed.

Will need to revisit this when we implement explicit provides: in setup.ini.

Don't crash in confirm page when installing source packages

Fix a spelling mistake in a tooltip

Add .appyveyor.yml

Make sure we process the last version description in setup.ini

Allow depends: to be empty

If a version section has no depends:, we'll end up using any previous
requires: or depends: seen.

This isn't quite right, and we should only use the requires: (if any) if a
particular version section has no depends:

For the moment, just allow an empty depends:, so a version section can be
explicitly marked as having no depends:

Apply default problem solutions in unattended mode

Also log problems
Also make --include-source work in unattended mode

Merge branch 'topic/libsolv'

Add a new page to let the user review and confirm actions

Add a new page to let the user review and confirm actions, after
dependencies have been added and problems solved.

Ideally, this would re-use the picker-page grid to present these actions,
but for the moment just write it as text (as we did before in the prereq
page) (This is still a slight improvement as it shows all actions for
review, not just the ones added by the solver)

Add handling for 'replace-versions:' in setup.ini

Sometimes we make mistakes and release versions which are broken or have a
version which is wildly incorrect.  We can remove those versions from future
download, but they can linger indefinitely in installs (until a distupgrade
is done), as there is a higher version already installed.

Allow setup.ini to contain a per-package 'replace-versions:' header, which
identifies any versions which should be forcibly dist-upgraded (i.e.
downgraded to the best version in setup.ini).

(It's not possible to just instruct the solver to disfavour these specific
versions, as that's not implemented, see libsolv issue #170)

Add 'depends2:' as an alternative to 'depends':

Prior versions of setup recognize 'depends:', but don't handle it properly:

(i) they always use the dependencies of the current version, not the version
actually being installed

(ii) they cannot parse any version-relation containing a relation and
version identifier (i.e. they can only handle a bare package name)

Thus, there is no safe way to start using 'depends:' lines without getting
undesirable behaviour from old versions of setup

(In case (i), it may install incorrect dependencies if a non-current version
is installed, potentially resulting in a broken package.

In case (ii), it will fail to parse the setup.ini with an error.)

Add 'depends2:', so calm can supply per-version dependencies in a form
useful to setup with libsolv-based dependency solving, without effecting
older versions of setup.

Preserve reinstall tasks when going back from dependency problem page.

Clarify the unsolved-problems warning

If the user unchecks the 'Accept default solutions' box and selects
'Next', don't imply that choosing a non-default solution would break
their system.

Remove all trailing ".any" from package names in problem report

Apply default solution to dependency problems

Mark the default solution in the problem report

Refactor of SolverSolution::update() so we can apply the default
solution(s).

Also:

Break out logging of the task list, so we can show it in the "dependency
problems exists, but don't use the default solution, just do what I ask"
case.

Break out 'include-source' process, so it can have effect in the case where
dependency problems exist.

v2:
Actually re-solve after adjusting the task list with default solutions.

Use the last solution as the default solution.  The solver sorts the
solutions, and I think the last one will normally be the one which removes
the problematic task from the task list

More crash avoidance in SolvableVersion member functions

The following call-stack means we end up calling SolvableVersion methods
before packagedb::prep(), i.e. before attributes have been internalized, so
their values aren't available.

PickView::init_headers
PickView::refresh
PickView::init
ChooserPage::createListview
ChooserPage::OnInit

We call PickView::refresh again after we've computed the initial solution,
so the columns always end up with the correct width.

Correctly order preparing packagedb for chooser

Do packagedb::prep() in ChooserPage::OnActivate(), as doing it in OnInit()
is wrong, as that only gets called once (but lazily).

Make packagedb::prep() idempotent after packagedb::init(), so it doesn't do
it's work again, if we come back to chooser from a later page.

Re-arrange applying command line package selection, and determining the
initial solution, which should only happen once, but after packagedb::prep()

Ensure packagedb and underlying solver pool is empty before we read setup.ini

We need to make sure the packagedb is empty if we step backwards to here.

v2:
Also clear solver stored in SolverSolution, which holds a pool pointer

Remove the function filemanip.cc:base()

This was called only on plain filenames, not full pathnames, so it was not
needed.  Moreover, it wasn't correctly handling filenames containing colons.

Avoid clobbering installed.db when no setup.ini is found

If no setup.ini is found, do_ini_thread is never called.  But we need to
ensure that packagedb::read is called, or else installed.db gets emptied.
Move the calls to packagedb::read and other packagedb functions from
do_ini_thread to ChooserPage::OnInit.