mach file-info bugzilla-component fails to run on testing/web-platform

RESOLVED FIXED in Firefox 55



Build Config
9 months ago
6 months ago


(Reporter: jmaher, Assigned: gps)


(Blocks: 1 bug)

Dependency tree / graph

Firefox Tracking Flags

(firefox55 fixed)


MozReview Requests


Submitter Diff Changes Open Issues Last Updated
Error loading review requests:


(1 attachment)



9 months ago
I was going to do some work on testing/web-platform related to BUG_COMPONENT, but I found an error:
elvis@JMAHER-WIN10 ~/mozilla-inbound
$ ./mach file-info bugzilla-component testing/web-platform
Error running mach:

    ['file-info', 'bugzilla-component', 'testing/web-platform']

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

The details of the failure are as follows:

ValueError: too many values to unpack

  File "c:\Users\elvis\mozilla-inbound\python/mozbuild/mozbuild/frontend/", line 108, in file_info_bugzilla
    for p, m in self._get_files_info(paths, rev=rev).items():
  File "c:\Users\elvis\mozilla-inbound\python/mozbuild/mozbuild/frontend/", line 218, in _get_files_info
    return reader.files_info(allpaths)
  File "c:\Users\elvis\mozilla-inbound\python/mozbuild\mozbuild\frontend\", line 1394, in files_info
    flags += self.test_defaults_for_path(ctxs)
  File "c:\Users\elvis\mozilla-inbound\python/mozbuild\mozbuild\frontend\", line 1418, in test_defaults_for_path
    for t in (mozpath.join(tests_root, path) for path, _ in obj):
  File "c:\Users\elvis\mozilla-inbound\python/mozbuild\mozbuild\frontend\", line 1418, in <genexpr>
    for t in (mozpath.join(tests_root, path) for path, _ in obj):

if I remove the definition for WEB_PLATFORM_TEST_MANIFESTS:

things work as expected.  This worked 2 weeks ago and testing/web-platform/ didn't have any changes to this section of the code in a long time.  I suspect something in mach broke.

Comment 1

6 months ago
I'm hitting this as well.
Blocks: 1352495

Comment 2

6 months ago
Any suggestions on trying to figure this out?
This is a build system bug.

Looks like WEB_PLATFORM_TEST_MANIFESTS behaves slightly differently from other test manifest variables. Should be an easy fix.
Assignee: nobody → gps
Component: mach → Build Config
Regressed in the following changeset:

changeset:   370160:beb43155b7a6
user:        James Graham <>
date:        Fri Jan 06 12:52:28 2017 +0000
summary:     Bug 1331899 - Fix wpt manifest iterator to return the test path, r=Ms2ger

diff --git a/testing/web-platform/tests/tools/manifest/ b/testing/web-platform/tests/tools/manifest/manifest.
--- a/testing/web-platform/tests/tools/manifest/
+++ b/testing/web-platform/tests/tools/manifest/
@@ -44,7 +44,7 @@ class Manifest(object):
             types = sorted(self._data.keys())
         for item_type in types:
             for path, tests in sorted(iteritems(self._data[item_type])):
-                yield item_type, tests
+                yield item_type, path, tests

     def reftest_nodes_by_url(self):
Depends on: 1331899
Comment hidden (mozreview-request)

Comment 6

6 months ago
<3 thanks gps!

Comment 7

6 months ago
Comment on attachment 8857687 [details]
Bug 1334525 - Handle tuple types and WPT manifests correctly;
Attachment #8857687 - Flags: review?(cmanchester) → review+

Comment 8

6 months ago
Pushed by
Handle tuple types and WPT manifests correctly; r=chmanchester
Last Resolved: 6 months ago
status-firefox55: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
You need to log in before you can comment on or make changes to this bug.