sami7777/triangles_v5
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix macOS build: restrict -z relro/now to Linux only
Build All Platforms / test-linux-unit (push) Failing after 40s
Details
Build All Platforms / build-linux-qt (push) Failing after 40s
Details
Build All Platforms / build-linux-daemon (push) Failing after 40s
Details
Build All Platforms / build-windows-qt (push) Has been cancelled
Details
Build All Platforms / build-windows-daemon (push) Has been cancelled
Details
Build All Platforms / build-macos (push) Has been cancelled
Details
Build All Platforms / release (push) Has been cancelled
Details
Build All Platforms / Trigger TRI-PI ARM64 Build (push) Has been cancelled
Details

Browse Source 
The -Wl,-z,relro and -Wl,-z,now flags are ELF-specific and not
supported by macOS's linker. Guard them with if(NOT APPLE).

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
Sami Ahmed
2026-04-05 04:04:48 -07:00
parent 0a0129cbcc
commit 104778fa61
1 changed files with 4 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
cmake/AddCompilerFlags.cmake
+4 -1
View File
@@ -22,7 +22,10 @@ if(NOT WIN32)
add_compile_options(-fno-stack-protector)
add_compile_options(-fstack-protector-all -Wstack-protector)
add_compile_definitions(_FORTIFY_SOURCE=2)
add_link_options(-Wl,-z,relro -Wl,-z,now)
# -z relro/now is ELF-only (Linux); macOS linker doesn't support it
if(NOT APPLE)
add_link_options(-Wl,-z,relro -Wl,-z,now)
endif()
endif()
# ── PIE (position-independent executables) ──