Skip to content

Conversation

@boegel
Copy link
Member

@boegel boegel commented Jun 7, 2025

(created using eb --new-pr)

$CARGO_NET_OFFLINE used to be set for every extension being installed (sort of by accident), but that's no longer the case due to the changes in easybuilders/easybuild-framework#4868, so this fixes a regression (not forcing cargo to not download crates) introduced in EasyBuild v5.1.0

…go` easyblock, so it's set for every extension when `CargoPythonBundle` easyblock is used
@boegel boegel changed the title set $CARGO_NET_OFFLINE to true in set_cargo_vars method of Cargo easyblock, so it's set for every extension when CargoPythonBundle easyblock is used set $CARGO_NET_OFFLINE to true in set_cargo_vars method of Cargo easyblock (only if offline mode is used), so it's set for every extension when CargoPythonBundle easyblock is used Jun 7, 2025
Copy link
Contributor

@Micket Micket left a comment

Choose a reason for hiding this comment

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

lgtm

@boegel
Copy link
Member Author

boegel commented Jun 7, 2025

Test report by @boegel

Overview of tested easyconfigs (in order)

  • SUCCESS skani-0.2.2-GCCcore-12.3.0.eb
  • SUCCESS maturin-1.1.0-GCCcore-12.3.0.eb
  • SUCCESS tiktoken-0.9.0-GCCcore-13.3.0.eb
  • SUCCESS Safetensors-0.4.3-gfbf-2023a.eb
  • SUCCESS tokenizers-0.19.1-GCCcore-13.2.0.eb
  • SUCCESS chopper-0.9.0-GCCcore-12.3.0.eb
  • SUCCESS JupyterLab-4.0.3-GCCcore-12.2.0.eb

Build succeeded for 7 out of 7 (7 easyconfigs in total)
node3603.doduo.os - Linux RHEL 9.4, x86_64, AMD EPYC 7552 48-Core Processor (zen2), Python 3.9.18
See https://gist.github.com/boegel/3a8efea755c91d03a514767de09c7def for a full test report.

@Micket Micket merged commit 0f41576 into easybuilders:develop Jun 7, 2025
17 checks passed
@boegel boegel deleted the 20250607100814_new_pr_cargo branch June 7, 2025 13:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants