Closed Bug 1395286 Opened 5 years ago Closed 4 years ago
Moving resources (e
.g . to content-accessible) causes packaging failures (at least for Android) without a clobber
59 bytes, text/x-review-board-request
The first error message I get is > 0:01.48 IOError: [Errno 2] No such file or directory: '../../../dist/bin/chrome/en-US/locale/en-US/browser/handling.properties' > 0:01.53 /home/jan/Mozilla/mozilla-central/toolkit/mozapps/installer/packager.mk:36: recipe for target 'stage-package' failed and if I backout bug 1255404, packaging then fails with > ERROR: The following duplicated files are not allowed: > 0:24.79 chrome/toolkit/res/viewsource.css > 0:24.79 contentaccessible/viewsource.css As far I can see, we're missing a clobber here.
Comment on attachment 8902855 [details] Bug 1395286 - Clobber to fix packaging failure on Android. https://reviewboard.mozilla.org/r/174550/#review179832 Usually for a clobber commit, you mention the original bug in the commit message (e.g. ). And then you file a bug about the need to clobber; the goal of that bug is to make us not require a clobber for similar cases in the future. In this case, I suggest you keep this bug open as the fix-clobber bug, and land this CLOBBER commit with the original bug number (e.g. bug 1255404).  https://hg.mozilla.org/mozilla-central/rev/278c3bf8f7f4
Attachment #8902855 - Flags: review?(nchen) → review+
Pushed by email@example.com: https://hg.mozilla.org/integration/autoland/rev/71ee70900245 Clobber to fix packaging failure on Android. r=jchen
(In reply to Jim Chen [:jchen] [:darchons] from comment #3) > In this case, I suggest you keep this bug open as the > fix-clobber bug, and land this CLOBBER commit with the original bug number > (e.g. bug 1255404). Sorry for not fully following this, but I didn't want to delay this until I can access my development machine again. I probably won't be looking into this further, but hazarding a guess regarding bug 863246, the files presumably get copied into the object directory at some stage and we only copy the current set of files, but don't delete any outdated files that are no longer necessary. I wonder why that didn't cause any problems for Desktop builds, though - is it just because people are less likely to package Desktop builds as it's not necessary for running the build, whereas for Android packaging is mandatory for obtaining a useable APK? Or is the Desktop packaging immune to those kinds of duplicated files?
Assignee: jh+bugzilla → nobody
Summary: Failure during packaging (related to handling.properties respectively viewsource.css) → Moving resources (e.g. to contentaccessible) causes packaging failures (at least on Android) without a clobber
Summary: Moving resources (e.g. to contentaccessible) causes packaging failures (at least on Android) without a clobber → Moving resources (e.g. to content-accessible) causes packaging failures (at least for Android) without a clobber
The leave-open keyword is there and there is no activity for 6 months. :nalexander, maybe it's time to close this bug?
Let's close this as a dupe of Bug 1500924.
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → DUPLICATE
Duplicate of bug: 1500924
You need to log in before you can comment on or make changes to this bug.