Closed Bug 1429815 Opened 7 years ago Closed 7 years ago

`mach watch` fails with "ValueError: Item already in manifest: chrome/content/*"

Categories

(Firefox Build System :: General, defect)

defect
Not set
normal

Tracking

(firefox60 fixed)

RESOLVED FIXED
mozilla60
Tracking Status
firefox60 --- fixed

People

(Reporter: bgrins, Assigned: nalexander)

References

Details

Attachments

(1 file)

On an m-c checkout, running `./mach watch` I get the following error: Error running mach: ['watch'] The error occurred in code that was called by the mach command. This is either a bug in the called code itself or in the way that mach is calling it. You should consider filing a bug for this issue. If filing a bug, please include the full output of mach, including this error message. The details of the failure are as follows: ValueError: Item already in manifest: chrome/content/* File "/mozilla-central/python/mozbuild/mozbuild/mach_commands.py", line 112, in watch return daemon.watch() File "/mozilla-central/python/mozbuild/mozbuild/faster_daemon.py", line 278, in watch for change in self.output_changes(verbose=verbose): File "/mozilla-central/python/mozbuild/mozbuild/faster_daemon.py", line 254, in output_changes for change in self.input_changes(verbose=verbose): File "/mozilla-central/python/mozbuild/mozbuild/faster_daemon.py", line 203, in input_changes input_to_outputs = self.file_copier.input_to_outputs_tree() File "/mozilla-central/python/mozbuild/mozbuild/util.py", line 962, in __get__ setattr(instance, name, self.func(instance)) File "/mozilla-central/python/mozbuild/mozbuild/faster_daemon.py", line 93, in file_copier 'faster', 'unified_install_dist_bin')) File "/mozilla-central/python/mozbuild/mozpack/manifests.py", line 121, in __init__ self._load_from_fileobj(fh) File "/mozilla-central/python/mozbuild/mozpack/manifests.py", line 158, in _load_from_fileobj self.add_pattern_link(base, pattern, dest) File "/mozilla-central/python/mozbuild/mozpack/manifests.py", line 297, in add_pattern_link (self.PATTERN_LINK, base, pattern, dest)) File "/mozilla-central/python/mozbuild/mozpack/manifests.py", line 332, in _add_entry raise ValueError('Item already in manifest: %s' % dest)
Nick, any idea what's going on here? I don't see `chrome/content/*` in the source: https://searchfox.org/mozilla-central/search?q=chrome%2Fcontent%2F*&path=, but I do see some hits in the objdir: https://dxr.mozilla.org/mozilla-central/search?q=chrome%2Fcontent%2F*%22&redirect=false
Flags: needinfo?(nalexander)
(In reply to Brian Grinstead [:bgrins] from comment #1) > Nick, any idea what's going on here? I don't see `chrome/content/*` in the > source: > https://searchfox.org/mozilla-central/search?q=chrome%2Fcontent%2F*&path=, > but I do see some hits in the objdir: > https://dxr.mozilla.org/mozilla-central/ > search?q=chrome%2Fcontent%2F*%22&redirect=false I dug into this; here's what is happening. There's a bug in InstallManifest::add_entries_from, some of the manifest entries bake their destination into both the manifest key and the manifest value, and add_entries_from with a non-empty "base" parameter to preprend to the destination only updates the manifest key and not the value. Patch inbound.
Flags: needinfo?(nalexander)
Assignee: nobody → nalexander
Status: NEW → ASSIGNED
Attachment #8944026 - Flags: review?(core-build-config-reviews)
Comment on attachment 8944026 [details] Bug 1429815 - Fix InstallManifest::add_entries_from with non-trivial base. https://reviewboard.mozilla.org/r/214208/#review222430
Attachment #8944026 - Flags: review?(gps) → review+
Pushed by nalexander@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/31a0af4e29bb Fix InstallManifest::add_entries_from with non-trivial base. r=gps
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla60
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: