Only show deprecation warning once if we're going to elevate

Provide prototype for SHMessageBoxCheckW()

Present in implib, but not prototyped in MinGW64 headers until 9.0.0

Don't create install package if there's no install: line

Don't erroneously create an install package if there's no install: line
for that version.

If it's for the latest version and something depends on this package,
the solver will produce a solution containing that package & version,
so we'll try to download it without having a path to fetch it from,
leading to download errors which are impossible to resolve.

Use SHMessageBoxCheck() for MessageBox with "don't show again" checkbox

Ignore 'CYGWIN=winsymlinks:lnk'

Ignore 'CYGWIN=winsymlinks:lnk', rather than converting it to
'--symlink-type lnk', which then fails with '*** --symlink-type lnk is
not implemented'.

Show a MessageBox warning if running on a deprecated Windows version

Show a MessageBox warning if we are running on a Windows version which
we have deprecated Cygwin support for:

- Windows 6.0 (Windows Vista, Windows Server 2008)
- 32-bit Windows

This warning can be disabled with '--no-warn-deprecated-windows'.

Update TODO

Improve symlinks to absolute paths made for ' --symlink-type native'

Match Cygwin DLL behaviour of not using '\\?\' prefix if the target path
is short enough not to need it.

This improves how the symlink is reported by CMD buildin 'DIR', and
handled by File Explorer's file properties dialog.

When in unattended mode, ignore attempts to close via GUI

Perhaps this should be controlled by a separate option, but making setup
non-interactive seems like a reasonable fit with the intent of
'--quiet-mode'.

This doesn't stop the installer being killed by TerminateProcess().

Addresses: https://cygwin.com/pipermail/cygwin/2021-November/250079.html

Further fix symlinks to absolute paths made for ' --symlink-type native'

After using 'to' with NtOpenFile(), convert it back to Win322
filenamespace form before using it with CreateSymbolicLinkW(), so that
the symlink works with native tools.

https://cygwin.com/pipermail/cygwin/2022-January/250398.html

Update displayed copyright to 2022

Fix symlinks to absolute paths made for ' --symlink-type native'

As currently written, we go to the trouble of making absto, and then
don't use it, which isn't right at all...

Fix SeCreateSymbolicLinkPrivilege detection

SeCreateSymbolicLinkPrivilege is enabled automatically on use, so it
only needs to be present in the token, not enabled.

Fix inverted symlink creation right logging

Push some dynamic dialog text into IDD_FILE_INUSE resource

Can't use that approach for the label for the IDRETRY button (since it
needs a fixed ID), so move that to string resources.

Also make text placed in IDC_FILE_INUSE_MSG localizable.

Move download progress message to string resources

Move confirm report strings to string resources

Move hash checking progress message to string resources

Use string resource for progress windows title

Move local package directory browser title to string resources

Also improve LoadStringW to handle a missing translation by falling back
to untranslated string.

Move root directory browser title to string resources

Note: IFileDialog now recommended over SHBrowseForFolder.

There's something a little wonky here: The browser is shown by
OnMessageCmd(), so a simple click on the unfocused button isn't enough?

Move localizable 'empty chooser' text to string resources

Don't localize logged 'exit message' set via setExitMsg()

Don't localize mbox strings written to log

Add LoadStringWEx to access string resource for a specifc locale.
Use that in mbox when formatting string for log.

Push some dynamic download/install dialog text into IDD_LOCAL_DIR DIALOG resource

Move the localizable dialog text which is dynamically modified depending
on install or download mode into the IDD_LOCAL_DIR DIALOG resource, and
show or hide it as appropriate.

Fix format() returning a std::string containing a terminating null

Send postinstall thread complete message last in that thread

Addresses: https://cygwin.com/pipermail/cygwin/2021-November/250084.html

After some staring at the code, it looks like it's possible that the
cause of this report is the linked list containing log entries to be
damaged if multiple threads call endLog() simultaneously.

Currently 'Changing gid to Administrators' is logged by the call to
nt_sec.setAdminGroup() just after the WM_APP_POSTINSTALL_THREAD_COMPLETE
message is been sent.

'Ending cygwin install' is logged just before main exits.

Make sending the WM_APP_POSTINSTLL_THREAD_COMPLETE message the last
thing that the postinstall thread does  This should avoid the potential
for endLog() becing called by multiple threads simultaenously.

