Skip to content

Do not use GNU find features on FreeBSD#345

Merged
sunfishcode merged 1 commit intoWebAssembly:mainfrom
kiancross:free-bsd-fix
Sep 6, 2023
Merged

Do not use GNU find features on FreeBSD#345
sunfishcode merged 1 commit intoWebAssembly:mainfrom
kiancross:free-bsd-fix

Conversation

@kiancross
Copy link
Copy Markdown
Contributor

FreeBSD (like MacOS) does not support the -executable argument for find. This commit checks if the OS is FreeBSD, and if so, uses the same workaround as already in place on MacOS.

A more portable solution might be to test if the -executable flag is available, and fallback to -perm if not.

FreeBSD (like MacOS) does not support the `-executable` argument for
`find`. This commit checks if the OS is FreeBSD, and if so, uses the
same workaround as already in place on MacOS.
@sunfishcode
Copy link
Copy Markdown
Member

Thanks!

@sunfishcode sunfishcode merged commit bbfb973 into WebAssembly:main Sep 6, 2023
@kiancross kiancross deleted the free-bsd-fix branch September 6, 2023 14:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants