Note: There are a few cases of duplicates in user autocompletion which are being worked on.

Fix for 638056 breaks MeeGo armv7 build

RESOLVED FIXED in mozilla6

Status

()

Core
Build Config
--
major
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: Lokesh Goel, Assigned: glandium)

Tracking

Trunk
mozilla6
ARM
MeeGo
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(3 attachments, 3 obsolete attachments)

(Reporter)

Description

6 years ago
Created attachment 528157 [details]
Full Error Log

PACKAGE VERSION:
mozilla-central 68416:59771590e920

ENVIRONMENT:
MeeGo(rpmbuild) armv7el

Looks like the patch for bug#638056 breaks the armel build for MeeGo. See the errors below:

/usr/bin/python2.6 /home/abuild/rpmbuild/BUILD/mozilla-central/js/src/config/pythonpath.py -I../config /home/abuild/rpmbuild/BUILD/mozilla-central/js/src/config/expandlibs_exec.py --uselist --  c++ -o js  -frtti -fno-exceptions -Wall -Wpointer-arith -Woverloaded-virtual -Wsynth -Wno-ctor-dtor-privacy -Wno-non-virtual-dtor -Wno-invalid-offsetof -Wno-variadic-macros -Werror=return-type -O2 -g -pipe -Wall -Wp,-D_FORTIFY_SOURCE=2 -fstack-protector --param=ssp-buffer-size=4 -Wformat -Wformat-security -fmessage-length=0 -march=armv7-a -mtune=cortex-a8 -mlittle-endian -mfpu=vfpv3 -mfloat-abi=softfp -D__SOFTFP__ -fno-strict-aliasing -pthread -march=armv5te -mthumb-interwork -msoft-float -pipe  -DNDEBUG -DTRIMMED -g -g -O2 -fno-tree-ter -fno-ipa-reference -fno-tree-switch-conversion  js.o jsworkers.o   -lpthread -Wl,--no-keep-memory,-rpath-link,/home/abuild/rpmbuild/BUILD/release/dist/bin/:/usr/lib:/lib   -Wl,-rpath-link,/bin -Wl,-rpath-link,/home/abuild/rpmbuild/BUILD/release/dist/lib  -L../../../dist/bin -L../../../dist/lib -L/usr/lib -lplds4 -lplc4 -lnspr4 -lpthread -ldl ../editline/libeditline.a ../libjs_static.a -ldl     
../libjs_static.a(jsapi.o): In function `~ExecutablePool':
/home/abuild/rpmbuild/BUILD/mozilla-central/js/src/assembler/jit/ExecutableAllocator.h:128: undefined reference to `JSC::ExecutablePool::systemRelease(JSC::ExecutablePool::Allocation const&)'

Also find attached the full rawlog of the build and the mozconfig used.
(Reporter)

Comment 1

6 years ago
Created attachment 528159 [details]
Mozconfig used
(Reporter)

Comment 2

6 years ago
Reversing 68416:59771590e920 fixes the issue.
(Reporter)

Updated

6 years ago
Severity: normal → major
Blocks: 638056
(Assignee)

Comment 3

6 years ago
What is TARGET_CPU on meego ? armel ?
(Reporter)

Comment 4

6 years ago
(In reply to comment #3)
> What is TARGET_CPU on meego ? armel ?

YES
(Assignee)

Comment 5

6 years ago
Created attachment 528272 [details] [diff] [review]
Add armel in the opt-in for the nitro assembler
Assignee: nobody → mh+mozilla
(Assignee)

Updated

6 years ago
Attachment #528272 - Flags: review?(cdleary)
(Reporter)

Comment 6

6 years ago
(In reply to comment #5)
> Created attachment 528272 [details] [diff] [review]
> Add armel in the opt-in for the nitro assembler
I am sorry, but this fix doesn't solve the problem. I tried this and it doesn't fix the issue.

We build for two arm architectures in MeeGo armv7el(softfp) and armv8el(hardfp), the TARGET_CPU for MeeGo armv7el target seems to be "armv7l".
(Reporter)

Comment 7

6 years ago
"armv7l" is the TARGET_CPU for both armv7el and armv8el. armv8el is actually armv7el + hardfp.
(Reporter)

Comment 8

6 years ago
Created attachment 528429 [details] [diff] [review]
This patch seems to solve the problem
(Reporter)

Comment 9

6 years ago
Created attachment 528435 [details] [diff] [review]
Probably this is a better way.
(Assignee)

Comment 10

6 years ago
Created attachment 528437 [details] [diff] [review]
Use arm% in the opt-in for the nitro assembler

Even better
Attachment #528272 - Attachment is obsolete: true
Attachment #528429 - Attachment is obsolete: true
Attachment #528435 - Attachment is obsolete: true
Attachment #528272 - Flags: review?(cdleary)
Attachment #528437 - Flags: review?(cdleary)
Attachment #528437 - Flags: review?(cdleary) → review+
(Assignee)

Comment 11

6 years ago
http://hg.mozilla.org/mozilla-central/rev/c833fb1623ca
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla6
You need to log in before you can comment on or make changes to this bug.