Skip to content

support revisions in crates extraction procedure in Cargo easyblock#3405

Merged
boegel merged 1 commit intoeasybuilders:developfrom
Flamefire:cargorev
Aug 14, 2024
Merged

support revisions in crates extraction procedure in Cargo easyblock#3405
boegel merged 1 commit intoeasybuilders:developfrom
Flamefire:cargorev

Conversation

@Flamefire
Copy link
Copy Markdown
Contributor

There might be entries like

source = "git+https://github.com/charliermarsh/rs-async-zip?rev=011b24604fa7bc223daaad7712c0694bac8f0a87#011b24604fa7bc223daaad7712c0694bac8f0a87"

We do handle "?branch=sha#sha" but not "?rev=sha#sha", so extend this to "rev". To be safe match the revision.

There might be entries like
> source = "git+https://github.com/charliermarsh/rs-async-zip?rev=011b24604fa7bc223daaad7712c0694bac8f0a87#011b24604fa7bc223daaad7712c0694bac8f0a87"

We do handle "?branch=sha#sha" but not "?rev=sha#sha", so extend this to "rev".
To be safe match the revision.
Copy link
Copy Markdown
Member

@boegel boegel 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 boegel changed the title support revisions in crates extraction of cargo packages support revisions in crates extraction procedure in Cargo easyblock Aug 14, 2024
@boegel boegel added this to the release after 4.9.2 milestone Aug 14, 2024
@boegel boegel merged commit 9d6e9d9 into easybuilders:develop Aug 14, 2024
@Flamefire Flamefire deleted the cargorev branch August 14, 2024 10:46
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