[libcxx-commits] [PATCH] D101303: [WIP][libc++][DI] Fix check-generated-output.

Mark de Wever via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Mon Apr 26 10:39:33 PDT 2021


Mordante added inline comments.


================
Comment at: libcxx/utils/ci/run-buildbot:10-11
 
 set -ex
 set -o pipefail
 
----------------
Quuxplusone wrote:
> @Mordante: Are you sure the script lets non-ASCII characters pass? My intent/understanding had been that `set -e` makes the build fail if any of these greps return nonzero; and I believe that that's what I was observing when I made the PR originally.
Yes I noticed the non-ASCII hyphen in some comment which was copied from the Standard. So I was surprised the build passed.
An example is this build https://buildkite.com/llvm-project/libcxx-ci/builds/2807#2ec7bc42-3bac-4a7b-98ac-c3b8d03390c7 


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D101303/new/

https://reviews.llvm.org/D101303



More information about the libcxx-commits mailing list