https://bugzilla.redhat.com/show_bug.cgi?id=2124697



--- Comment #67 from Stefan Hajnoczi <[email protected]> ---
Thanks, Fabio!

The rpm could invoke cargo directly, although the build won't be 100% the same
as tested upstream. I think it's preferable to use meson instead of bypassing
it.

I think the build is almost correct already because rpm invokes meson with
buildtype "plain". That tells meson not to set optimization flags and
cargo-build.sh honors that (see case "${meson_optimization}"). They are
inherited from the environment.

However, cargo-build.sh currently overrides RUSTFLAGS. Instead, it should use
RUSTFLAGS="$RUSTFLAGS ${rustflags[*]}". That way rpm can pass any options via
RUSTFLAGS while non-rpm builds can still set compiler flags in cargo-build.sh.

How does that sound?


-- 
You are receiving this mail because:
You are always notified about changes to this product and component
You are on the CC list for the bug.
https://bugzilla.redhat.com/show_bug.cgi?id=2124697
_______________________________________________
package-review mailing list -- [email protected]
To unsubscribe send an email to [email protected]
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/[email protected]
Do not reply to spam, report it: 
https://pagure.io/fedora-infrastructure/new_issue

Reply via email to