rust-lang / jobserver-rs

Apache License 2.0
69 stars 39 forks source link

fix: last `--jobserver-auth` wins #67

Closed weihanglo closed 6 months ago

weihanglo commented 7 months ago

From the GNU make manual^1:

Be aware that the MAKEFLAGS variable may contain multiple instances of the --jobserver-auth= option. Only the last instance is relevant.

Hence this commit makes it so.

With this commit, --jobserver-auth also takes precedence over --jobserver-fds, even when --jobserver-fds is the last instance of these flags. This is made intentionally since --jobserver-fds was an undocumented internal-only flag before --jobserver-auth made public, according to this announcement^2.

fixes #66