Closed Bug 790827 Opened 10 years ago Closed 10 years ago

make package-tests broken on B2G

Categories

(Firefox Build System :: General, defect)

All
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
mozilla18

People

(Reporter: jgriffin, Assigned: jgriffin)

Details

Attachments

(1 file, 1 obsolete file)

make package-tests fails on B2G because it tries to package some nonexistent Android files:

make package-tests
rm -rf ./dist/test-package-stage
/home/jgriffin/armB2G/objdir-gecko/config/nsinstall -D ./dist/test-package-stage
/home/jgriffin/armB2G/objdir-gecko/config/nsinstall -D ./dist/test-package-stage/bin
/home/jgriffin/armB2G/objdir-gecko/config/nsinstall -D ./dist/test-package-stage/bin/components
/home/jgriffin/armB2G/objdir-gecko/config/nsinstall -D ./dist/test-package-stage/certs
/home/jgriffin/armB2G/objdir-gecko/config/nsinstall -D ./dist/test-package-stage/jetpack
/home/jgriffin/armB2G/objdir-gecko/config/nsinstall -D ./dist/test-package-stage/firebug
/home/jgriffin/armB2G/objdir-gecko/config/nsinstall -D ./dist/test-package-stage/peptest
/home/jgriffin/armB2G/objdir-gecko/config/nsinstall -D ./dist/test-package-stage/mozbase
/home/jgriffin/armB2G/objdir-gecko/config/nsinstall -D ./dist/test-package-stage/modules
/home/jgriffin/armB2G/objdir-gecko/config/nsinstall ./build/mobile/sutagent/android/sutAgentAndroid.apk ./dist/test-package-stage/bin
/home/jgriffin/armB2G/objdir-gecko/config/nsinstall: cannot access ./build/mobile/sutagent/android/sutAgentAndroid.apk: No such file or directory
make: *** [stage-android] Error 1
pushed to try to make sure I didn't break Android packaging: https://tbpl.mozilla.org/?tree=Try&rev=8eb07d070701
Comment on attachment 660642 [details] [diff] [review]
Don't package Android apk's for B2G

You should be able to use MOZ_WIDGET_TOOLKIT here. Please don't condition things on MOZ_BUILD_APP, that way lies madness.
Changed to MOZ_WIDGET_TOOLKIT and pushed to try
Attachment #660642 - Attachment is obsolete: true
Try run for c51d45844b52 is complete.
Detailed breakdown of the results available here:
    https://tbpl.mozilla.org/?tree=Try&rev=c51d45844b52
Results (out of 14 total builds):
    success: 14
Builds (or logs if builds failed) available at:
http://ftp.mozilla.org/pub/mozilla.org/firefox/try-builds/jgriffin@mozilla.com-c51d45844b52
Comment on attachment 660979 [details] [diff] [review]
Don't package Android apk's for B2G

The try run came back ok for both android and non-android packages.
Attachment #660979 - Flags: review?(ted.mielczarek)
Attachment #660979 - Flags: review?(ted.mielczarek) → review+
https://hg.mozilla.org/mozilla-central/rev/e83d9f2e0699
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.