[libcxx-commits] [PATCH] D99862: [libc++] [CI] Validate the output of the generated scripts.

[Mark de Wever via Phabricator via libcxx-commits]

Mordante added a comment.

In D99862#2674415 <https://reviews.llvm.org/D99862#2674415>, @curdeius wrote:

> I meant the temporary change to provoke the failure:). That's already reverted from what I see. :)

Yes I reverted that.



================
Comment at: libcxx/utils/ci/run-buildbot:144
+    # Check if the diffs are empty, fail otherwise.
+    ! grep -q '^--- a' ${BUILD_DIR}/generated_output.patch
+;;
----------------
ldionne wrote:
> It's annoying that we'll be modifying our git index just to run this test, i.e. if it generates changes, then those changes will "pollute" the working directory.
> 
> I think the cleanest approach to this would be to change the various header/test generation files to behave more like `clang-format` and to produce a diff that we can then apply instead of making the changes in place, but that would be more work indeed. That way, we could run this test without modifying our local state.
> 
> That's definitely not blocking for this review, but something to keep in mind.
Good to know. I wasn't aware the same checkout was reused by other jobs. If wanted I can add a `git reset --hard HEAD` after generating the diff.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99862