Skip to content

Commit 7b7fc67

Browse files
committed
Auto merge of #27625 - wthrowe:find, r=alexcrichton
New enough find on Linux doesn't support "-perm +..." and suggests using "-perm /..." instead, but that doesn't work on Windows. Hopefully all platforms are happy with this expanded version. I don't have access to a Windows development system to test this, so someone needs to verify that this actually works there before merging. Closes #19981.
2 parents bb954cf + 904e428 commit 7b7fc67

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

mk/tests.mk

+3-2
Original file line numberDiff line numberDiff line change
@@ -268,9 +268,10 @@ tidy-basic:
268268
.PHONY: tidy-binaries
269269
tidy-binaries:
270270
@$(call E, check: binaries)
271-
$(Q)find $(S)src -type f -perm +a+x \
271+
$(Q)find $(S)src -type f \
272+
\( -perm -u+x -or -perm -g+x -or -perm -o+x \) \
272273
-not -name '*.rs' -and -not -name '*.py' \
273-
-and -not -name '*.sh' \
274+
-and -not -name '*.sh' -and -not -name '*.pp' \
274275
| grep '^$(S)src/jemalloc' -v \
275276
| grep '^$(S)src/libuv' -v \
276277
| grep '^$(S)src/llvm' -v \

0 commit comments

Comments
 (0)