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

Basic functionality (and initial repro crate) #5

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

tarcieri
Copy link
Member

Adds extremely basic functionality for invoking cargo build --locked as cargo repro build.

Though this may seem fairly pointless in and of itself, the goal of a followup commit would be to collect environmental information during this step (OS/release, rustc/cargo version, CWD, environment variables, git commit, C/C++ compiler versions if applicable) and use that during
the verification process to detect and highlight mismatches.

This commit attempts to split the CLI app (i.e. cargo-repro) from a library-level crate containing the core functionality (ala cargo-audit and the rustsec crate), in case there is interest in driving these sorts of builds from external tooling.

Adds extremely basic functionality for invoking `cargo build --locked`
as `cargo repro build`.

Though this may seem fairly pointless in and of itself, the goal of a
followup commit would be to collect environmental information during
this step (OS/release, rustc/cargo version, CWD, environment variables,
git commit, C/C++ compiler versions if applicable) and use that during
the verification process to detect and highlight mismatches.

This commit attempts to split the CLI app (i.e. `cargo-repro`) from a
library-level crate containing the core functionality (ala `cargo-audit`
and the `rustsec` crate), in case there is interest in driving these
sorts of builds from external tooling.
@tarcieri tarcieri force-pushed the basic-functionality branch from a8a4ac5 to c587c68 Compare August 12, 2019 17:07
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.

1 participant