Skip to content

Commit

Permalink
Auto merge of #18795 - servo:jdm-patch-2, r=nox
Browse files Browse the repository at this point in the history
Make update_manifest.sh more verbose.

This is an attempt to figure out why #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 -->
  • Loading branch information
bors-servo committed Oct 10, 2017
2 parents 54f6e87 + 398a15e commit 2a179ce
Showing 1 changed file with 5 additions and 1 deletion.
6 changes: 5 additions & 1 deletion etc/ci/manifest_changed.sh
Original file line number Diff line number Diff line change
Expand Up @@ -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}"
Expand Down

0 comments on commit 2a179ce

Please sign in to comment.