diff options
| author | 2023-06-17 01:35:35 -0400 | |
|---|---|---|
| committer | 2023-06-17 01:48:46 -0400 | |
| commit | 4cbdce17b69f4ab1f8da29c6718bec09e11b8125 (patch) | |
| tree | 4dcef8819eacf2fb3d096a041dfaa714d193b658 /externals/nx_tzdb | |
| parent | cmake: Add warn about cross compiling, disable android (diff) | |
| download | yuzu-4cbdce17b69f4ab1f8da29c6718bec09e11b8125.tar.gz yuzu-4cbdce17b69f4ab1f8da29c6718bec09e11b8125.tar.xz yuzu-4cbdce17b69f4ab1f8da29c6718bec09e11b8125.zip | |
nx_tzdb: Directly reference variables in if statements
Addresses review feedback.
Diffstat (limited to 'externals/nx_tzdb')
| -rw-r--r-- | externals/nx_tzdb/CMakeLists.txt | 14 | ||||
| -rw-r--r-- | externals/nx_tzdb/NxTzdbCreateHeader.cmake | 8 |
2 files changed, 11 insertions, 11 deletions
diff --git a/externals/nx_tzdb/CMakeLists.txt b/externals/nx_tzdb/CMakeLists.txt index 8429f82e5..d5a1c6317 100644 --- a/externals/nx_tzdb/CMakeLists.txt +++ b/externals/nx_tzdb/CMakeLists.txt | |||
| @@ -11,16 +11,16 @@ find_program(GNU_DATE date) | |||
| 11 | 11 | ||
| 12 | set(CAN_BUILD_NX_TZDB true) | 12 | set(CAN_BUILD_NX_TZDB true) |
| 13 | 13 | ||
| 14 | if ("${GIT}" STREQUAL "GIT-NOTFOUND") | 14 | if (NOT GIT) |
| 15 | set(CAN_BUILD_NX_TZDB false) | 15 | set(CAN_BUILD_NX_TZDB false) |
| 16 | endif() | 16 | endif() |
| 17 | if ("${GNU_MAKE}" STREQUAL "GNU_MAKE-NOTFOUND") | 17 | if (NOT GNU_MAKE) |
| 18 | set(CAN_BUILD_NX_TZDB false) | 18 | set(CAN_BUILD_NX_TZDB false) |
| 19 | endif() | 19 | endif() |
| 20 | if ("${GNU_DATE}" STREQUAL "GNU_DATE-NOTFOUND") | 20 | if (NOT GNU_DATE) |
| 21 | set(CAN_BUILD_NX_TZDB false) | 21 | set(CAN_BUILD_NX_TZDB false) |
| 22 | endif() | 22 | endif() |
| 23 | if ("${CMAKE_SYSTEM_NAME}" STREQUAL "Windows" OR ANDROID) | 23 | if (CMAKE_SYSTEM_NAME STREQUAL "Windows" OR ANDROID) |
| 24 | # tzdb_to_nx currently requires a posix-compliant host | 24 | # tzdb_to_nx currently requires a posix-compliant host |
| 25 | # MinGW and Android are handled here due to the executable format being different from the host system | 25 | # MinGW and Android are handled here due to the executable format being different from the host system |
| 26 | # TODO (lat9nq): cross-compiling support | 26 | # TODO (lat9nq): cross-compiling support |
| @@ -32,14 +32,14 @@ set(NX_TZDB_ARCHIVE "${CMAKE_CURRENT_BINARY_DIR}/${NX_TZDB_VERSION}.zip") | |||
| 32 | 32 | ||
| 33 | set(NX_TZDB_ROMFS_DIR "${CMAKE_CURRENT_BINARY_DIR}/nx_tzdb") | 33 | set(NX_TZDB_ROMFS_DIR "${CMAKE_CURRENT_BINARY_DIR}/nx_tzdb") |
| 34 | 34 | ||
| 35 | if ((NOT ${CAN_BUILD_NX_TZDB} OR ${YUZU_DOWNLOAD_TIME_ZONE_DATA}) AND NOT EXISTS ${NX_TZDB_ARCHIVE}) | 35 | if ((NOT CAN_BUILD_NX_TZDB OR YUZU_DOWNLOAD_TIME_ZONE_DATA) AND NOT EXISTS ${NX_TZDB_ARCHIVE}) |
| 36 | set(NX_TZDB_DOWNLOAD_URL "https://github.com/lat9nq/tzdb_to_nx/releases/download/${NX_TZDB_VERSION}/${NX_TZDB_VERSION}.zip") | 36 | set(NX_TZDB_DOWNLOAD_URL "https://github.com/lat9nq/tzdb_to_nx/releases/download/${NX_TZDB_VERSION}/${NX_TZDB_VERSION}.zip") |
| 37 | 37 | ||
| 38 | message(STATUS "Downloading time zone data from ${NX_TZDB_DOWNLOAD_URL}...") | 38 | message(STATUS "Downloading time zone data from ${NX_TZDB_DOWNLOAD_URL}...") |
| 39 | file(DOWNLOAD ${NX_TZDB_DOWNLOAD_URL} ${NX_TZDB_ARCHIVE} | 39 | file(DOWNLOAD ${NX_TZDB_DOWNLOAD_URL} ${NX_TZDB_ARCHIVE} |
| 40 | STATUS NX_TZDB_DOWNLOAD_STATUS) | 40 | STATUS NX_TZDB_DOWNLOAD_STATUS) |
| 41 | list(GET NX_TZDB_DOWNLOAD_STATUS 0 NX_TZDB_DOWNLOAD_STATUS_CODE) | 41 | list(GET NX_TZDB_DOWNLOAD_STATUS 0 NX_TZDB_DOWNLOAD_STATUS_CODE) |
| 42 | if (NOT "${NX_TZDB_DOWNLOAD_STATUS_CODE}" STREQUAL "0") | 42 | if (NOT NX_TZDB_DOWNLOAD_STATUS_CODE EQUAL 0) |
| 43 | message(FATAL_ERROR "Time zone data download failed (status code ${NX_TZDB_DOWNLOAD_STATUS_CODE})") | 43 | message(FATAL_ERROR "Time zone data download failed (status code ${NX_TZDB_DOWNLOAD_STATUS_CODE})") |
| 44 | endif() | 44 | endif() |
| 45 | 45 | ||
| @@ -48,7 +48,7 @@ if ((NOT ${CAN_BUILD_NX_TZDB} OR ${YUZU_DOWNLOAD_TIME_ZONE_DATA}) AND NOT EXISTS | |||
| 48 | ${NX_TZDB_ARCHIVE} | 48 | ${NX_TZDB_ARCHIVE} |
| 49 | DESTINATION | 49 | DESTINATION |
| 50 | ${NX_TZDB_ROMFS_DIR}) | 50 | ${NX_TZDB_ROMFS_DIR}) |
| 51 | elseif (${CAN_BUILD_NX_TZDB} AND NOT ${YUZU_DOWNLOAD_TIME_ZONE_DATA}) | 51 | elseif (CAN_BUILD_NX_TZDB AND NOT YUZU_DOWNLOAD_TIME_ZONE_DATA) |
| 52 | add_subdirectory(tzdb_to_nx) | 52 | add_subdirectory(tzdb_to_nx) |
| 53 | add_dependencies(nx_tzdb x80e) | 53 | add_dependencies(nx_tzdb x80e) |
| 54 | 54 | ||
diff --git a/externals/nx_tzdb/NxTzdbCreateHeader.cmake b/externals/nx_tzdb/NxTzdbCreateHeader.cmake index 69166aa5b..8c29e1167 100644 --- a/externals/nx_tzdb/NxTzdbCreateHeader.cmake +++ b/externals/nx_tzdb/NxTzdbCreateHeader.cmake | |||
| @@ -15,7 +15,7 @@ set(DIRECTORY_NAME ${HEADER_NAME}) | |||
| 15 | 15 | ||
| 16 | set(FILE_DATA "") | 16 | set(FILE_DATA "") |
| 17 | foreach(ZONE_FILE ${FILE_LIST}) | 17 | foreach(ZONE_FILE ${FILE_LIST}) |
| 18 | if ("${ZONE_FILE}" STREQUAL "\n") | 18 | if (ZONE_FILE STREQUAL "\n") |
| 19 | continue() | 19 | continue() |
| 20 | endif() | 20 | endif() |
| 21 | 21 | ||
| @@ -26,13 +26,13 @@ foreach(ZONE_FILE ${FILE_LIST}) | |||
| 26 | foreach(I RANGE 0 ${ZONE_DATA_LEN} 2) | 26 | foreach(I RANGE 0 ${ZONE_DATA_LEN} 2) |
| 27 | math(EXPR BREAK_LINE "(${I} + 2) % 38") | 27 | math(EXPR BREAK_LINE "(${I} + 2) % 38") |
| 28 | 28 | ||
| 29 | string(SUBSTRING "${ZONE_DATA}" "${I}" "2" HEX_DATA) | 29 | string(SUBSTRING "${ZONE_DATA}" "${I}" 2 HEX_DATA) |
| 30 | if ("${HEX_DATA}" STREQUAL "") | 30 | if (NOT HEX_DATA) |
| 31 | break() | 31 | break() |
| 32 | endif() | 32 | endif() |
| 33 | 33 | ||
| 34 | string(APPEND FILE_DATA "0x${HEX_DATA},") | 34 | string(APPEND FILE_DATA "0x${HEX_DATA},") |
| 35 | if ("${BREAK_LINE}" STREQUAL "0") | 35 | if (BREAK_LINE EQUAL 0) |
| 36 | string(APPEND FILE_DATA "\n") | 36 | string(APPEND FILE_DATA "\n") |
| 37 | else() | 37 | else() |
| 38 | string(APPEND FILE_DATA " ") | 38 | string(APPEND FILE_DATA " ") |