Merge branch 'issue-1523' into calamares

[users] Run an expected-to-fail test in creating groups

[users] Expand tests for groups a little

CMake: comment on BUILD_AS_TEST

Merge pull request #1561 from gportay/partition-set-swap-partition-name

[partition] Allow a configurable swap Partition name

Merge pull request #1562 from gportay/partition-group-efi-things-together

[partition] Group EFI things together

[libcalamares] Fix tests failures from JobQueue asserts

Some compile flags changed recently, triggering assert()
in the jobqueue when there is more than one. There's no
real reason for JobQueue to be a singleton, but it wants
to be. So clean up pointers a little more enthusiastically.

Merge branch 'partition-warnings' into calamares

[partition] Reduce warnings from KPMCore

- make KPMHelper.h an "everything include" and suppress warnings
  from it, then use it in the tests.

[partition] Reduce warnings

- Add a helper header that munges the warnings-settings so that
  KPMcore headers can be included.

[locale] Reduce warnings (C++17)

Now that Calamares is compiled as C++17, we get this:

src/modules/locale/timezonewidget/TimeZoneImage.cpp:28:55: warning: out-of-line definition of constexpr static data member is redundant in C++17 and is deprecated [-Wdeprecated]
/* static constexpr */ const QSize TimeZoneImageList::imageSize;

[partition] Reduce warnings (Clang, FreeBSD)

The code doesn't match the comment: there are no by-ref captures
in the code, and the shadowing of parameters and local variables
is confusing. Remove one variable that is passed in as an argument
(and just pass the value as argument) and copy-capture the other
rather than doing weird argument passing.

[partition] Reduce warnings (Clang, FreeBSD)

- remove unused this captures from lambda
- rename variables that are short, cryptic, and shadowed
- remove documentation for parameters that don't exist

[partition] Group EFI things together

Docs: fix the table

[partition] Update partition.conf

Docs: try to get the table of links right

[partition] Allow a configurable swap Partition name

This commit adds the new configuration `swapPartitionName` to the file
partition.conf.

This option sets the partition name to the swap partition that is
created. If this option is unset, the partition is left unnamed.

[keyboard] SPDX-tagging

Merge pull request #1558 from gportay/partition-cleanups

[partition] cleanups

Merge pull request #1559 from gportay/libcalamres-cleanups

[libcalamres] cleanups

[partition] Rename execute to createPartitions

[partition] Fix coding style

Fix missing SPDX-FileCopyrightText in header

[partition] Rename the KPMCore Partition local to part

[partition] Rename iterator in for loop to entry

[partition] Simplify the method execute

- Rename the "size" locals using "sectors" in their name. Size may be
  confusing or not enough specific as it can be interpreted a size in
  Byte.
    partSizeMap -> partSectorsMap,
    totalSize -> totalSectors,
    availablesize -> availableSectors,
    size -> sectors,
    minSize -> minSectors
    maxSize -> maxSectors
- Create a the new local currentSector to iterate over the sectors;
  instead of using the parameter firstSector.
- Remove the variable end that does not help much; too many variable
  already. Expand its expression instead.

[partition] Introduce new constructors for PartitionEntry

- Introduces new constructors for PartitionEntry: copy constructory and
  constructor with all attributes.
- Use the new constructor in method addEntry().

[partition] Simplify the retrieval of the size attributes

- The variant helper toString() takes a default value since commit
  c9f942ad6 ([libcalamares] Add default value to variant helpers).
- Set the default value to 0 and simplify the retreival of size values
  by calling the helper toString() and removing the temporary variables.

[libcalamares] Fix constness of functions

[libcalamares] Fix coding style

[partition] Remove the call the method contains

- The variant helper getString() calls contains() already.

[partition] Move initLayout logic to object PartitionLayout

- The logic of the method initLayout belongs to the object
  PartitionLayout. Move logic to that object.
- Use a single method initLayout in object PartitionCoreModule.
- Member m_partLayout in object PartitionCoreModule is no longer
  allocated.

[partition] Remove unused PartitionLayout constructor

- The constructor PartitionLayout with PartitionEntry appears to be
  unused since its always. Drop it!

[partition] Remove unused addEntry method

- The method addEntry with PartitionEntry appears to be unused since its
  always. Drop it!

[keyboard] Fix group-switcher multiplying

FIXES #1556

Merge branch 'calamares' into issue-1523

CI: use a larger tmpfs for the build (my local build/ is 112M)

i18n: [python] Automatic merge of Transifex translations

i18n: [desktop] Automatic merge of Transifex translations

i18n: [calamares] Automatic merge of Transifex translations

Changes: add Vietnamese translation

Changes: document Artem's work

Merge branch 'fix-keyboardq' into calamares

This lands Artem's work on supporting non-ASCII layouts better,
which currently only applies to Russian layouts -- it works with
a table, so it is easy to extend for other layouts.