Logging is still lousy with race conditions as the std::stringbuf all
threads are writing to is global (leading to messages from multiple
threads being concatenated here), and appending to the linked list isn't
thread-safe, but this should at least avoid this particular infinite
loop.

Drop some cruft in logging

The 'hack' referred to in endEntry() was removed in c8356810, but the
comment wasn't updated.

Add an option to set the GUI language

Add an option to set the GUI language, primarily for testing purposes.

We need to call SetThreadUILanguage() not only in the main thread, but
in every thread which might access localized resources (directly or
indirectly).

Add french res.rc

Add french res.rc, with dialog, message box, progress and actions string
translations from Eric Laussage, converted to utf8 encoding.

v2:
Update with translations for dynamically modified text in IDD_DESKTOP
and IDD_CHOOSE DIALOG resource

v3:
Add mbox caption, more mbox strings
MB_RETRYCONTINUE button label

Use UTF8 encoding in .rc file

Use UTF8 encoding in resources .rc file (rather than defaulting to
Windows codepage 1252)

Move some non-localizable string from resources

Push MB_RETRYCONTINUE MessageBox customization into mbox wrapper

Push the MB_RETRYCONTINUE MessageBox customization into the mbox
wrapper, and use it for a generic error during file extraction.

Use generic formatting mbox wrapper

Use generic formatting mbox wrapper for some:
- message boxes whose contents are currently hand-made using a format string.
- message boxes whose contents can be put into a string resource.

This also makes the various captions used by these messageboxes
consistent.

Future work: Aim to phase out remaining uses of the existing mbox()
wrapper:
- via note(), fatal(), yesno()
- where we a need to use a fully dynamic string (e.g. from ini parser)

Add a more generic mbox wrapper

This mbox wrapper understands loading format strings from string
resources and unattended mode.

Factor out the unattended_result logic from the existing mbox wrapper,
so it can be shared with that.

Drop message box for invalid --arch option

Make this consistent with all other cases of an invalid option, where
the error is just reported to stdout.

Fix uses of MessageBox() which could be problematic in unattended mode

Direct uses of MessageBox should be avoided, as they need special
attention to check that they are avoided in unattended mode (as they
could cause setup to stop and wait for user interaction).

Fix a few remaining uses that aren't guarded by checks of
unattended_mode, by using mbox() instead.

(These are new or slightly non-trivial uses which didn't get changed in
a123de66).

Review uses of MessageBox()

Use a consistent caption "Cygwin Setup" (for consistency with mbox()).

Also drop pointless CANCEL logic and clean up handling of IGNORE for
ABORTRETRYINGORE mbox when local package directory can't be accessed.

Also remove the CANCEL button from the MessageBox displayed for a
'message:' hint, which quits setup, with no indication that's what it's
going to do.

Also drop some commented out code for a modeless propsheet

Remove one version of fatal()

This is only used in two places, which are "this can't happen"
situations, where it doesn't actually make much sense to exit setup.

Push some dynamic download/install dialog text into IDD_DESKTOP DIALOG resource

Move the localizable dialog text which is dynamically modified depending
on install or download mode into the IDD_DESKTOP DIALOG resource, and
show or hide it as appropriate.

Push some dynamic download/install dialog text into IDD_CHOOSE DIALOG resource

Move the localizable dialog text which is dynamically modified depending
on install or download mode into the IDD_CHOOSE DIALOG resource, and
show or hide it as appropriate.

Make '--quiet' imply 'download and install packages'

Make '--quiet' imply 'download and install packages', rather than
'whatever source mode was chosen in the last run'.  This aligns
behaviour with the help text about installation type.

This can be overriden with '--download' or '--local-install'. Specifying
both already has the meaning 'download and install packages', but let's
not require than in combination with '--quiet' to get definite
behaviour.

Move column header strings to string table resource

Move view mode strings to string table resource

Use package action strings from resources for column width

Use the package action strings from string table resource for computing
the column width.

(This should have been done as part of f34a20e7, but we didn't notice
the strings were also duplicated here.)

This requires a bit of enhancement to ListView::noteColumnWidth() to
work for wstrings as well.

Default symlink mode from CYGWIN env var

Propagate --symlink-type setting to post-install scripts

Propagate --symlink-type setting to post-install scripts, by setting the
CYGWIN env var appropriately.

Add a command line option to choose symlink type used

The default is 'sys', the historical behaviour of setup.

v2:
Adjust to use StringChoiceOption
Align option names with winsymlink values

