Closed Bug 1378440 Opened 3 years ago Closed 3 years ago

add `mach cargo check` command


(Firefox Build System :: General, enhancement)

Not set


(firefox56 fixed)

Tracking Status
firefox56 --- fixed


(Reporter: froydnj, Assigned: froydnj)




(2 files)

No reason Gecko developers shouldn't have this.
All the machinery we set up for `cargo build` is equally useful for
other commands, such as `cargo check`.  Let's refactor things so that
it's easy to reuse said machinery for `cargo check`.
Attachment #8883632 - Flags: review?(giles)
Cargo recently introduced the `cargo check` command for shortening the
edit-compile cycle when working on large programs.  Since we don't
really support invoking `cargo` directly, let's wire up this command to
`mach`.  Gecko developers can then `mach cargo check` to ensure their
changes typecheck.

I'm not completely happy with the changes, but they were the easiest
way I could think of doing things, since I didn't want to store information on
what *kind* of crate we had at the mach level.  I just wanted to say "check",
and let the Makefile sort out what needs doing.  Let me know what you think.
Attachment #8883633 - Flags: review?(giles)
Comment on attachment 8883633 [details] [diff] [review]
part 2 - add `mach cargo check`

Review of attachment 8883633 [details] [diff] [review]:

Looks good. Thanks for doing this!
Attachment #8883633 - Flags: review?(giles) → review+
Attachment #8883632 - Flags: review?(giles) → review+
Pushed by
part 1 - make Cargo invocation macro more generic; r=rillian
part 2 - add `mach cargo check`; r=rillian
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla56
Duplicate of this bug: 1354401
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.