The motivation for deduplicating GAP_ROOT_PATHS still stands, but the
old comment about the sage...

Bring the sage list of (attempted) autoloads to parity with the
default GAP list.

With only libgap-dev, we are missing the "gap" executable.

Several doctests in this module require the GAP package PolyCyclic.
We add "# optional - gap_pac...

Several doctests in this module require the GAP package PolyCyclic.
We add "# optional - gap_pac...

Instead of leaving GAP_ROOT_PATHS blank for the gap SPKG, we now use
'${prefix}' placeholders to...

This more closely resembles what sdh_configure() is doing.

Based on,

* https://packages.fedoraproject.org/pkgs/gap/
* https://build.opensuse.org/packa...

Bring the sage list of (attempted) autoloads to parity with the
default GAP list.

The doctest for the normal_subgroups() of DihedralGroup(8) cannot
simply list them all, because ...

Several doctests in these two modules require the GAP package PolyCyclic.
We add "# optional - g...

From the GAP documentation:

By default, some needed and suggested GAP packages are loaded, if...

Instead of atlasrep and tomlib, let's use primgrp and smallgrp in this
doctest for test_packages...

One of our examples for MathonPseudocyclicStronglyRegularGraph()
constructs a matrix from a norm...

To support a system GAP installation, we now prepend Sage's old
GAP_LIB_DIR and GAP_SHARE_DIR di...

Otherwise it displays the big, ugly, fully-qualified name.

We added this for the CI to try, and it didn't work.

Several doctests in this module require the GAP package PolyCyclic.
We add "# optional - gap_pac...

Now that we're passing "-A" to the GAP command, the PolyCyclic package
won't be loaded even if i...

The GAP subpackages listed at,

https://packages.fedoraproject.org/pkgs/gap/

are not included...

These two variables are best combined into one, GAP_ROOT_PATHS. This
is how they're used anyway:...

Be nice and mention to users that only the PolyCyclic package (because
Sage needs it) will be au...

Installing gap_packages now means that they are loadable, not
necessarily that they are loaded. ...

Change the one "checking the" message to match all of the "checking
for" messages.

Mention that we're checking for only the bare minimum set of packages
needed for the sage test s...

Replace the string '${prefix}' with SAGE_LOCAL in GAP_ROOT_PATHS. The
spkg-configure.m4 for GAP ...

It's possible for the gap CLI to be installed but libgap to be
unusable, for example on systems ...

Now that we're initalizing libgap with "-A", the PolyCyclic package
won't be loaded even if it i...

The order in which DihedralGroup(8).normal_subgroups() is listed is
apparently not fixed. To gua...

We have a GAP test in this document that prints (directly via GAP) the
normal subgroups of Dihed...

While testing for a GAP package, we now try to load the package
first. Without this, testing of ...

Now that polycyclic and a few other packages have been moved back into
the core gap SPKG, the de...

We used to prepend two sage-specific paths to GAP_ROOT_PATHS here,
but we now handle that in gap...

Reorganize the GAP variables in sage.env, and then move the _gap_cmd
processing out of sage.env ...

When initializing libgap, we use "-l" to specify a list of root paths.
In the past we constructe...

The presence of a GAP package is (at least) trinary: it can be
installed and loaded, installed a...

Our doctest for as_permutation_group() checks that algorithm="smaller"
produces a group that is ...

There's a test for unhide() in this module that assumes the PolyCyclic
GAP package will always b...

Otherwise, when there are duplicates, we list every package twice (or
more) in all_installed_pac...

This tells GAP not to try to load a bunch of packages that aren't part
of the default install, t...

It's just a better name for the command that runs a GAP command.

These can be loaded at runtime with, for example,

sage: libgap.LoadPackage("PolyCyclic")

And...

Several doctests in this module require the GAP package PolyCyclic.
We add "# optional - gap_pac...

Several doctests in this module require the GAP package PolyCyclic.
We add "# optional - gap_pac...

Without this, when certain tests fail, we print a bunch of junk to the
screen (and to config.log).

When the system GAP is installed and the SPKG gap is not, the extra
gap_packages need to know wh...

This hides the "user" path for GAP packages. We don't do it for libgap,
and I don't think it mak...

For some reason, GAP can decide to return a different set generators
for one of the subgroups in...

Instead of searching only GAP_LIB_DIR for GAP packages, we now search all
entries in GAP_ROOT_PA...

If the user doesn't want to use either the sage package manager or his
system's package manager,...

This more closely resembles what sdh_configure() is doing.

If we're using the system GAP, we still want sage's CLI gap command to
be able to find any GAP p...

These two variables were concatenated into a string that is eventually
hashed. Now, the GAP_ROOT...

Upon review it was agreed that the gap SPKG should not aim for a
minimal installation. We bring ...

The example commands in this file are doctested, so they need
"optional" tags and the expected o...

There's a long list of "recommended" packages that we install for GAP
because, otherwise, GAP co...

From the GAP documentation:

By default, some needed and suggested GAP packages are loaded, if...

This variable whose counterpart exists in sage.env already will be set
by Sage's ./configure scr...