Factor out StringChoiceOption

Factor out logic for 'a string option which is one of a set of choices'
from CompactOsStringOption as StringChoiceOption.

v2:
Allow different behaviour for option without a choice, and option absent.

Future work: This doesn't say anything other that "Error during option
processing" if you've given an invalid choice string, or left it out
when it's required, because libgetopt++ doesn't output anything, just
returns success or failure after processing the command line arguments.

Add symlink capabilities to user-agent telemetry

Report if (i) we have the symlink creation privilege, and (ii) if
developer mode is on, so unprivileged symlink creation is allowed.

Add NTSecurity::hasSymlinkCreationRights()

Also report if SeCreateSymbolicLink privilege is available to log.

Add separate symlink-creation phase when extracting archive

Add a separate symlink-creation phase when extracting an archive, by
ignoring symlinks on the first pass, rewinding the archive, and
then extracting only symlinks on the second pass.

This helps a lot with native symlinks (which require the destination to
exist when created, so we can determine if it is a file or directory).

Alternative implementations:

We could collect symlinks, and defer making them until the end of
extracting the archive.  We'd also need to report errors if making those
symlinks failed.

We could close and re-open the archive, rather than rewinding it. Error
handling if the archive isn't accessible the second time could be
complex.

Add seek() method to archive and compress file classes

Add seek() method to archive and compressed file isostream classes
(which can only rewind to the start).

Also clean up some cruft in archive class.

