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

Exract manifest handling into its own crate #1112

Open
brson opened this issue May 12, 2017 · 2 comments
Open

Exract manifest handling into its own crate #1112

brson opened this issue May 12, 2017 · 2 comments

Comments

@brson
Copy link
Contributor

brson commented May 12, 2017

The build-manifest tool inside rust-lang/rust is responsible for creating manifests, and the manifest module in rustup-dist is responsible for reading and validating manifests. There have already been bugs where manifests produced by the Rust build system failed to validate. It would be preferable if the code was in a single crate and shared between the two projects.

Extract manifest.rs into its own crate, then plug that crate back into rustup, and into build-manifest. Modify build-manifest to run the validate method before writing the manifest. Extract manifest-specific tests as well. Look for other code to share between them.

@pickfire
Copy link
Contributor

pickfire commented Aug 9, 2018

I can help on this. Any idea how to share code between rust and rustup.rs?

@brson
Copy link
Contributor Author

brson commented Aug 11, 2018

@pickfire the op seems to describe a good starting point. Since this is old, the first thing to do is probably acquaint oneself with the files mentioned in the op (I don't remember the details).

Imagine creating a single official command line tool and library for all manifest manipulation.

At this point though there may be 3rd party crates that handle manifests better than either rustup or rustboot, and one might consider whether that code could be pulled in to both our tools.

I'm not around to help on this anymore, but I hope you find someone that can guide you to improve rustup. It probably needs love.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants