Cisco-Talos / clamav

ClamAV - Documentation is here: https://docs.clamav.net
https://www.clamav.net/
GNU General Public License v2.0
4.43k stars 706 forks source link

cmake/FindRust.cmake: inherit RUSTFLAGS from the environment #1301

Closed liushuyu closed 3 months ago

liushuyu commented 4 months ago

This pull request changes the FindRust.cmake script so the RUSTFLAGS set from the environment won't be overridden.

This is needed because some downstream distributors (e.g. Linux distros) will need to append several Rust flags to get reproducible builds and/or make the project build on certain architectures.

micahsnyder commented 4 months ago

Thanks @liushuyu this makes sense to me. Will test it soon as our internal CI system is working right. (everything seems to be broken right now, very frustrating!)