Closed
Bug 1151370
Opened 9 years ago
Closed 9 years ago
mozinfo.py fails 'bits' undefined on powerpc64
Categories
(Firefox Build System :: General, defect)
Tracking
(firefox40 fixed)
RESOLVED
FIXED
mozilla40
Tracking | Status | |
---|---|---|
firefox40 | --- | fixed |
People
(Reporter: stevensn, Assigned: ahal)
References
Details
Attachments
(3 files, 1 obsolete file)
1.31 KB,
patch
|
Details | Diff | Splinter Review | |
671 bytes,
patch
|
khuey
:
review+
|
Details | Diff | Splinter Review |
39 bytes,
text/x-review-board-request
|
ahal
:
review+
|
Details |
mozinfo.py is failing on ppc64 because 'bits' is not defined in the dictionary. Bug 1142050 populates bits for ppc and ppc64 but not 'powerpc64' or 'powerpc' It also is missing sparc and possibly other platforms.
Reporter | ||
Comment 1•9 years ago
|
||
Reporter | ||
Comment 2•9 years ago
|
||
Comment on attachment 8588498 [details] [diff] [review] add more platforms to mozinfo.py Landry, Have I missed any platforms that you are aware people are building on? My thinking is that this should default to 32 bits if the platform isn't in the list instead of failing.
Flags: needinfo?(landry)
Comment 3•9 years ago
|
||
On OpenBSD/ppc, TARGET_CPU is 'powerpc' too. Maybe we should also add various mips* targets... glandium usually targets more (alpha?s390?) with debian :)
Flags: needinfo?(landry) → needinfo?(mh+mozilla)
Comment 4•9 years ago
|
||
I'm not overly concerned with covering 100% of CPUs here. Fix what's broken for you, and if there are others that you know are being used that's fine too, but I'm fine with this only supporting architectures that people are actively using.
Reporter | ||
Comment 6•9 years ago
|
||
Andrew, doesn't https://hg.mozilla.org/mozilla-central/rev/0b0d3ba11869 add the guess_platform function?
Reporter | ||
Comment 7•9 years ago
|
||
branislav, which MIPS cpu targets are you supporting (and are they 64 or 32 bits)
Flags: needinfo?(branislav.rankov)
Assignee | ||
Comment 8•9 years ago
|
||
(In reply to Steve Singer (:stevensn) from comment #6) > Andrew, doesn't https://hg.mozilla.org/mozilla-central/rev/0b0d3ba11869 add > the guess_platform function? Oh I see what you mean. Bug 1142050 uses 'bits' which causes this failure, but it doesn't populate anything under the bits section. Sorry, the wording in comment 0 confused me a bit.
Blocks: 1142050
Comment 9•9 years ago
|
||
platform_guess is really only useful for platforms built on Mozilla automation. I'd rather fix those functions to return an empty string on non "supported" platforms, than adding partial support for some classes of hardware.
Updated•9 years ago
|
Flags: needinfo?(mh+mozilla)
Comment 10•9 years ago
|
||
For MIPS, you should add: - mips and mipsel as 32 bit - mips64 and mips64el as 64 bit.
Flags: needinfo?(branislav.rankov)
Assignee | ||
Comment 11•9 years ago
|
||
/r/6693 - Bug 1151370 - Only guess platform/buildtype in mozinfo.py if processor is known, r=ted Pull down this commit: hg pull -r 3cbb0cb504bf17d91936e2b995dac6c8d9f8badc https://reviewboard-hg.mozilla.org/gecko/
Attachment #8589079 -
Flags: review?(ted)
Assignee | ||
Comment 12•9 years ago
|
||
Apologies for the regression, the above should be all you need. I agree with ted and glandium that there's not much value in adding additional processors unless they're actually needed somewhere. The other patch just wrongfully assumed that 'bits' was always set.
Comment 13•9 years ago
|
||
Attachment #8591322 -
Flags: review?(khuey)
Comment 14•9 years ago
|
||
Was something commited in the end ? I see ted said 'ship it' on the reviewboard link for attachment 8589079 [details] but nothing landed, as a build on ppc still fails as of now:
1:00.30 File "/home/landry/src/mozilla-central/python/mozbuild/mozbuild/mozinfo.py", line 101, in guess_platform
1:00.30 elif d['bits'] == 64:
1:00.30 KeyError: 'bits'
Assignee | ||
Comment 15•9 years ago
|
||
(In reply to Landry Breuil (:gaston) from comment #14) > Was something commited in the end ? I see ted said 'ship it' on the > reviewboard link for attachment 8589079 [details] but nothing landed Oh weird, usually clicking Ship It! is supposed to flip the bugzilla attachment to an r+ but that didn't happen in this case for some reason. I'll land it now.
Assignee | ||
Comment 16•9 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/f1a49ef21a1c
Assignee | ||
Comment 17•9 years ago
|
||
Comment on attachment 8589079 [details] MozReview Request: bz://1151370/ahal Mozreview didn't flip the r+ for some reason. Landed: https://treeherder.mozilla.org/#/jobs?repo=mozilla-inbound&revision=f1a49ef21a1c
Attachment #8589079 -
Flags: review?(ted) → review+
https://hg.mozilla.org/mozilla-central/rev/f1a49ef21a1c
Assignee: nobody → ahalberstadt
Status: NEW → RESOLVED
Closed: 9 years ago
status-firefox40:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla40
Attachment #8591322 -
Flags: review?(khuey) → review+
Updated•9 years ago
|
Keywords: checkin-needed
Comment 19•9 years ago
|
||
I'm not 100% sure but with what was commited in https://hg.mozilla.org/mozilla-central/rev/f1a49ef21a1c attachment #8591322 [details] [diff] [review] is not needed now.
Updated•9 years ago
|
Keywords: checkin-needed
Assignee | ||
Comment 20•9 years ago
|
||
Attachment #8589079 -
Attachment is obsolete: true
Attachment #8619975 -
Flags: review+
Assignee | ||
Comment 21•9 years ago
|
||
Updated•6 years ago
|
Product: Core → Firefox Build System
You need to log in
before you can comment on or make changes to this bug.
Description
•