This is an archive of the discontinued LLVM Phabricator instance.

[profile] Make a test work if run by the super-user
ClosedPublic

Authored by vsk on Mar 29 2016, 7:43 AM.

Details

Summary

This test checks if we get the appropriate error message when writing to an unwritable file. However, it fails on bots which run check-all using sudo (in general this is something we'd like to support). Fix up the test so it works even if run as sudo.

Note: I'm not sure if this patch breaks Windows compatibility. If someone can tell, please comment :).

Diff Detail

Event Timeline

vsk updated this revision to Diff 51909.Mar 29 2016, 7:43 AM
vsk retitled this revision from to [profile] Make a test work if run by the super-user.
vsk updated this object.
vsk added reviewers: davidxl, kubamracek.
vsk added a subscriber: llvm-commits.
vsk accepted this revision.Mar 29 2016, 12:33 PM
vsk added a reviewer: vsk.

Thanks, committed r264773!

This revision is now accepted and ready to land.Mar 29 2016, 12:33 PM
vsk closed this revision.Mar 29 2016, 12:33 PM