Join GitHub today
GitHub is home to over 31 million developers working together to host and review code, manage projects, and build software together.
Sign upExract manifest handling into its own crate #1112
Comments
brson
added
the
help wanted
label
May 12, 2017
Diggsey
added this to Build
in Issue Categorisation
May 12, 2017
mikhail-m1
referenced this issue
May 16, 2017
Open
Teach rustup/rustbuild about optional components #1006
This comment has been minimized.
This comment has been minimized.
|
I can help on this. Any idea how to share code between rust and rustup.rs? |
This comment has been minimized.
This comment has been minimized.
|
@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. |
brson commentedMay 12, 2017
The
build-manifesttool insiderust-lang/rustis responsible for creating manifests, and themanifestmodule inrustup-distis 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.rsinto its own crate, then plug that crate back into rustup, and intobuild-manifest. Modifybuild-manifestto run thevalidatemethod before writing the manifest. Extract manifest-specific tests as well. Look for other code to share between them.