This still needs testing on a .gz archive (slightly involved as there
aren't any currently!)

Factor out the iteration over archive files to install

Add support for creating native symlinks

Add support for creating WSL symlinks

Only build for x86_64 in github workflow

Only build for x86_64 in github workflow, since the requirements for
i686 are now impossible to obtain (Fedora <=31 for non-DW2 EH compiler,
but coprs for EOL'ed Fedoras have been expired)

Add new option '--compact-os ALGORITHM'.

If specified, selected Compact OS compression algorithm is applied
to files below /bin, /sbin and /usr.  Most DLL files are excluded
because rebase will open these files again for writing.

Fix a crash in ListView StringCache

Fix a sporadic crash in chooser introduced in f34a20e7.

Limit the mempcy() to fill StringCache to the size of the string we are
putting in the cache, not the the size of the cache (which will fault if
it happens to over-run into a inaccessible page).

Cancel search timer after 'clear' button pressed

Make 'Clear' button on IDD_CHOOSE less snug

Simplify uninstall-only warning

Simplify uninstall-only warning when running in 'Install from Local
Directory' mode with a non-existent package directory.

Strictly, this is wrong as written, since is_64 reflects the --arch
option setting, not the arch of the executable.

In any case, mentioning the executable name seems to just distract from
the warning we are trying to deliver here.

Fix which column we size to accomodate action labels

This wasn't actually operating on the right column (due to a typo in
ce9f6dd0), but fortunately that was benign (as the longest version is
probably longer than any action).

Fix native machine reported on x86_64 without IsWow64Process2()

If the OS version is old enough not to have IsWow64Process2(), and
IsWow64Process() returns FALSE, this only indicates x86 for an x86
process.

Fixes 4936dbc.

Add --allow-test-packages

Handle '--packages package=version'

Handle '--packages package=version' to allow specifing the version of a
package to install on the command line.

isManuallyWanted() now returns the target packageversion (if specified),
or an empty packageversion (which is translated into an instruction to
the solver to choose the version).

In the 'upgrade' case, this changes from using the complex logic of
packagemeta::trustp() to determine the target version, when unspecified
on the command line, to allowing the solver to make that decision (which
should be broadly the same i.e. not downgrading test versions, etc.).

This also subtly changes the behaviour when the package version is not
specified.  Instead of forcing the current version, we allow the solver
to choose the version, so it may not pick that version, if other
constraints exist, which prevent a solution containing that version.

There's probably some future work which can be done to simplify the
remaining uses packagemeta::trustp(), which are only related to the UI.

Add GUI language to default User-Agent string

Putative changes to bitness reporting for Win64-on-ARM64

Check compiler exception handling model during configure

Since F32, Fedora ships a MinGW x86 compiler which uses DW2 exception
handling [1].  This does not support throwing exceptions from Windows
callbacks, which we use.

[1] https://fedoraproject.org/wiki/Changes/Mingw32GccDwarf2

Add a github action to cross-build on Fedora

Add scripts used in packaging

Link with libssp

Fedora builds some of the libraries we use with -D_FORTIFY_SOURCE=2, so
we need to link with libssp to provide __memcpy_chk etc.

Future work: build setup itself with -D_FORTIFY_SOURCE=2.

Turn on __USE_MINGW_ANSI_STDIO everywhere

__USE_MINGW_ANSI_STDIO is turned on by C++ STL headers, so we're getting
this define due to that in most places, except where we happen to
include stdio.h before them. archive_tar.cc happens to do that, and
wants ANSI stdio since da31ad35, so turn it on unconditionally.

It's also turned on for C++11 by MinGW headers >= 8.0.0, which Cygwin
already has, so this already works when building there, and we're going
to have to deal with any other problem that might cause elsewhere in the
source eventually anyhow.

This fixes building since da31ad35 on F33, which only has mingw-headers
7.0.0.

Update displayed copyright to 2021

Revert "Remove pointless variable from do_download_site_info_thread()"

This reverts commit d15c11e7954977ee27712989b36e7102fc70639e.

This wasn't pointless, as it was causing the site list to be fetched
only once, even if we passed through that page more than once.

Move package action strings to string table resource

Move package action strings to string table resource.
Switch ListView control to wide-char.

Add a UTF-8 string to wstring utility function.

Remove long-obsolete 'DISCARABLE' from resource file

The 'DISCARDABLE' memory attribute was only signficant on Win16.

Move installation progress strings to string table resource

Allow progress status to be set directly from a resource

Allow progress status to be set directly from a (localizable) resource.

Add LoadStringW to std::wstring utility function

Fix version used when not built from a git checkout

Warn about unknown package names

Warn about unknown package names used with '--packages' or
'--remove-packages' options.

Downgrade logging of looking for AV service

Downgrade to debug always emitted logging about trying to find AV
service.

Fix error with gcc 10.2 and -Werror=format

archive_tar.cc: In member function ‘virtual const string archive_tar::next_file_name()’:
archive_tar.cc:161:37: error: 'I' flag used with ‘%o’ gnu_scanf format [-Werror=format=]
archive_tar.cc:161:37: error: format ‘%o’ expects argument of type ‘unsigned int*’, but argument 3 has type ‘size_t*’ {aka ‘long long unsigned int*’} [-Werror=format=]
archive_tar.cc:181:38: error: 'I' flag used with ‘%o’ gnu_scanf format [-Werror=format=]
archive_tar.cc:181:38: error: format ‘%o’ expects argument of type ‘unsigned int*’, but argument 3 has type ‘size_t*’ {aka ‘long long unsigned int*’} [-Werror=format=]

Propagate the value of CYGWIN environment variable into scripts

Propagate the value of the CYGWIN environment variable into the
sanitized environment we use to run scripts.

Note: as it stands, this isn't always useful as the initial environment
isn't propagated when setup re-runs itself elevated.

Migrate selected sites from http:// to https://

If a selected site URL saved from the last run starts with "http://",
and an identical URL except starting with "https://" is in the current
mirror list, migrate it from "http://" to "https://".

Factor out site_list_insert in site.cc

Factor out site_list_insert in site.cc, and add comments to descibe what
what nutty thing this wacky code is doing, after we've expended the
mental effort to figure it out.

Remove pointless variable from do_download_site_info_thread()

Properly record user_picked again

Properly record user_picked again, which was broken by changes in 2.895

Really parse build-depends: case-insensitively

Refine splash page text

Make "you can run this again to add, remove or update packages" a
separate paragraph to increase the chances of it getting read.

Position logo relative to top when resizing splash page

Improve how paired boolean options appear in help output

Include a prefix in the boolean option long name appearing in help
output (although we should probably pick the default, rather than first,
prefix)

Add dpiAware to manifest

Make application manifest architecture generic

It was made architecture specific in 2258d2a1, but then made the same
again for both architectures in aa09dcbb.

Parse build-depends: case-insensitively

Fix an allocation length error

Allow for the terminating null.

Disable old Cygwin setup signing key by default

Add a new option '--enable-old-keys', for if you really need to install
from an old mirror for some reason.

'--disable-old-keys' is still accepted, but is the default now.

Implement paired boolean options

Add some infrastructure for options which may appear with multiple
prefixes.

Add an implementation of paired boolean options like '--enable-foo' and
'--disable-foo', or '--bar' and '--no-bar'.

Add/fix tests