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.

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

Updated workspace manifests to use relative paths to sibling packages #14

Merged
merged 2 commits into from
Nov 6, 2022

Conversation

metasim
Copy link
Contributor

@metasim metasim commented Nov 6, 2022

Ensures consistent inter-package cross referencing when imported as a source dependency.

Note: makes use of workspace features released in Rust 1.64.

@metasim metasim force-pushed the fix/relative-workspace branch from 9b2bc4b to e7e3bbf Compare November 6, 2022 19:56
Copy link
Member

@b4l b4l left a comment

Choose a reason for hiding this comment

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

Relativ path are already set here as far as I understand it:

[patch.crates-io]

@metasim
Copy link
Contributor Author

metasim commented Nov 6, 2022

Relativ path are already set here as far as I understand it

They don't work for cases where you want to use a package outside of crates.io, e.g. as a local path or git URL. Here are a couple examples of other projects using this approach:

@metasim
Copy link
Contributor Author

metasim commented Nov 6, 2022

CI depends on #15 being fixed.

for consistent referencing when imported as a source dependency.

Also updated project repository link.
@metasim metasim force-pushed the fix/relative-workspace branch from e7e3bbf to a34a812 Compare November 6, 2022 21:45
@b4l
Copy link
Member

b4l commented Nov 6, 2022

@metasim so should we remove the patch.crates-io section then?

@metasim
Copy link
Contributor Author

metasim commented Nov 6, 2022

@metasim so should we remove the patch.crates-io section then?

I'll do that, but TBH, I've never published a multi-package crate, so we might need to roll it back...

@metasim metasim requested a review from b4l November 6, 2022 22:21
Copy link
Member

@b4l b4l left a comment

Choose a reason for hiding this comment

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

Thanks!

@metasim metasim merged commit 7e91804 into georust:main Nov 6, 2022
@metasim metasim deleted the fix/relative-workspace branch November 6, 2022 22:55
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