Expose Environment Variables during Build

[cwshugg]

Hi - this PR implements a way to expose environment variables during the build command that's executed during exec_build(). This is done with the aim to give dependencies the ability to detect a cargo-fuzz build versus a non-cargo-fuzz build.

I implemented this feature while doing work to get cargo-fuzz working on Windows to build and fuzz a DLL. Certain dependencies may need to use different built settings when being built for production versus for fuzzing. Exposing these environment variables at build-time allow build.rs scripts to detect cargo-fuzz builds.

Currently just two environment variables are exposed, but more could of course be added:

• CARGO_FUZZ - A flag that's set to 1 when cargo-fuzz is performing a build. Useful for detecting cargo-fuzz without wanting any extra information.
• CARGO_FUZZ_MANIFEST - Exposes the path to the cargo-fuzz Cargo.toml manifest that was used when cargo fuzz [build/run/etc.] was invoked. Useful in the event a user needs to determine which fuzzing project ran the command.

Use-Case Example

As I mentioned above, a user may need to modify the build flags in a fuzzing dependency, but only when the dependency is being built for fuzzing. A brief example of how this would work:

// build.rs

use std::env;

fn main()
{
    // ...
    if let Ok(_) = env::var("CARGO_FUZZ")
    {
        // we know this dependency is being built for fuzzing!
        println!("cargo:rustc-link-arg=CUSTOM_FUZZING_LINKER_ARG");
        println!("cargo:rustc-flags=CUSTOM_FUZZING_RUSTFLAGS");
        println!("cargo:rustc-cfg=my_experimental_feature=ENABLE");
    }
    // ...
}

[cwshugg]

Hi @fitzgen - just bumping this to you. Would love to get your feedback on the change and potentially get it merged.

[fitzgen]

Thanks! Do you mind splitting these features out into different PRs so we can discuss them independently?

[cwshugg]

@fitzgen - Sure thing, thanks for taking a look. Here is the second PR.

[fitzgen]

Thanks! A couple things to address before we merge this.

[cwshugg]

@fitzgen - Thanks for reviewing these PRs! I've addressed your comments; let me know what you thin.

[fitzgen]

Thanks! Just fyi I’m on vacation and won’t be able to look at this again until the middle of next week.