Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
Bug 1523912
Summary: | firefox 57.0.1 doesn't build on armv7 | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Kevin Fenzi <kevin> |
Component: | rust | Assignee: | Josh Stone <jistone> |
Status: | CLOSED CURRENTRELEASE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | unspecified | Docs Contact: | |
Priority: | unspecified | ||
Version: | 28 | CC: | alexl, awilliam, gecko-bugs-nobody, herrold, ignatenko, jhorak, jistone, john.j5live, kengert, mattdm, pbrobinson, pjasicek, pwhalen, rhughes, rstrode, rust-sig, samtygier, sandmann, stransky, TicoTimo |
Target Milestone: | --- | ||
Target Release: | --- | ||
Hardware: | arm | ||
OS: | Unspecified | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | If docs needed, set a value | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2018-04-09 12:05:46 UTC | Type: | Bug |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: | |||
Bug Depends On: | |||
Bug Blocks: | 245418 |
Description
Kevin Fenzi
2017-12-09 01:04:28 UTC
This looks to be a compiler issue, moving to rust. We disabled Firefox 57 builds on all Fedoras due to this issue. Martin, is there an upstream Firefox (or Rust) bug for this? I'm not aware of any upstream bugreport about it. It sounds similar to rust#45854, where I had memory issues on armv7 even building rustc itself. https://github.com/rust-lang/rust/issues/45854 My workaround so far is to disable rust debuginfo on arm. It looks like firefox/config/rules.mk supports RUSTFLAGS, so you could add -Cdebuginfo=0 to try this yourself. Rust 1.23 also claims some modest reduction of memory use in rustc, 5-10%, so I guess we can see if that helps when I rebase in the next few days... I added the -Cdebuginfo=0 workaround to firefox-57.0.4-2 builds. It doesn't look like that took effect. In the build log, I see your + RUSTFLAGS=-Cdebuginfo=0 But you also have to export shell variables. Later in the build you can see: make[5]: Entering directory '/builddir/build/BUILD/firefox-57.0.4/objdir/toolkit/library/rust' force-cargo-library-build env RUSTFLAGS=' -C debuginfo=2 ' [...] I tried a rawhide build with Rust 1.23, enabling arm but no RUSTFLAGS change. First, I found that `#[deny(warnings)]` in servo/components/style/lib.rs caused an unused_imports warning on AsciiExt to be an error. This change is mentioned in the release notes, but it only becomes a breaking change when you deny warnings. https://blog.rust-lang.org/2018/01/04/Rust-1.23.html There's an upstream fix that masks that warning on each import: https://github.com/servo/servo/commit/e7a654dd13f589e127193267bcb576ffd661c11d But this doesn't apply cleanly on Firefox 57. I just allowed that warning for the whole crate to get it building, adding this in firefox.spec %prep: sed -i.allow-warnings -e '/#!\[deny(warnings)\]/a #![allow(unused_imports)]' \ servo/components/style/lib.rs This scratch build succeeded! https://koji.fedoraproject.org/koji/taskinfo?taskID=24088682 I will be building Rust 1.23 for F27 and F26 today, so hopefully this solves it. Great! Can you provide me a patch to firefox.spec for that? I don't think you need that patch anymore with Firefox 58, as it includes the upstream fix for the AsciiExt warnings. I see in koji you also just did builds that passed with exported RUSTFLAGS to block arm debuginfo, but this was also still with Rust 1.22.1. Rust 1.23 just landed on F27 and F26 this morning, and has been on rawhide since January 8. If you could try without that RUSTFLAGS workaround, I think it's in better shape now. diff --git a/firefox.spec b/firefox.spec index f07e71cf5888..2f3299921bb6 100644 --- a/firefox.spec +++ b/firefox.spec @@ -522,9 +522,6 @@ export MOZ_DEBUG_FLAGS=" " %ifarch s390 %{arm} ppc aarch64 MOZ_LINK_FLAGS="-Wl,--no-keep-memory -Wl,--reduce-memory-overheads" %endif -%ifarch %{arm} -export RUSTFLAGS="-Cdebuginfo=0" -%endif export CFLAGS=$MOZ_OPT_FLAGS export CXXFLAGS=$MOZ_OPT_FLAGS export LDFLAGS=$MOZ_LINK_FLAGS This bug appears to have been reported against 'rawhide' during the Fedora 28 development cycle. Changing version to '28'. Seems fine with firefox 59 |