Moving resources (e.g. to content-accessible) causes packaging failures (at least for Android) without a clobber

RESOLVED DUPLICATE of bug 1500924

Status

()

defect
RESOLVED DUPLICATE of bug 1500924
2 years ago
6 months ago

People

(Reporter: JanH, Unassigned)

Tracking

57 Branch
All
Android
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

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.
Assignee: nobody → jh+bugzilla
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)

Comment 3

2 years ago
mozreview-review
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. [1]). 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).

[1] https://hg.mozilla.org/mozilla-central/rev/278c3bf8f7f4
Attachment #8902855 - Flags: review?(nchen) → review+

Comment 4

2 years ago
Pushed by mozilla@buttercookie.de:
https://hg.mozilla.org/integration/autoland/rev/71ee70900245
Clobber to fix packaging failure on Android. r=jchen
Keywords: leave-open
(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
Blocks: 1395486
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?
Flags: needinfo?(nalexander)
Let's close this as a dupe of Bug 1500924.
Status: NEW → RESOLVED
Last Resolved: 6 months ago
Flags: needinfo?(nalexander)
Resolution: --- → DUPLICATE
Duplicate of bug: 1500924
You need to log in before you can comment on or make changes to this bug.