This is an archive of the discontinued LLVM Phabricator instance.

[libc++] Fix the check-format job
ClosedPublic

Authored by ldionne on Jun 21 2023, 6:10 AM.

Details

Reviewers
philnik
Mordante
Group Reviewers
Restricted Project
Commits
rGb3569280586a: [libc++] Fix the check-format job
Summary

Previously, it wouldn't take into account files in ignore_format.txt
(at least not on OSX) because the find command would return file names
like libcxx/src//new_handler.cpp, which never matched the file names
in ignore_format.txt.

Diff Detail

Event Timeline

ldionne created this revision.Jun 21 2023, 6:10 AM
Herald added a project: Restricted Project. · View Herald TranscriptJun 21 2023, 6:10 AM
Herald added a subscriber: arichardson. · View Herald Transcript
ldionne requested review of this revision.Jun 21 2023, 6:10 AM
Herald added a project: Restricted Project. · View Herald TranscriptJun 21 2023, 6:10 AM
Herald added a reviewer: Restricted Project. · View Herald Transcript
Mordante accepted this revision.Jun 21 2023, 9:31 AM
Mordante added a subscriber: Mordante.

LGTM!

This revision is now accepted and ready to land.Jun 21 2023, 9:31 AM
This revision was automatically updated to reflect the committed changes.