Skip to content

Upgrade to palp 2.21 #40229

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 1 commit into
base: develop
Choose a base branch
from
Open

Upgrade to palp 2.21 #40229

wants to merge 1 commit into from

Conversation

vbraun
Copy link
Member

@vbraun vbraun commented Jun 8, 2025

Also fixes build with gcc-15

@vbraun vbraun mentioned this pull request Jun 8, 2025
Open
2 tasks
@github-actions GitHub Actions
Copy link

github-actions bot commented Jun 8, 2025 •
edited
Loading

Documentation preview for this PR (built with commit 1b5972a; changes) is ready! 🎉
This preview will update shortly after each push to this PR.

dimpase
dimpase approved these changes Jun 8, 2025
View reviewed changes
Copy link
Member

@dimpase dimpase left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

2.21 works on Gentoo, thus...

vbraun pushed a commit to vbraun/sage that referenced this pull request Jun 9, 2025
b858685 
sagemathgh-40229: Upgrade to palp 2.21
    
Also fixes build with gcc-15
    
URL: sagemath#40229
Reported by: Volker Braun
Reviewer(s): Dima Pasechnik
@vbraun
Copy link
Member Author

vbraun commented Jun 9, 2025

Annoyingly this is borked on osx

[spkg-install] Building PALP optimized for 4 dimensions
[spkg-install] Building palp-2.21
[spkg-install] make[5]: *** No rule to make target `poly-d.x', needed by `poly.x'.
[spkg-install] make[5]: *** No rule to make target `class-d.x', needed by `class.x'.
[spkg-install] make[5]: *** No rule to make target `cws-d.x', needed by `cws.x'.
[spkg-install] make[5]: *** No rule to make target `nef-d.x', needed by `nef.x'.
[spkg-install] make[5]: *** No rule to make target `mori-d.x', needed by `mori.x'.
[spkg-install] make[5]: Target `all' not remade because of errors.

because the Makefile uses =-sign after the canned recipe define which is only supported on GNU make 3.82. And Apple, in their infinite wisdom, still ships the absolutely ancient 3.81

@vbraun
Upgrade to palp 2.21
1b5972a 
Also fixes build with gcc-15
@vbraun vbraun force-pushed the upgrade-palp-2.21 branch from f59629c to 1b5972a Compare June 9, 2025 12:47
vbraun pushed a commit to vbraun/sage that referenced this pull request Jun 9, 2025
e5fb550 
sagemathgh-40229: Upgrade to palp 2.21
    
Also fixes build with gcc-15
    
URL: sagemath#40229
Reported by: Volker Braun
Reviewer(s): Dima Pasechnik
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dimpase dimpase dimpase approved these changes

Assignees
No one assigned
Labels
c: packages: standard s: positive review
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@vbraun @dimpase