-
Notifications
You must be signed in to change notification settings - Fork 73
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Review Draft Publication: June 2019 #177
Conversation
This is failing, I think because of validator warnings on the encoding tables. /cc @sideshowbarker to verify that warnings about Unicode normalization forms would cause a build failure. In that case we should probably implement #178. It appears the last successful build, 4 months ago, did not run the validator on the build output. https://travis-ci.org/whatwg/encoding/builds/491657427 |
It’s failing for the warnings because in https://github.com/whatwg/whatwg.org/blob/17fbe5071c59194dec4ce9442fc3ab1749052cb8/resources.whatwg.org/build/deploy.sh#L186-L190 we explicitly give the if [ -z "$CHECKER_FILTER" ]; then
./vnu-runtime-image/bin/vnu --skip-non-html --Werror "$WEB_ROOT"
else
./vnu-runtime-image/bin/vnu --skip-non-html --Werror --filterpattern "$CHECKER_FILTER" "$WEB_ROOT"
fi That causes warnings to be treated as errors; so if there are any warnings, the process exits non-zero. But as I mentioned in #178 (comment), I think we can work around this by making the checker suppress/filter out those warnings. |
This change sets the CHECKER_FILTER environment variable with a value that causes some particular warnings to be suppressed when the https://github.com/whatwg/whatwg.org/blob/master/resources.whatwg.org/build/deploy.sh script invokes the HTML checker on the Encoding build output. The particular warnings that get suppressed are: * Text run is not in Unicode Normalization Form C. * Document uses the Unicode Private Use Area(s), which should not be used in publicly exchanged documents. (Charmod C073) Otherwise, without this change, deployment of the Encoding spec will fail, because the https://github.com/whatwg/whatwg.org/blob/master/resources.whatwg.org/build/deploy.sh script causes the HTML checker to be invoked with the --Werror argument, which causes warnings to be treated as errors, which in turn causes the HTML checker process to exit non-zero. Fixes #178 Relates to #177
This change sets the CHECKER_FILTER environment variable with a value that causes some particular warnings to be suppressed when the https://github.com/whatwg/whatwg.org/blob/master/resources.whatwg.org/build/deploy.sh script invokes the HTML checker on the Encoding build output. The particular warnings that get suppressed are: * Text run is not in Unicode Normalization Form C. * Document uses the Unicode Private Use Area(s), which should not be used in publicly exchanged documents. (Charmod C073) Otherwise, without this change, deployment of the Encoding spec will fail, because the https://github.com/whatwg/whatwg.org/blob/master/resources.whatwg.org/build/deploy.sh script causes the HTML checker to be invoked with the --Werror argument, which causes warnings to be treated as errors, which in turn causes the HTML checker process to exit non-zero. Fixes #178 Relates to #177
This change sets the CHECKER_FILTER environment variable with a value that causes some particular warnings to be suppressed when the https://github.com/whatwg/whatwg.org/blob/master/resources.whatwg.org/build/deploy.sh script invokes the HTML checker on the Encoding build output. The particular warnings that get suppressed are: * Text run is not in Unicode Normalization Form C. * Document uses the Unicode Private Use Area(s), which should not be used in publicly exchanged documents. (Charmod C073) Otherwise, without this change, deployment of the Encoding spec will fail, because the https://github.com/whatwg/whatwg.org/blob/master/resources.whatwg.org/build/deploy.sh script causes the HTML checker to be invoked with the --Werror argument, which causes warnings to be treated as errors, which in turn causes the HTML checker process to exit non-zero. Fixes #178 Relates to #177
This change sets the CHECKER_FILTER environment variable with a value that causes some particular warnings to be suppressed when the https://github.com/whatwg/whatwg.org/blob/master/resources.whatwg.org/build/deploy.sh script invokes the HTML checker on the Encoding build output. The particular warnings that get suppressed are: * Text run is not in Unicode Normalization Form C. * Document uses the Unicode Private Use Area(s), which should not be used in publicly exchanged documents. (Charmod C073) Otherwise, without this change, deployment of the Encoding spec will fail, because the https://github.com/whatwg/whatwg.org/blob/master/resources.whatwg.org/build/deploy.sh script causes the HTML checker to be invoked with the --Werror argument, which causes warnings to be treated as errors, which in turn causes the HTML checker process to exit non-zero. Fixes #178 Relates to #177
This change adds an .htmlcheckerfilter file containing regular-expression patterns that causes some particular warnings to be suppressed when the https://github.com/whatwg/whatwg.org/blob/master/resources.whatwg.org/build/deploy.sh script invokes the HTML checker on the Encoding build output. The particular warnings that get suppressed are: * Text run is not in Unicode Normalization Form C. * Document uses the Unicode Private Use Area(s), which should not be used in publicly exchanged documents. (Charmod C073) Otherwise, without this change, deployment of the Encoding spec will fail, because the https://github.com/whatwg/whatwg.org/blob/master/resources.whatwg.org/build/deploy.sh script causes the HTML checker to be invoked with the --Werror argument, which causes warnings to be treated as errors, which in turn causes the HTML checker process to exit non-zero. Fixes #178 Relates to #177
This change adds an .htmlcheckerfilter file containing regular-expression patterns that causes some particular warnings to be suppressed when the https://github.com/whatwg/whatwg.org/blob/master/resources.whatwg.org/build/deploy.sh script invokes the HTML checker on the Encoding build output. The particular warnings that get suppressed are: * Text run is not in Unicode Normalization Form C. * Document uses the Unicode Private Use Area(s), which should not be used in publicly exchanged documents. (Charmod C073) Otherwise, without this change, deployment of the Encoding spec will fail, because the https://github.com/whatwg/whatwg.org/blob/master/resources.whatwg.org/build/deploy.sh script causes the HTML checker to be invoked with the --Werror argument, which causes warnings to be treated as errors, which in turn causes the HTML checker process to exit non-zero. Fixes #178 Relates to #177
This change adds an .htmlcheckerfilter file containing regular-expression patterns that causes some particular warnings to be suppressed when the https://github.com/whatwg/whatwg.org/blob/master/resources.whatwg.org/build/deploy.sh script invokes the HTML checker on the Encoding build output. The particular warnings that get suppressed are: * Text run is not in Unicode Normalization Form C. * Document uses the Unicode Private Use Area(s), which should not be used in publicly exchanged documents. (Charmod C073) Otherwise, without this change, deployment of the Encoding spec will fail, because the https://github.com/whatwg/whatwg.org/blob/master/resources.whatwg.org/build/deploy.sh script causes the HTML checker to be invoked with the --Werror argument, which causes warnings to be treated as errors, which in turn causes the HTML checker process to exit non-zero. Fixes #178. Relates to #177.
789457c
to
0882eed
Compare
A Review Draft for this Workstream will be published shortly, by merging this pull request.
Under the WHATWG IPR Policy, Participants may, within 45 days after publication of a Review Draft, exclude certain Essential Patent Claims from the Review Draft Licensing Obligations. See the IPR Policy for details.