Auto merge of #18795 - servo:jdm-patch-2, r=nox

Make update_manifest.sh more verbose.

This is an attempt to figure out why https://github.com/servo/servo/issues/17450 occurs.

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/18795)
<!-- Reviewable:end -->
This commit is contained in:
bors-servo 2017-10-09 20:06:34 -05:00 committed by GitHub
commit 2a179cec2b

View file

@ -8,13 +8,17 @@ set -o errexit
set -o nounset
set -o pipefail
echo "About to update manifest."
# We shouldn't need any binary at all to update the manifests.
# Adding "SKIP_TESTS" to skip tests, it doesn't really skip the tests.
# It will run "run_wpt" with "'test_list': ['SKIP_TESTS']",
# and then pass it into wptrunner, which won't be able to find any tests named
# "SKIP_TESTS", and thus won't run any.
# Adding "--binary=" to skip looking for a compiled servo binary.
./mach test-wpt --manifest-update --binary= SKIP_TESTS > /dev/null
./mach test-wpt --manifest-update --binary= SKIP_TESTS
echo "Updated manifest; about to check if any changes were made to it."
diff="$(git diff -- tests/*/MANIFEST.json)"
echo "${diff}"