Re: Please use "web-platform-tests --manifest-update" for updating wpt tests

2016-04-20 Thread James Graham
On 20/04/16 14:13, Nathan Froyd wrote: On Wed, Apr 20, 2016 at 8:59 AM, James Graham wrote: On 20/04/16 13:53, Josh Matthews wrote: Servo has a script [1] that runs on the build machine that executes --manifest-update and checks whether the contents of MANFEST.json is

Re: Please use "web-platform-tests --manifest-update" for updating wpt tests

2016-04-20 Thread Nathan Froyd
On Wed, Apr 20, 2016 at 8:59 AM, James Graham wrote: > On 20/04/16 13:53, Josh Matthews wrote: >> Servo has a script [1] that runs on the build machine that executes >> --manifest-update and checks whether the contents of MANFEST.json is >> different before and after. We

Re: Please use "web-platform-tests --manifest-update" for updating wpt tests

2016-04-20 Thread James Graham
On 20/04/16 13:53, Josh Matthews wrote: Servo has a script [1] that runs on the build machine that executes --manifest-update and checks whether the contents of MANFEST.json is different before and after. We could do the same for Gecko and make it turn the job orange on treeherder. I plan to

Re: Please use "web-platform-tests --manifest-update" for updating wpt tests

2016-04-20 Thread Josh Matthews
Servo has a script [1] that runs on the build machine that executes --manifest-update and checks whether the contents of MANFEST.json is different before and after. We could do the same for Gecko and make it turn the job orange on treeherder. Cheers, Josh [1]

Please use "web-platform-tests --manifest-update" for updating wpt tests

2016-04-20 Thread Jeremy Chen
Hi all, For web-platform-test, it seems some people still update manifest.json manually, instead of running "--manifest-update" mach command. Running "--manifest-update" assures the manifest to be in lexicographic order; however, manually updating manifest may sometimes accidentally add test info