Closed Bug 1301173 Opened 8 years ago Closed 6 years ago

Changing the rustc std target requires a clobber

Categories

(Firefox Build System :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: ted, Unassigned)

References

(Depends on 1 open bug)

Details

I pushed a change that touched some rust code, and it failed on inbound:
https://treeherder.mozilla.org/logviewer.html#?job_id=35483076&repo=mozilla-inbound#L3484
12:17:06     INFO -  error: can't find crate for `std` [E0463]
12:17:06     INFO -  error: aborting due to previous error
12:17:06     INFO -  error: Could not compile `gkrust`.
12:17:06     INFO -  Caused by:
12:17:06     INFO -    Process didn't exit successfully: `/builds/slave/m-in-lx-0000000000000000000000/build/src/rustc/bin/rustc /builds/slave/m-in-lx-0000000000000000000000/build/src/toolkit/library/rust/lib.rs --crate-name gkrust --crate-type staticlib -C opt-level=2 -C panic=abort -C lto -g -C metadata=7bec5ca2d425b47b --out-dir /builds/slave/m-in-lx-0000000000000000000000/build/src/obj-firefox/toolkit/library/rust/./i686-unknown-linux-gnu/release/deps --emit=dep-info,link --target i686-unknown-linux-gnu -L dependency=/builds/slave/m-in-lx-0000000000000000000000/build/src/obj-firefox/toolkit/library/rust/./i686-unknown-linux-gnu/release/deps --extern mp4parse_capi=/builds/slave/m-in-lx-0000000000000000000000/build/src/obj-firefox/toolkit/library/rust/./i686-unknown-linux-gnu/release/deps/libmp4parse_capi.rlib` (exit code: 101)
12:17:06     INFO -  gmake[5]: *** [force-cargo-build] Error 101


This landed a few hours after rillian's rustc update. That build didn't run configure, so I wonder if we have an issue when we build a dep build that touches rust code on top of an existing objdir that used a different rustc versino?
The rustc update doesn't contain i686 target files, but has i585 target files instead. That requires configure to rerun to pickup the new target. So, in fact, this is a needs-clobber issue.
Ah, good call!
Depends on: clobber
Summary: Problems with dep builds and rustc updates? → Changing the rustc std target requires a clobber
Looks like we can close this. Even aside from clobber being appropriate above, we'd dropped i586 support, so it likely works with the toolchain package we have today.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.