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.

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

openblas: Remove python from dependencies #34835

Closed
mkoeppe opened this issue Dec 9, 2022 · 8 comments
Closed

openblas: Remove python from dependencies #34835

mkoeppe opened this issue Dec 9, 2022 · 8 comments
Labels
c: build p: major / 3 t: enhancement
Milestone
sage-9.8

Comments

@mkoeppe
Copy link
Contributor

mkoeppe commented Dec 9, 2022

The installation uses sage-bootstrap-python, so the order-only dependency on $(PYTHON) can be removed

CC: @dimpase

Component: build

Author: Matthias Koeppe

Branch/Commit: e6bb8e6

Reviewer: Dima Pasechnik

Issue created by migration from https://trac.sagemath.org/ticket/34835
@mkoeppe mkoeppe added this to the sage-9.8 milestone Dec 9, 2022
@mkoeppe
Copy link
Contributor Author

mkoeppe commented Dec 9, 2022

Branch: u/mkoeppe/openblas__remove_python_from_dependencies

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Dec 9, 2022

Commit: e6bb8e6

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Dec 9, 2022

New commits:

e6bb8e6build/pkgs/openblas/dependencies: Remove python

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Dec 9, 2022

Author: Matthias Koeppe

@dimpase
Copy link
Member

dimpase commented Dec 9, 2022

comment:3

lgtm

@dimpase
Copy link
Member

dimpase commented Dec 9, 2022

Reviewer: Dima Pasechnik

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Dec 9, 2022

comment:5

Thanks!

@vbraun
Copy link
Member

vbraun commented Dec 14, 2022

Changed branch from u/mkoeppe/openblas__remove_python_from_dependencies to e6bb8e6

@vbraun vbraun closed this as completed in eaff61b Dec 14, 2022
kryzar pushed a commit to kryzar/sage that referenced this issue Feb 6, 2023
@mkoeppe
Merge sagemath#34835
50b8e2e
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
c: build p: major / 3 t: enhancement
Projects
None yet
Milestone
sage-9.8
Development

No branches or pull requests
3 participants
@vbraun @dimpase @mkoeppe