- changed status to resolved
Don't use GAPInfo.PackagesLoaded
Issue #18
resolved
Right now, there is code like this in fining:
if not "grape" in RecNames(GAPInfo.PackagesLoaded) then
Error("You must load the Grape package in order to use IncidenceGraph.\n");
fi;
But note that GAPInfo.PackagesLoaded
is not documented. The equivalent way using documented APIs is this:
if not IsPackageMarkedForLoading("grape", "") then
Error("You must load the Grape package in order to use IncidenceGraph.\n");
fi;
In GAP 4.10, there will also be a function IsPackageLoaded
, which performs an additional check and for which the second (version) argument is optional, so that then you could also write:
if not IsPackageLoaded("grape") then
Error("You must load the Grape package in order to use IncidenceGraph.\n");
fi;
Comments (1)
-
repo owner - Log in to comment
done! new version of fining will require gap4r10