This is an archive of the discontinued LLVM Phabricator instance.

[arcanist] rm -f for write protected temp files
ClosedPublic

Authored by nickdesaulniers on Apr 17 2023, 11:41 AM.

Details

Summary

When running arc diff on a newly added file,
utils/arcanist/clang-format.sh is run. For new files there is no change,
but the script will bail. Without the use of the -f flag, we get
interactive prompts like:
rm: remove write-protected regular file '/tmp/tmp.ReMybrBw35'?

Diff Detail

Event Timeline

Herald added a project: Restricted Project. · View Herald TranscriptApr 17 2023, 11:41 AM
nickdesaulniers requested review of this revision.Apr 17 2023, 11:41 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 17 2023, 11:41 AM
MaskRay accepted this revision.Apr 17 2023, 12:23 PM
This revision is now accepted and ready to land.Apr 17 2023, 12:23 PM
This revision was automatically updated to reflect the committed changes.