tree de55c7cf573b5feece4d04bfc8c3c46cb477c8af
parent dab5817bcf2b0f9db645bbba5cc6f9c71e9c0e38
parent f55bea7c2fd1a91106ed8458aa5d4e1054cbeed9
author bors[bot] <26634292+bors[bot]@users.noreply.github.com> 1612895182 +0000
committer GitHub <noreply@github.com> 1612895182 +0000
gpgsig -----BEGIN PGP SIGNATURE-----
 
 wsBcBAABCAAQBQJgItPOCRBK7hj4Ov3rIwAAdHIIAKv7rzjU9bYm+36+khj0AbJ3
 WehSwxrOFKw8dfzpvq13r27XQ5skDlyonYmbSFZkP0uStjiPkB2CO+19cCwVcSim
 WknbuKozn4t1oGMrTl2PNyOegre661FAdPyR70H9zbO46oIglHdPcqmle/MvICL/
 IqoHtppEyTE8A91B/oXuQ3S2APO8Alps1EdWHBNPAhPiCP3fM2ToDYQ3OFMOkXQz
 bHe4gtOEJDhXpWcfZL/HonosUftESVfegdrv/IHaUm6Y8idR1/lPHTsFCfCzAHCA
 bdYgtXwWtXYkv+J9UgrAjHvCaX7EJDPjaT6skE7B2stanOHnbUQFGAGSZ3B/ZFA=
 =+rrk
 -----END PGP SIGNATURE-----
 

Merge #271

271: Allow warnings during local development r=alistair23 a=jrvanwhy

This keep's CI's behavior where it errors if libtock-rs has warnings.

This is a second attempt at PR #270. We can't specify `RUSTFLAGS` directly because doing so overrides all `rustflags` configurations in `.cargo/config`. Those `rustflags` configs are important because they configure relocation and set the linker script. Instead, I append the `"deny warnings"` config to `.cargo/config` in CI itself.

I tested this by opening two PRs in my fork: [one to test that this PR still builds](https://github.com/jrvanwhy/libtock-rs/pull/15), and [one to test it fails if a warning is present](https://github.com/jrvanwhy/libtock-rs/pull/13).

I am not terribly happy with this solution (feels really hacky rather than kinda hacky), so any alternative implementation ideas would be appreciated!

Co-authored-by: Johnathan Van Why <jrvanwhy@google.com>