Closed
Bug 799806
Opened 12 years ago
Closed 12 years ago
libmar compile failures for Firefox 17.0b1 repacks (all platforms)
Categories
(Release Engineering :: Release Automation: Other, defect, P2)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: nthomas, Assigned: nthomas)
References
Details
Attachments
(3 files)
(deleted),
patch
|
jhopkins
:
review+
jhopkins
:
checked-in+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
bhearsum
:
review+
nthomas
:
checked-in+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
Details | Diff | Splinter Review |
/usr/bin/gcc-4.2 -o host_mar_create.o -c -DXP_UNIX -DXP_MACOSX -DNO_X11 -O3 -I/builds/slave/rel-m-beta-osx64-rpk-6/mozilla-beta/modules/libmar/src -I. -I../../../dist/include -I/builds/slave/rel-m-beta-osx64-rpk-6/mozilla-beta/obj-l10n/dist/include/nspr -I/builds/slave/rel-m-beta-osx64-rpk-6/mozilla-beta/obj-l10n/dist/include/nss -I/builds/slave/rel-m-beta-osx64-rpk-6/mozilla-beta/obj-l10n/dist/include/nspr /builds/slave/rel-m-beta-osx64-rpk-6/mozilla-beta/modules/libmar/src/mar_create.c
In file included from /builds/slave/rel-m-beta-osx64-rpk-6/mozilla-beta/modules/libmar/src/mar_create.c:12:
/builds/slave/rel-m-beta-osx64-rpk-6/mozilla-beta/modules/libmar/src/mar_private.h:12:37: error: mozilla/StandardInteger.h: No such file or directory
In file included from /builds/slave/rel-m-beta-osx64-rpk-6/mozilla-beta/modules/libmar/src/mar_create.c:12:
/builds/slave/rel-m-beta-osx64-rpk-6/mozilla-beta/modules/libmar/src/mar_private.h:16: error: 'uint32_t' undeclared here (not in a function)
/builds/slave/rel-m-beta-osx64-rpk-6/mozilla-beta/modules/libmar/src/mar_private.h:17: error: 'uint64_t' undeclared here (not in a function)
/builds/slave/rel-m-beta-osx64-rpk-6/mozilla-beta/modules/libmar/src/mar_private.h:43: error: 'uint32_t' undeclared here (not in a function)
make[2]: *** [host_mar_create.o] Error 1
make[1]: *** [libs] Error 2
make: *** [default] Error 2
The changes from bug 785066 were never ported from the nightly situation to the release automation.
Assignee | ||
Comment 1•12 years ago
|
||
Windows fail looks like
e:\builds\moz2_slave\rel-m-beta-w32-rpk-5\mozilla-beta\modules\libmar\src\mar_private.h(12) : fatal error C1083: Cannot open include file: 'mozilla/StandardInteger.h': No such file or directory
Summary: libmar compile failures for Firefox 17.0b1 linux/linux64/mac → libmar compile failures for Firefox 17.0b1 repacks (all platforms)
Assignee | ||
Comment 2•12 years ago
|
||
This is my attempt to port the changes from bug 785066 over. To get StandardInteger.h we have to build in mfbt/, which is part of tier_base.
Attachment #669835 -
Flags: review?(jhopkins)
Assignee | ||
Comment 3•12 years ago
|
||
Comment on attachment 669835 [details] [diff] [review]
[tools] build tier_base too
I ran this patch on a mac slave (with appropriate env vars set) and it successfully compiled libmar and created mar files for the son locale. I blocked it uploading anything to leave a clean slate on ftp.
Updated•12 years ago
|
Attachment #669835 -
Flags: review?(jhopkins) → review+
Comment 4•12 years ago
|
||
Comment on attachment 669835 [details] [diff] [review]
[tools] build tier_base too
Landed in https://hg.mozilla.org/build/tools/rev/798cbd07c55b
Attachment #669835 -
Flags: checked-in+
Comment 5•12 years ago
|
||
Reconfig'd on buildbot-master13
Comment 6•12 years ago
|
||
mozilla-beta repacks are passing with this patch.
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 7•12 years ago
|
||
This broke Thunderbird repacks, which have a non-empty mozilla_dir and start trying to do:
command: START
command: make
command: cwd: comm-release/obj-l10n/mozilla/tier_base
command: env: {'MOZ_MAKE_COMPLETE_MAR': '1', 'UPLOAD_USER': 'tbirdbld', ...
command: output:
command: END (0.01s elapsed)
which ends up at
OSError: [Errno 2] No such file or directory: 'comm-release/obj-l10n/mozilla/tier_base'
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Assignee | ||
Comment 8•12 years ago
|
||
Attachment #670202 -
Flags: review?(bhearsum)
Comment 9•12 years ago
|
||
Comment on attachment 670202 [details] [diff] [review]
[tools] make it work for Thunderbird too
Review of attachment 670202 [details] [diff] [review]:
-----------------------------------------------------------------
This is a horrible, horrible hack. r+ because it appears it will work, but can you file a bug or otherwise track finding a better solution for this?
Attachment #670202 -
Flags: review?(bhearsum) → review+
Assignee | ||
Comment 10•12 years ago
|
||
Comment on attachment 670202 [details] [diff] [review]
[tools] make it work for Thunderbird too
http://hg.mozilla.org/build/tools/rev/652471b93bda and retagged THUNDERBIRD_16_0_1_{RELEASE,BUILD1}.
Attachment #670202 -
Flags: checked-in+
Assignee | ||
Updated•12 years ago
|
Status: REOPENED → RESOLVED
Closed: 12 years ago → 12 years ago
Resolution: --- → FIXED
Comment 11•12 years ago
|
||
I'm verifying this before requesting for a review.
Updated•11 years ago
|
Product: mozilla.org → Release Engineering
You need to log in
before you can comment on or make changes to this bug.
Description
•