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.

Sign up for GitHub

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

Jump to bottom

Rebuild and rerun build scripts when RUSTC_WRAPPER has changed #10973

Closed
wants to merge 3 commits into from
Closed

Rebuild and rerun build scripts when RUSTC_WRAPPER has changed #10973

wants to merge 3 commits into from

Commits on Aug 11, 2022

  1. Add tests of build script rerun on RUSTC_WRAPPER change 

    Currently fails on the "Reruns build script because RUSTC_WRAPPER
changes the fingerprint" step.

    ---- freshness::rerun_build_script_if_add_rustc_wrapper stdout ----
    running `/git/cargo/target/debug/cargo check`
    running `/git/cargo/target/debug/cargo check`
    running `/git/cargo/target/debug/cargo build`
    running `/git/cargo/target/debug/cargo check`
    thread 'freshness::rerun_build_script_if_add_rustc_wrapper' panicked at 'assertion failed: file_created_by_build_script.exists()', tests/testsuite/freshness.rs:888:5
    note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
    @dtolnay
    dtolnay committed Aug 11, 2022
    Configuration menu
    Copy the full SHA
    6b55e3d View commit details
    Browse the repository at this point in the history

  2. Include RUSTC_WRAPPER in fingerprint for build script rerun

    @dtolnay
    dtolnay committed Aug 11, 2022
    Configuration menu
    Copy the full SHA
    db83dfb View commit details
    Browse the repository at this point in the history

  3. Check that build script is rebuilt with wrapper, not just rerun

    @dtolnay
    dtolnay committed Aug 11, 2022
    Configuration menu
    Copy the full SHA
    7b53cf8 View commit details
    Browse the repository at this point in the history