The test case added in D62718 did not work unless the user was root because write bits were not set for the output file. This change uses only permissions with user write (0200) to ensure tests pass regardless of the users permissions.
Details
Details
- Reviewers
jhenderson rupprecht MaskRay • espindola alexander-shaposhnikov - Commits
- rG72deb0476d4a: Merging r366527: --------------------------------------------------------------…
rL366758: Merging r366527:
rL366527: [test] [llvm-objcopy] Fix broken test case
rG553c29faa262: [test] [llvm-objcopy] Fix broken test case
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
llvm/test/tools/llvm-objcopy/ELF/respect-umask.test | ||
---|---|---|
34 | You can join the two lines: ls -l %t2 | cut -f 1 -d ' ' | cmp - %t.0600 |
You can join the two lines:
ls -l %t2 | cut -f 1 -d ' ' | cmp - %t.0600