Closed Bug 1087089 Opened 5 years ago Closed 5 years ago

OSX64 Repacks failing during 2.31b1 build.

Categories

(SeaMonkey :: Release Engineering, defect)

x86_64
macOS
defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: ewong, Assigned: Callek)

References

(Blocks 1 open bug)

Details

Attachments

(1 file, 1 obsolete file)

Getting a traceback during the MacOSX64 Repacks:

Packaging seamonkey.xpi...
if test -d "/builds/slave/rel-c-beta-osx64-rpk/build/comm-beta/obj-l10n/dist/l10n-stage/SeaMonkey/SeaMonkey.app/Contents/MacOS/seamonkey"; then cd /builds/slave/rel-c-beta-osx64-rpk/build/comm-beta/obj-l10n/dist/l10n-stage/SeaMonkey/SeaMonkey.app/Contents/MacOS/seamonkey/; /usr/bin/zip -r9mX ../seamonkey.xpi * -x \*/.mkdir.done; cd ..; rm -rf /builds/slave/rel-c-beta-osx64-rpk/build/comm-beta/obj-l10n/dist/l10n-stage/SeaMonkey/SeaMonkey.app/Contents/MacOS/seamonkey; fi
Packaging seamonkey-bin.xpi...
if test -d "/builds/slave/rel-c-beta-osx64-rpk/build/comm-beta/obj-l10n/dist/l10n-stage/SeaMonkey/SeaMonkey.app/Contents/MacOS/seamonkey-bin"; then cd /builds/slave/rel-c-beta-osx64-rpk/build/comm-beta/obj-l10n/dist/l10n-stage/SeaMonkey/SeaMonkey.app/Contents/MacOS/seamonkey-bin/; /usr/bin/zip -r9mX ../seamonkey-bin.xpi * -x \*/.mkdir.done; cd ..; rm -rf /builds/slave/rel-c-beta-osx64-rpk/build/comm-beta/obj-l10n/dist/l10n-stage/SeaMonkey/SeaMonkey.app/Contents/MacOS/seamonkey-bin; fi
Packaging updater.app.xpi...
if test -d "/builds/slave/rel-c-beta-osx64-rpk/build/comm-beta/obj-l10n/dist/l10n-stage/SeaMonkey/SeaMonkey.app/Contents/MacOS/updater.app"; then cd /builds/slave/rel-c-beta-osx64-rpk/build/comm-beta/obj-l10n/dist/l10n-stage/SeaMonkey/SeaMonkey.app/Contents/MacOS/updater.app/; /usr/bin/zip -r9mX ../updater.app.xpi * -x \*/.mkdir.done; cd ..; rm -rf /builds/slave/rel-c-beta-osx64-rpk/build/comm-beta/obj-l10n/dist/l10n-stage/SeaMonkey/SeaMonkey.app/Contents/MacOS/updater.app; fi
  adding: Contents/ (stored 0%)
  adding: Contents/Info.plist (deflated 59%)
  adding: Contents/MacOS/ (stored 0%)
  adding: Contents/MacOS/updater (deflated 59%)
  adding: Contents/PkgInfo (stored 0%)
  adding: Contents/Resources/ (stored 0%)
  adding: Contents/Resources/English.lproj/ (stored 0%)
  adding: Contents/Resources/English.lproj/InfoPlist.strings (deflated 49%)
  adding: Contents/Resources/English.lproj/MainMenu.nib/ (stored 0%)
  adding: Contents/Resources/English.lproj/MainMenu.nib/classes.nib (deflated 49%)
  adding: Contents/Resources/English.lproj/MainMenu.nib/info.nib (deflated 46%)
  adding: Contents/Resources/English.lproj/MainMenu.nib/keyedobjects.nib (deflated 30%)
  adding: Contents/Resources/updater.icns (deflated 45%)
/builds/slave/rel-c-beta-osx64-rpk/build/comm-beta/obj-l10n/_virtualenv/bin/python /builds/slave/rel-c-beta-osx64-rpk/build/comm-beta/mozilla/toolkit/mozapps/installer/l10n-repack.py /builds/slave/rel-c-beta-osx64-rpk/build/comm-beta/obj-l10n/dist/l10n-stage/SeaMonkey/SeaMonkey.app/Contents/MacOS ../../dist/xpi-stage/locale-zh-TW \
		--non-resource defaults/messenger/mailViews.dat defaults/profile/panels.rdf defaults/profile/mimeTypes.rdf defaults/profile/chrome/userChrome-example.css defaults/profile/chrome/userContent-example.css 
Traceback (most recent call last):
  File "/builds/slave/rel-c-beta-osx64-rpk/build/comm-beta/mozilla/toolkit/mozapps/installer/l10n-repack.py", line 48, in <module>
    main()
  File "/builds/slave/rel-c-beta-osx64-rpk/build/comm-beta/mozilla/toolkit/mozapps/installer/l10n-repack.py", line 44, in main
    l10n.repack(args.build, args.l10n, args.non_resource, NON_CHROME)
  File "/builds/slave/rel-c-beta-osx64-rpk/build/comm-beta/mozilla/python/mozbuild/mozpack/packager/l10n.py", line 172, in repack
    _repack(app_finder, l10n_finder, copier, formatter, non_chrome)
  File "/builds/slave/rel-c-beta-osx64-rpk/build/comm-beta/mozilla/python/mozbuild/mozpack/packager/l10n.py", line 63, in _repack
    locale = app.locales[0]
IndexError: list index out of range
make[1]: *** [repackage-zip] Error 1
make: *** [repackage-zip-zh-TW] Error 2

atm, no clue what's going on right now.
Blocks: 1072713
Is the fix from bug 1084578 worth looking at for this?
OS: Windows Vista → Mac OS X
Hardware: x86 → x86_64
Attached patch [comm-beta] fixes (obsolete) — Splinter Review
Will take the review of whomever feels most comfortable here.

This does a few things, but all of them were needed.

the pack-ext stuff was causing all directories in Resources/ to be turned into an xpi on us, the @AB@ was from another bug (https://bugzilla.mozilla.org/page.cgi?id=splinter.html&bug=1084578&attachment=8507565)

And the STAGEDIST stuff was defining things incorrectly while the var is properly defined in the toolkit makefile we include here.
Assignee: nobody → bugspam.Callek
Status: NEW → ASSIGNED
Attachment #8509244 - Flags: review?(mh+mozilla)
Attachment #8509244 - Flags: review?(iann_bugzilla)
Attachment #8509244 - Flags: review?(ewong)
whops forgot to qref
Attachment #8509244 - Attachment is obsolete: true
Attachment #8509244 - Flags: review?(mh+mozilla)
Attachment #8509244 - Flags: review?(iann_bugzilla)
Attachment #8509244 - Flags: review?(ewong)
Attachment #8509247 - Flags: review?(mh+mozilla)
Attachment #8509247 - Flags: review?(iann_bugzilla)
Attachment #8509247 - Flags: review?(ewong)
Attachment #8509247 - Flags: review?(mh+mozilla) → review+
Blocks: SM2.31b1
Comment on attachment 8509247 [details] [diff] [review]
[comm-beta] fixes

Review of attachment 8509247 [details] [diff] [review]:
-----------------------------------------------------------------

Seems to work, but since glandium already did the review, I think both IanN's and mine review aren't needed (?). I'm at least removing mine.
Attachment #8509247 - Flags: review?(ewong)
Attachment #8509247 - Flags: review?(iann_bugzilla) → review+
this is fixed.
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.