Tim Crawford

`rustc-link-arg` was stabilized (rust-lang/cargo#9557). However, they decided that transitive link arguments are bad and you must always explicitly opt-in for custom link arguments (rust-lang/cargo#9554). So every binary must still copy...

> The just-run-rustup-show trick is great iff you already have rustup installed on your ci machine. Any tips for that? If you're using shared runners, it's already there and you...

Could be done with #63.

> After reading some PRs, I can see more and more sudo commands getting introduced into the code. Such as which ones? I don't see any current PRs adding sudo....

> you removed "and making it either configurable with a variable", which is pretty important. I am simply dismissing the option of being able to remove the use of sudo...

> If you prefer it removed, I can do so. Yes. SPDX should replace the boilerplate license block with easy to understand identifiers. The "This file is part of..." lines...

Testing on a galp3-c: - When configured for release, system does not boot (doesn't reach first power event; LEDs are off) - When parport debugging is enabled, system boots, but...

> Is there anything special that needs to configure for release, or is it the default? It's the default. That is, there's no modifications to `board.mk`. As a "debug" build,...

`system76_ectool config` when using `SCAR1` works: ``` Default Camera State: When set, the camera is enabled on system reset Min: 1 Max: 0 Current Value: 1 Fn Lock: When enabled,...

Suppose the format changes to handle a new config. How will the EC and tool detect and handle this? I've previously used a simple incrementing version number to detect such...