The QML and Widget steps now completely share the Config backend,
which is why there's so many commits here: the Widget page needed
to have most of its code ripped out, and the models for
keyboard data were broken in various ways and needed fixing.

FIXES #1211

[keyboard] Reduce debugging output

[keyboard] Simplify back down to lambdas

- With debugging and untangling done, the lambdas are simple
  and short, so return to using them. One point of improvement
  might be to give the models suitable slots themselves,
  to avoid even this indirection.

[keyboard] Sanity in setCurrentIndex() parameters

[keyboard] Tidy up the debugging output, add docs

[keyboard] Replace broken-ish variants model with k-v list

- Use the just-refactored XKBListModel to store the xkb key-value
  pairs for variants, drop most of the complicated implementation,
  store just a single list of values.

[keyboard] Factor out a 2-column k-v list

[keyboard] Use the models from Config

- Remove code that is duplicated in Config.
- Hook up UI for physical keyboard model, and back.
- For now, introduce some named slots with debugging output.
  This makes debugging a lot easier since we have function names
  to work with rather than anonymous lambdas

[keyboard] Hook up the model-selection again

[keyboard] Tear up connections between widgets and model

[keyboard] Restore the notion of PC105 default index

[keyboard] Apply coding style

[keyboard] Re-do the keyboard physical models model from scratch

[keyboard] Typo in comment

[keyboard] Rename UI widgets to make code clearer

[keyboard] Move remaining variables to Config

- Config already *has* everythind, but drop the useless copies
  and duplicated code from the Page
- Plug the models model into the Page
- While here, document the model / layout / variant distinctions

The code doesn't fill the UI properly, and the drop-down
for the models combobox is not right, but at least the data
is shared.

[keyboard] Matching of layouts is very suspect

Merge branch 'calamares' into fix-keyboardq

Merge pull request #1553 from LordTermor/calamares

Initial additional layout support for non-ASCII layouts

[keyboard] Simplified variable name

[keyboard] Minor logic rework

[keyboard] Adjusted indents

[keyboard] fixed typo

[keyboard] Remove superfluous variable (set to true, never changed)

[keyboard] Implemented X11 config writing for additional layout

[keyboard] Fixed condition bug

[keyboard] Added explanatory comment for xkbmap_query_grp_option

[keyboard] Swapped primary and additional layouts in selection

[keyboard] Removed unnecessary repacking in xkbmap_layout_args

[keyboard] Minor additional layout info rework

[keyboard] Support for additional layout if current layout is not ASCII-
capable in live system

[keyboardq] Remove superfluous variable (set to true, never changed)

[keyboardq] Fix include style, remove unnecessary declarations

[keyboard] Migrate to Config::onActivate() which is badly-named

[keyboard] Remove unused code

- this has already migrated to Config but had not been removed locally

[keyboard] Use Config's own finalize()

[keyboard] Replace own copy of status by the one from Config

[keyboard] Use Config methods rather than own copy

- this continues the port of the keyboard module to use the Config
  object, which was horribly botched earlier.

[keyboard] Rename mysterious Config::init() to descriptive name

[keyboardq] Load configuration into Config object

[keyboardq] Drop unnecessary variable -- just use the Config status

Changes: credits for this round

CMake: restore NOTREACHED, without the macro-mess

- gcc (up to at least version 10) is worse at recognizing that all
  cases have been handled, so it complains about all the switches
  that cover enum values.

[keyboardq] fix build

[libcalamares] Search for balance between warnings and annotations

- CI's gcc is too old to analyse a switch() for completeness,
  so the CI build fails.

[partition] Don't needlessly expose a test symbol

README: switch to C++17.

CMake: drop the NOTREACHED macro

 - both clang and g++ support __builtin_unreachable(); (as Kevin
   Kofler pointed out) so we don't need the macro to do different things;
 - the compilers have gotten better at detecting unreachable code,
   so instead of inserting macros or fiddly bits, just drop them
   and the unreachable code they comment.

CMake: prefer normal C++17 [[fallthrough]] annotation

CMake: don't put linker flags in compile-flags variables

[libcalamares] Avoid object-slice

g++ warns that error_already_set is polymorphic, and we're catching
by-value (although we don't use that value). Avoid that.

[libcalamaresui] Warnings-- for Qt 5.15 deprecations

Introduce a GUI-oriented compatibility header that introduces aliases
for some enum values that are deprecated in Qt 5.15

[libcalamares] Remove useless variable

- describe() is for debugging purposes, doesn't need to calculate
  whether the requirements are accepted.

CMake: simplify C and C++ flags

- reduce the difference between clang and g++ builds, factor
  common flags out of the CMake-if
- drop special boost-warning-suppression, we do that differently
  most of the time in the affected source files

CMake: switch to C++17

[partition] Remove unused variables

[partition] Warnings-- in tests related to virtual destructors