Skip to content

Add license-files and no-binary-package entries#42229

Open
cxzhong wants to merge 2 commits into
sagemath:developfrom
cxzhong:patch-2
Open

Add license-files and no-binary-package entries#42229
cxzhong wants to merge 2 commits into
sagemath:developfrom
cxzhong:patch-2

Conversation

@cxzhong
Copy link
Copy Markdown
Contributor

@cxzhong cxzhong commented May 22, 2026

This fixes the uv building

📝 Checklist

  • The title is concise and informative.
  • The description explains in detail what this PR is about.
  • I have linked a relevant issue or discussion.
  • I have created tests covering the changes.
  • I have updated the documentation and checked the documentation preview.

⌛ Dependencies

@github-actions
Copy link
Copy Markdown

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

@cxzhong cxzhong closed this May 22, 2026
@cxzhong cxzhong reopened this May 22, 2026
@cxzhong cxzhong closed this May 22, 2026
@cxzhong cxzhong reopened this May 22, 2026
@cxzhong cxzhong marked this pull request as ready for review May 22, 2026 13:59
@cxzhong
Copy link
Copy Markdown
Contributor Author

cxzhong commented May 23, 2026

The uv build: https://github.com/cxzhong/sage/actions/runs/26278272434
It does not appears the segfault about cypari2

@cxzhong cxzhong requested a review from tobiasdiez May 28, 2026 09:25
@cxzhong
Copy link
Copy Markdown
Contributor Author

cxzhong commented May 28, 2026

Now, the solution is add no-binary-wheel into the pyproject.toml, since they are interface package, building from source can prevent mismatch cause segfault

Copy link
Copy Markdown
Contributor

@tobiasdiez tobiasdiez left a comment

Choose a reason for hiding this comment

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

This is still sub-optimal but probably the best we can do at the moment. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants