This is an archive of the discontinued LLVM Phabricator instance.

[libc++] Add helper script libcxx-lit for running tests
ClosedPublic

Authored by ldionne on Apr 18 2023, 6:27 AM.

Details

Diff Detail

Event Timeline

ldionne created this revision.Apr 18 2023, 6:27 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 18 2023, 6:27 AM
Herald added a subscriber: arichardson. · View Herald Transcript
ldionne requested review of this revision.Apr 18 2023, 6:27 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 18 2023, 6:27 AM
Herald added a reviewer: Restricted Project. · View Herald Transcript
ldionne added inline comments.Apr 18 2023, 6:28 AM
libcxx/utils/libcxx-lit
37

Does realpath work on Linux?

jloser accepted this revision.Apr 18 2023, 7:09 AM
jloser added a subscriber: jloser.

Thanks for adding this for others. This is the same script I use locally as well that I got from you ages ago.

libcxx/utils/libcxx-lit
37
arichardson added inline comments.Apr 18 2023, 10:25 PM
libcxx/utils/libcxx-lit
37

I believe realpath should be available on most systems but may not always be installed by default. Also I seem to recall it not being there on macos? But is this actually needed? Shouldn't both cmake and lit be fine with a relative path?

ldionne updated this revision to Diff 514998.Apr 19 2023, 9:41 AM

Drop realpath

ldionne accepted this revision.Apr 19 2023, 12:35 PM
This revision is now accepted and ready to land.Apr 19 2023, 12:35 PM
This revision was landed with ongoing or failed builds.Apr 19 2023, 12:36 PM
This revision was automatically updated to reflect the committed changes.