Skip to content

avoid name clash in cargo crates downloaded from git repositories#3481

Closed
Flamefire wants to merge 1 commit intoeasybuilders:developfrom
Flamefire:cargo-name-clash
Closed

avoid name clash in cargo crates downloaded from git repositories#3481
Flamefire wants to merge 1 commit intoeasybuilders:developfrom
Flamefire:cargo-name-clash

Conversation

@Flamefire
Copy link
Copy Markdown
Contributor

Crates referenced by their git revision have a version but might have different revisions.
The source filename ignored the revision such that sources from different revisions were considered the same file which either fails the checksum verification or the build.
Append the revision if specified to disambiguate them.

Crates referenced by their git revision have a version but might have
different revisions.
The source filename ignored the revision such that sources from
different revisions were considered the same file which either fails the
checksum verification or the build.
Append the revision if specified to disambiguate them.
Copy link
Copy Markdown
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

@Micket
Copy link
Copy Markdown
Contributor

Micket commented Nov 7, 2024

Going with the combined PR to save myself multiple steps of testing

@Micket Micket closed this Nov 7, 2024
@Flamefire Flamefire deleted the cargo-name-clash branch November 8, 2024 09:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants