GerbilSoft/rom-properties
1
Fork 0
mirror of https://github.com/GerbilSoft/rom-properties.git synced 2025-06-18 11:35:38 -04:00
Code Issues Actions 2 Packages Projects Releases Wiki Activity

[cmake] gcc.cmake: Fix DT_RELR enablement.
Some checks are pending
Codecov / run (push) Waiting to run
Details
CodeQL / Analyze (cpp) (push) Waiting to run
Details

Browse Source 
It seems this was broken since it was initially implemented in v2.0
in commit d732fbc010
([cmake] platform/gcc.cmake: Initial support for detecting DT_RELR.)
because TMP_HAVE_DT_RELR was unset too early.
This commit is contained in:
David Korth 2025-04-28 01:03:31 -04:00
parent 3b8d47fa0b
commit 7d379f826d
2 changed files with 3 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
NEWS.md
View File

@ -28,6 +28,8 @@
why this changed in Qt6. A workaround has been applied to fix it.
* ISO: Fix a typo that broke "Volume Size" on CD-i volumes.
* Affects: v1.8 - v2.5
* Build system: DT_RELR detection was broken and didn't work properly
since it was implemented in v2.0. It now works properly.
* Other changes:
* Added support for localsearch-3.8, the new name of Tracker.

2
cmake/platform/gcc.cmake
View File

@ -219,12 +219,12 @@ IF(UNIX AND NOT APPLE)
SET(TMP_HAVE_DT_RELR FALSE)
MESSAGE(STATUS "Checking if the system supports DT_RELR - no, needs glibc-2.36 or later")
ENDIF()
UNSET(TMP_HAVE_DT_RELR)
ELSE(_ld_out MATCHES "-z pack-relative-relocs")
SET(TMP_HAVE_DT_RELR FALSE)
MESSAGE(STATUS "Checking if the system supports DT_RELR - no, needs binutils-2.38 or later")
ENDIF(_ld_out MATCHES "-z pack-relative-relocs")
SET(HAVE_DT_RELR ${TMP_HAVE_DT_RELR} CACHE INTERNAL "System supports DT_RELR")
UNSET(TMP_HAVE_DT_RELR)
ENDIF(NOT DEFINED HAVE_DT_RELR)
IF(HAVE_DT_RELR)