Refactor return expression causing trouble with GNAT 13 (#1557)

Remove blue background setting from Windows shortcut (#1555)

The shortcut from the Windows installer was setting a background color for PowerShell command prompt. The idea was to match the look of the PowerShell you would normally start, and avoid confusion with the CMD prompt which has a black background. But that didn't work very well as the background color just disappeared when running commands. Also it seems like Windows now uses a black background for the PowerShell anyway.

Fix bugs causing spurious diffs on updates without changes (#1550)

* Fix bugs causing spurious diffs on update

Solutions were the same, but some types internally where using pointers instead
of pointed objects for comparison.

WIP

* Separate style checking from experimental Ada switches

Bugfix: exception when trying to select a tool without the assistant while the configured one is missing (#1551)

* Fix exception when trying to select a toolchain

This problem arised only if trying to select a toolchain with --select <tool>
while the configured tool was missing on disk.

* Test for fix

Documentation about upgrading to alr 2.x (#1547)

Fix regex for version detection of Fedora externals (#1545)

* Fix improper regex in rpm-based detector

* Test for bugfix

* Document pending breaking changes

These require an `alr` or index version bump

* Self-review

Don't fail fast in toolchain matrix CI checks

Remove benign warning during toolchain discovery (#1540)

Use e3 facilities for test skipping (#1537)

Move To_Portable to body and make it a regular function (#1541)

Streamline index forking (#1535)

Harmonize and document test modifier variables (#1536)

Fix deployment over pre-existing dependency dir (#1542)

Fix redeploying of toolchains (#1533)

* Test that fails prior to bugfix

* Fix for cache invalidation

An unrelated bug is fixed that happened when not using an actual binary crate
for a compiler in the testsuite.

Better location for large downloads (#1528)

Discard $XDG_CACHE_HOME in favor of $XDG_DATA_HOME, as some distro guidelines
erroneously propose to mount the cache on tmpfs.

Document PAT creation and hint in `alr publish` help (#1529)

Add missing with statements for Ada.Directories (#1525)

Report the name of an unreadable directory (#1523)

* Report the name of an unreadable directory.

  * src/alire/alire-index_on_disk.adb (New_Handler.Process_Local_Index):
      If the Path isn't a readable directory (e.g. because it doesn't
      exist), add it to the message in Outcome_Failure.
  * testsuite/tests/index/local-index-not-found/test.py: updated for
      new 'not a readable directory' report format.

* Handle Windows separator doubling.

Build appImage with GNAT 10 on Ubuntu 20.04 (#1519)

Update list of supported platforms in index spec (#1513)

Merge pull request #1514 from atalii/doc-chdir

Document --chdir in user-changes

Document --chdir in user-changes

Offer uninstalled compiler as fallback with warning (#1508)

* Allow solving with unistalled tools as fallback

* Fix precedence of installed compiler over remote ones

* Add warning in solution diff about toolchain download

* Fallback on installed compiler over uninstalled one

This is after checking that the preferred compiler cannot fulfill the compiler
dependency.

* Tweaks to restore former behaviors

* Fix test that inadvertently used virtual crate

It should have been real all along without alternatives

* Rebase tweaks

* Reword information message

Bump version post-beta1

Load configuration earlier from proper location (#1501)

* Move configuration loading earlier

Also make sure it's impossible to use configuration before the proper path is
set.

* New test

* Better encapsulation

* Windows fix

Filter out Unicode in SI_Unit conversion (#1506)

New warning when external GNAT selected but another compiler version is needed (#1500)

* New warning for externally selected gnat

* New test

Fix regression when `alr with`ing a virtual crate without actual releases (#1507)

* Do not report a virtual crate as inexistent

* Test to avoid future regression

Use Ada quotes in test skeletons

Amend style switches (expand -gnatyg). (#1497)

-gnatyg is "check standard GNAT style rules, same as ydISux", but in
GCC 14 it will include a new switch, -gnatyz (check parentheses not
required by operator precedence rules) which will mean that use of
extra parens to clarify intent without deep knowledge of the rules
will trigger fatal warnings; for example in alire-conditional_trees.adb

   function Contains_ORs (This : Tree) return Boolean is
      ((not This.Is_Empty) and then This.Root.Contains_ORs);
       ^

Refer GCC Bugzilla PR 112446.

  * alire_common.gpr (Style_Check_Switches): Remove -gnatyg and replace
    by the pre-GCC 14 equivalent.

Fix unwanted stack trace on `alr publish --tar` with broken manifest (#1499)

* Avoid unhandled exception in `alr publish --tar`

Fixes #1214

Also unify a couple of error messages

* Update test to check this fix

* Self-review

Fix bug for interactive crate descr ending with `"` (#1495)

Add Savannah GNU repository to trusted remotes

Diagnose missing executable when running an action (#1493)

* Fix command not found in actions

* New test

Don't trigger `alr` help on `alr exec -- command --help` (#1491)

* Fix "-h/--help" being catched after "--"

* Bump clic dependency

* Fix test on Windows

Simpler error on `alr with` for unknown crates (#1489)

Generate loadable project during `alr init` (#1486)

Fix list of default licenses (#1490)

* Remove duplication in error message output

* Early detect bad license among choices

* Fix bad license

* Fix error deduplication

* New test for default licenses

Rename AppImage workflow job

implement -C/--chdir flag (#1479)

* implement -C/--chdir flag

* use `alr` instead of Alire in help text.

* test --chdir flag

* clean up with-chdir test

Fix documented default prefix for install (#1482)

See comments in issue #1481

Honor the NO_COLOR environment variable (#1480)

According to https://no-color.org/

remove duplicate section of user-changes.md (#1477)

"Enable shared dependencies by default" was in there twice.

Separate AppImage to own workflow (#1475)

Merge remote-tracking branch 'alire/release/2.0'

Bump version to 2.0-beta1

Better triggers for actions on release creation

Better triggers for actions on release creation

Merge remote-tracking branch 'alire/master' into release/2.0

Enable shared dependencies by default (#1449)

* Set shared dependencies as the default

* user-changes.md: Explain shared builds

bugfix: the wrong folder was searched for nested crates (#1473)

We were attempting to enter a folder that not always will exist.

Better collision avoidance for tmp filenames (#1469)

Merge remote-tracking branch 'alire/master' into release/2.0

Auto-update index periodically (#1447)

* Auto-refresh indexes after some time (default 24h)

* ask user on first autoupdate

* User changes

* Test behavior on network failure

New `alr printenv --last-build` (#1444)

* New `alr printenv --last-build`

This may be useful when no direct editing of manifests is convenient, as
otherwise `printenv` will always use the manifest's configuration.

* make new behavior the default with fallback

* CI fixes

Merge remote-tracking branch 'alire/master' into release/2.0

Comprehensive shared builds tests and related tweaks (#1459)

* Added `shared-build` driver

* Added `build_mode` key to the Python driver

* Fix config generation in `alr update` in shared mode

* Remove need for compiler in absence of dependencies

* Fixed monorepo tests and base dir

* Delay config generation in sandboxed mode too

* Include explicit switches in build hash inputs

This is necessary because for shared builds we're never regenerating the
configuration, so changes in switches went undetected

* Require complete config for sandboxed build

* Self-review

* Update new tests

Remove ".git" after cloning, with option to disable (#1463)

* Remove ".git" after cloning, with option to disable

Also make clones shallow (they already were for pins)

* Fix test

Fix corner case related to git+file:// URLs (#1465)

Omit global switches in subcommand-specific help (#1462)

* Omit global switches in subcmd-specific help

* Update clic dependency

Remove ".git" after cloning, with option to disable (#1463)

* Remove ".git" after cloning, with option to disable

Also make clones shallow (they already were for pins)

* Fix test

Fix corner case related to git+file:// URLs (#1465)

Omit global switches in subcommand-specific help (#1462)

* Omit global switches in subcmd-specific help

* Update clic dependency

Fix location of default install prefix (#1461)

Fix location of default install prefix

Set shared dependencies as the default

user-changes.md: Explain shared builds

Comprehensive tests for shared builds

Added `shared-build` driver

Added `build_mode` key to the Python driver

Fix config generation in `alr update` in shared mode

Remove need for compiler in absence of dependencies

Fixed monorepo tests and base dir

Delay config generation in sandboxed mode too

Include explicit switches in build hash inputs

This is necessary because for shared builds we're never regenerating the
configuration, so changes in switches went undetected

Require complete config for sandboxed build

Self-review

Print UTF-8 strings using `GNAT.IO` in `Alr.Commands` subsystem (#1458)

Supersedes #1457

Fixes #1456

Remove spurious -g (#1454)

Fix bugs for corner cases with shared dependencies (#1453)

* Fix a couple of bugs related to new build system

* Test for the fixed bug

* Self-review

Cleanup after shared releases impl (#1448)

* Omit default mention in shared deps help

* Remove Export_Env arguments all-around

Now that we delay all actions to build time, we can set environment just at
that time and avoid complications.

* Debug workflow and dependency hashes

* Self-review

Cosmetic change to binary origin image when unavailable (#1446)

Fix bug in dependency traversal order (#1445)

* Fix bug in solution traversal related to linked releases

* New test

* CI fixes

Remove old release-sharing mechanism remnants (#1451)

Use the same Find function in "show" as in "show --jekyll" (#1452)

This fixes #1402.

New `ALR_TRACEBACK_ENABLED` env. var. (#1450)

* Add traceback enabling env var

* Test for new config var

Run delayed post-fetch actions during build (#1441)

* Refactored flags into new crate

* Delay post-fetch to first build (ensures complete env)

* Folderize flags

* Self-review

* Simpler flag use, better encapsulation

* Fix Windows-only test

Add some hardcoded externals to hash input (#1440)

Include hash of dependencies in build hash (#1439)

Since a dependency may use different specs based on config values, environment
variables, GPR externals... we include the hash of dependencies in dependents'
hash inputs.

Reuse hashing mechanism to regen config in sandboxed mode (#1438)

* Regenerate configs on demand (sandboxed deps)

Leverage the new build hashing to make sure when a configuration needs
regenerating. This is pessimistic but safer than what we had, and more
future-proof.

* fix wrong switches generation

* test fixes

* New test, checking no unneeded rebuilds happen

* Fixes for CI-detected issues

Generate crate config on demand for shared dependencies (#1435)

We needn't to generate it more than once, when the unique build folder is
created.

Add crate config variables to build hash (#1434)

* Add crate configuration values to build hash inputs

* Tests for configuration hashing

Delay syncing shared build dir until the actual build (#1433)

This is necessary because the build dir hash depends on the crate configuration
being complete, and this is only required at the time of building.

Add crate environment to build hash inputs (#1432)

* Add crate-modified env vars to hash input

* Test environment hashing

Refactor config builtins (#1431)

* Refactor configuration builtins

Gives them a tagged type that simplifies use and aglutinates default,
description, checks, which were previously not formally required or scattered.

* Colorize help output

Add compiler version to build hash (#1442)

* Update pins and dependencies

* Include GPR externals in hash computation

* Avoid recursivity when loading environment

* Test of hashing externals

* Add compiler version to the build hash

* Tests for compiler in build hash input

* Improvements to test

Add GPR externals and env. vars. to build hash computation (#1428)

* Update pins and dependencies

* Include GPR externals in hash computation

* Avoid recursivity when loading environment

* Test of hashing externals

Fix use of canary file during `alr install` (#1429)

New Force_Delete (#1426)

Replace `rsync` with `GNATCOLL.VFS.Copy` (#1427)

MacOS; export PATH variables for includes and libraries. (#1420)

* MacOS; export PATH variables for includes and libraries.

Homebrew and MacPorts install include files and libraries in places
where GCC won't look by default.

GCC will use these environment variables if set:

   C_INCLUDE_PATH     for C includes
   CPLUS_INCLUDE_PATH for C++ includes
   LIBRARY_PATH       for libraries

Both of the distribution managers place (symbolic links to) include
files in ${top_level}/include and libraries in ${top_level}/lib.

   For Homebrew on Intel silicon, top_level is normally /usr/local.
   For Homebrew on Apple silicon, top_level is normally /opt/homebrew.
   For MacPorts,                  top_level is normally /opt/local

  * src/alire/alire-platforms-current.ads (Load_Environment): add note
      on macOS use.
  * src/alire/os_macos/alire-platforms-current__macos.adb
    (context): added Alire.Environment (was limited), Ada.Directories.
    (Brew_Access): new.
    (Homebrew_Present): if Brew_Access is not null.
    (Detected_Distribution): made into an expression function.
    (Containing_Containing_Dir): new, used in Distribution_Root.
    (Distribution_Root): reworked.
    (Load_Environment): if either distribution is present, arrange to
      export the environment variables to suit.

* Update testsuite to match new macOS distribution detection.

  * testsuite/drivers/helpers.py (distribution): if on macOS, check whether
      the distribution management tool is on the PATH. We used to check for
      the environment variable HOMEBREW_PREFIX, but users don't have to
      arrange for this to be set in order to run Homebrew.

      First, if 'brew' is found, the distribution is Homebrew.
      If not and 'port' is found, the distribution is MacPorts.
      Otherwise, the distribution is unknown.

* In the macOS CI workflow, run the test script once only.

  * .github/workflows/ci-macos.yml (Run test script): remove the second
      call, which set up HOMEBREW_PREFIX (now no longer used by alr),
      and remove the note '(without Homebrew)' in the first.

Use build profile in build hash (#1425)

* Compute hash using build profile

* Fix problem with multiple hashes in one run

Rooted in that default build profiles were used first and later the actual
profiles caused different folders to be used, in turn causing errors with the
conflicting CRATE_ALIRE_PREFIX variables.

* Write hash inputs to build dirs

* Testsuite additions and fixes

* Self-review

* Update pins and dependencies

Refactor Alire.Shared into Alire.Toolchains (#1423)

Trivial safeguard in `Force_Delete` (#1422)

New shared builds mechanism (mockup) (#1419)

* New shared builds mechanism (mockup)

* Self-review

* Fixes for Windows

Don't treat warnings as erros when no style checks (#1412)

Better random generation of temp file names (#1414)

The issue was with the default random seed being the same during a whole
second, so batch runs in the testsuite did clash from time to time.

Fix nightly build (#1413)

New `alr publish --request-review` (#1409)

* Descriminate more PR states

Only missing is when there are changes requested

* Colored states

* Fixed review evaluation

* New `alr publish --request-review`

* Document new feature in user visible changes

* Self-review

New `alr publish --cancel` (#1406)

* New `alr publish --cancel <num> --reason <text>`

* Added JSON escaping

* Update alire.toml pins

* user-changes.md: new `--status`, `--cancel` switches