Skip to content

Commit

Permalink
Replace shell LHAPDF installation with Python-based one
Browse files Browse the repository at this point in the history
It ensures that the tools are available on every platform, since the
script only makes use of Python standard library.
Otherwise it was not even possible to rely on `wget` or `realpath`...
  • Loading branch information
alecandido committed Mar 26, 2024
1 parent 9f39dec commit 6175f4e
Showing 1 changed file with 3 additions and 15 deletions.
18 changes: 3 additions & 15 deletions .github/workflows/cli-wheels.yml
Original file line number Diff line number Diff line change
Expand Up @@ -45,11 +45,7 @@ jobs:
pip install maturin auditwheel
- name: Install non-Rust dependencies
run: |
for script in {install,download,variables}; do
curl -O \
https://raw.githubusercontent.com/NNPDF/workflows/v2/packages/lhapdf/${script}.sh
done
sh install.sh
python maintainer/install-lhapdf.py
- name: Build wheels
run: |
# `--find-interpreter` is needed to generate wheels for
Expand Down Expand Up @@ -94,11 +90,7 @@ jobs:
pip install maturin delocate
- name: Install non-Rust dependencies
run: |
for script in {install,download,variables}; do
curl -O \
https://raw.githubusercontent.com/NNPDF/workflows/v2/packages/lhapdf/${script}.sh
done
sh install.sh
python maintainer/install-lhapdf.py
- name: Build wheels
run: |
# `--find-interpreter` is needed to generate wheels for
Expand Down Expand Up @@ -144,11 +136,7 @@ jobs:
pip install maturin delvewheel
- name: Install non-Rust dependencies
run: |
for script in {install,download,variables}; do
curl -O \
https://raw.githubusercontent.com/NNPDF/workflows/v2/packages/lhapdf/${script}.sh
done
sh install.sh
python maintainer/install-lhapdf.py
- name: Build wheels
run: |
# `--find-interpreter` is needed to generate wheels for
Expand Down

0 comments on commit 6175f4e

Please sign in to comment.