Ryan Carsten Schmidt

Results 570 comments of Ryan Carsten Schmidt

> Could you please check if this works for you Thanks; I'll try it. > proposes to use `perl -MCwd -e 'print Cwd::abs_path shift'` instead of `readlink -f`, which should...

> ```diff > + ${HOST_PREFIX}install_name_tool \${changes} -id \${3}/\${i} \${libdir}\${lib} > ``` Almost: you forgot the `/` between `\${libdir}` and `\${lib}`. ``` error: /Library/Developer/CommandLineTools/usr/bin/install_name_tool: can't open file: /opt/local/var/macports/build/wxWidgets-3.2-b818c50a/work/wxWidgets-3.2.8.1/build/liblibwx_baseu-3.2.0.dylib (No such file...

> Adding `exit $?` might be a good idea, but maybe we should add `set -e` in the beginning of the script instead? For something as short as this, it...

> A MacPorts user provided a different patch Ok I've looked at that patch: ```diff --- configure.in.orig 2025-05-26 02:15:56.000000000 +0900 +++ configure.in 2025-07-29 22:57:00.000000000 +0900 @@ -4142,15 +4142,13 @@ DYLIB_RPATH_POSTLINK="${HOST_PREFIX}install_name_tool...

> but I'm afraid the alternative patch just restores the original version of this code which means that it reintroduces #23143 fixed 7fdc13241316dc73cf71576992c2c1fb287f235b (Fix dylib symlinks on Apple Silicon Macs,...

In 1.4.5 the first line of config/config.guess and config/config.sub and config/install-sh is `#!/usr/bin/sh` while the first line of the other files in the config/ directory is `#!/bin/sh`. In 1.4.4 the...

But also, `Fixed` is a type that was introduced on the original Macintosh in 1984. It was defined for use with the fixed-point arithmetic utility routines `FixRatio`, `FixMul`, and `FixRound`...

> `TARGET_OS_MAC` is defined in [TargetOSMacros.def](https://github.com/llvm/llvm-project/pull/74676/files#diff-40e48434c42a87f7fef15cee3f435c8a3977b478cd77cd3c33acbe364f0116c3) which was added in LLVM/Clang 18x - see [llvm/llvm-project#74676](https://github.com/llvm/llvm-project/pull/74676). You're talking about version 18 of llvm.org's llvm/clang, but Apple's llvm/clang (i.e. the one included...

Alternately, perhaps what you want is for `EnableCompilerFlag` to accept spaces in its flag argument and for `EnableCompilerFlag` to convert the spaces to semicolons prior to passing the value to...

No this is evidently not correct either. From the cmake-build-and-test-check log: ``` /usr/bin/ld: warning: -z ;noexecstack ignored ``` so the flag is still not being passed around in the correct...