Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[liepring] Update to 2.6 #310

Merged
merged 1 commit into from
Apr 14, 2022
Merged

[liepring] Update to 2.6 #310

merged 1 commit into from
Apr 14, 2022

Conversation

gap-package-distribution-bot[bot]
Copy link
Contributor

@gap-package-distribution-bot gap-package-distribution-bot bot commented Apr 12, 2022

No description provided.

@gap-package-distribution-bot gap-package-distribution-bot bot added the automated pr Automatically applied to PRs created by a GH workflow label Apr 12, 2022
@gap-package-distribution-bot
Copy link
Contributor Author

gap-package-distribution-bot bot commented Apr 12, 2022

Package Evaluation Report for GAP master

Job Properties

Testing: master/2022-04-14-16:45:32-0dcbdc69 vs master/2022-04-14-03:02:53-929766ad

Generated by Workflow: https://github.com/gap-system/PackageDistro/actions/runs/2168323615

In total, 152 packages were tested, out of which 120 succeeded, 12 failed and 20 were skipped.

❗ Packages still failing

12 package(s) failed tests also on the previous version.

Click to show package(s)!

✔️ Packages still succeeding

120 package(s) succeeded tests also on the previous version.

Click to show package(s)!

➖ Packages that were skipped

20 package(s) skipped tests also on the previous version.

Click to show package(s)!

@gap-package-distribution-bot gap-package-distribution-bot bot force-pushed the automatic/liepring branch 3 times, most recently from ad00c72 to f61fe56 Compare April 13, 2022 11:09
@fingolfin
Copy link
Member

Something is wrong with our test setup: for some reason, liepring is missing among the packages in the container. Looking at the logs I see:

...
downloading https://github.com/gap-packages/liepring/releases/download/v2.6/liepring-2.6.tar.gz to _archives/liepring-2.6.tar.gz
...
2022-04-13T10:11:58.3985480Z removing _archives/liepring-1.9.2.tar.gz
...
2022-04-13T10:12:28.7082991Z Extracting /home/runner/work/PackageDistro/PackageDistro/_archives/liepring-2.6.tar.gz

But then later, when BuildPackage.sh is run over all packages, liepring is missing:

2022-04-13T10:13:26.8070496Z ##[group]liealgdb-2.2.1
2022-04-13T10:13:26.8070996Z 
2022-04-13T10:13:26.8071492Z Wed Apr 13 10:13:26 UTC 2022
2022-04-13T10:13:26.8071672Z 
2022-04-13T10:13:26.8072132Z ==== Checking liealgdb-2.2.1
2022-04-13T10:13:26.8074463Z No building required for liealgdb-2.2.1
2022-04-13T10:13:26.8085430Z ##[endgroup]
2022-04-13T10:13:26.8189064Z ##[group]liering-2.4.2
2022-04-13T10:13:26.8189486Z 
2022-04-13T10:13:26.8192082Z Wed Apr 13 10:13:26 UTC 2022
2022-04-13T10:13:26.8192416Z 
2022-04-13T10:13:26.8192868Z ==== Checking liering-2.4.2
2022-04-13T10:13:26.8193709Z No building required for liering-2.4.2
2022-04-13T10:13:26.8202033Z ##[endgroup]

And subsequently in e.g. LoadAllPackages and later in the package tests we see reports that GAP is not able to find it at all.

Really strange

@fingolfin
Copy link
Member

Seems to have been temporary?

@fingolfin fingolfin closed this Apr 14, 2022
@fingolfin fingolfin reopened this Apr 14, 2022
@fingolfin fingolfin merged commit 1c4ece0 into main Apr 14, 2022
@fingolfin fingolfin deleted the automatic/liepring branch April 14, 2022 17:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automated pr Automatically applied to PRs created by a GH workflow
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant