Closed Bug 1103906 Opened 5 years ago Closed 5 years ago

Update to latest web-platform-tests

Categories

(Testing :: web-platform-tests, defect)

x86_64
Linux
defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED
mozilla36

People

(Reporter: jgraham, Assigned: jgraham)

References

Details

Attachments

(1 file)

No description provided.
Attachment #8527605 - Flags: review?(Ms2ger) → review?(ato)
Comment on attachment 8527605 [details] [diff] [review]
Enable logger for mach web-platform-tests-update script.

Review of attachment 8527605 [details] [diff] [review]:
-----------------------------------------------------------------

::: testing/web-platform/mach_commands.py
@@ +73,5 @@
>          if kwargs["config"] is None:
>              kwargs["config"] = os.path.join(self.topsrcdir, 'testing', 'web-platform', 'wptrunner.ini')
>  
>          wptcommandline.set_from_config(kwargs)
> +        update.setup_logging(kwargs, {"mach": sys.stdout})

Missing from testing/web-platform/harness/wptrunner/update.py?
Well sure, it depends on https://hg.mozilla.org/try/rev/77044070ccfc which contains all the wpt updates which are upstream, or in review on critic (I hope).
Comment on attachment 8527605 [details] [diff] [review]
Enable logger for mach web-platform-tests-update script.

Review of attachment 8527605 [details] [diff] [review]:
-----------------------------------------------------------------

OK, the patch looks fine.  The relevant bit I already reviewed here: https://critic.hoppipolla.co.uk/r/3267
Attachment #8527605 - Flags: review?(ato) → review+
Blocks: 1098200
You need to log in before you can comment on or make changes to this bug.