Improve consistency of ending sentences with a full-stop

Improve the consistency between translations of ending sentences with a
full-stop.

Always end the sentence under the page title with a full-stop.

Tidy up spacing in french translation

Consistently use a space before ':' (apparently "space before and after
full-height punctuation" is a standard typographical rule for French).

Remove unneeded trailing whitespace.

Use a .pot translation template

Add a res.pot file, created with rc2po.
Add rule to generate a .po file containing current translations using rc2po.
Add rule to incorporate updated translations from .po file using po2rc.

Both tools are part of the python translate toolkit [1]

[1] http://toolkit.translatehouse.org/

Adjustment to dialog layout for translation

To enable the english resource to be used as a template, adjust the size
of the IDD_SOURCE.IDC_SOURCE_NETINST control so it's large enough for
translated text.

Further tweak formatting in resource files

In particular, translation-toolkit's .rc file parser can't currently
handle seeing arithmetic rather just than an integer.

Move english resources to a separate file

Move english resources to a separate file, so it can serve as a template
resource file for translations in other languages.

Use latest Cygwin setup on AppVeyor

Use latest Cygwin setup, rather than what happens to be present on AppVeyor
Also switch to building on x86_64

Tweak formatting in resource files

Tweak the formatting in resource files in places which cause a problem
with translation-toolkit.

Updates and corrections to French translation

Fix warnings with autoconf-2.7

AC_PROG_LEX() with an option (which is needed to avoid a warning) is new
in autoconf-2.7, so also raise minimum autoconf version to 2.70.

Make '--package-manager' imply 'download and install packages'

Refine 147fc15d

Imply both '-D' and '-L' with '-M', as well as '-q' (so that mode gets
definite behaviour rather than 'whatever you chose last time' as well).

Tweak help text for '-D' and '-L' to make more sense when they are used
in combination.

Handle tar type flag 'K' (GNU long link extension)

Handle tar type flag'K' (GNU long link extension)
Also report unexpected filename in headers with long name extension type
Also clean up some cruft in archive_tar.cc

Currently, the only existing package which contains type flag 'K'
entries is f21-backgrounds-extras.

This also now correctly handles tar archive header linknames of exactly
100 characters (even when not preceeded by a 'K' type header), where the
linkname field is not null terminated as it should be, rather than
allowing the 'ustar\0' magic immediately following it to be interpreted
as part of the linkname.

Show overall progress for checking packages in package cache

Show overall progress for testing presence and validating packages in
package cache.

This gives a better sense of progress when installing many packages
which are already cached (e.g. a fresh install, but all required
packagea are already in the package cache, so this step, rather than the
download one, dominates).

Clear download progress/rate text after download

Don't leave download progress/rate text displayed while we compute the
hash of the file we've just downloaded.

Downgrade some xz compression debug output

This not very helpful output goes to verbose log for every decompressed archive.

Reset GUI feedback state after ini parsing

If we're in --local-install mode, we're shortly going to disappear into
packagemeta::ScanDownloadedFiles() which (currently) gives no feedback.

Don't make it look like we're still in the middle of parsing ini files
while we do that.

Also fix a typo introduced in b3cc8ab6.

Unconditionally fixup source package ids

In the case when a given package version appears in setup.ini first
with, then without an install: line, the source package is recreated by
packagemeta::add_version() to add the additional package source site,
but the install package is not - so it now contains a stale source
package id.

Unconditionally fixup source package ids, so that install package points
to the correct source package.

Also internalize libsolv attributes after doing fixup, the changes it
makes are visible (so this was probably never working right).

Add '--batch' to gpg command in 'make upload'

'--batch' is recommended for any non-interative use of gpg.

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