You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
In PR #79@zickgraf implemented attaching "the" manual PDF to a release.
Unfortunately, this lead to a regression, because there are packages which have multiple manuals; hence PackageDoc can be a record or a list of records. Thus this line can run into an error:
The immediate workaround would be to use something like this:
r := GAPInfo.PackageInfoCurrent.PackageDoc.PDFFile;;
if IsList(r) then r:=r[1]; fi;
Print("PDFFile=\"",r,"\"\n");
However, that would then only upload the "first" PDF; but what if there are multiple manuals, each with a PDF? Perhaps we should upload them all? I guess one could try to set PDFFile to a bash array of manual names...
The text was updated successfully, but these errors were encountered:
In PR #79 @zickgraf implemented attaching "the" manual PDF to a release.
Unfortunately, this lead to a regression, because there are packages which have multiple manuals; hence
PackageDoc
can be a record or a list of records. Thus this line can run into an error:The immediate workaround would be to use something like this:
However, that would then only upload the "first" PDF; but what if there are multiple manuals, each with a PDF? Perhaps we should upload them all? I guess one could try to set PDFFile to a bash array of manual names...
The text was updated successfully, but these errors were encountered: