search

tpoechtrager / cctools-port

Apple cctools port for Linux and *BSD
721 stars 165 forks source link

better otool wrapper #145

Closed Un1q32 closed 1 month ago

Un1q32 commented 7 months ago

The main change is that it tries otool first, then llvm-otool if that fails. This fixes the otool wrapper on macOS, where llvm-otool isn't in PATH but otool is.

Un1q32 commented 6 months ago

bump

Un1q32 commented 3 months ago

bump 2

Un1q32 commented 3 months ago

It's worth noting that this won't matter too much if cctools 1010.6 is merged, since it brings back Apple's otool. Would still be nice for anyone who needs 1009.2 in particular for whatever reason.

tpoechtrager commented 1 month ago

I never merged this because it could theoretically lead to a fork bomb. When the otool wrapper is installed without a host prefix it would execute itself infinitely.

Un1q32 commented 1 month ago

I never merged this because it could theoretically lead to a fork bomb. When the otool wrapper is installed without a host prefix it would execute itself infinitely.

I don't think that would be a forkbomb, since there is no fork(), just exec(), so it wouldn't crash a system, but it's still true that it would be a bug

Un1q32 commented 1 month ago

Yeah it just pins one cpu core until interrupted, I could mod it to use known absolute paths for otool but now that real otool sources are back in 1010.6 I don't see much point.