chore: fix platform specific build options

pnck · May 2, 2024 · 69da87c · 69da87c
1 parent 643df72
commit 69da87c
Show file tree

Hide file tree

Showing 7 changed files with 30 additions and 26 deletions.
diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
@@ -81,12 +81,14 @@ jobs:
  - name: Build Tests
  working-directory: ${{ github.workspace }}
  run: |
- xcodebuild clean build -workspace foo_input_ncm.xcworkspace -scheme foo_input_ncm_tests -configuration ${{ matrix.config }} -derivedDataPath .
+ xcodebuild -project test/unit/macos/foo_input_ncm_tests.xcodeproj -scheme gtest -configuration ${{ matrix.config }} -derivedDataPath .
+ xcodebuild -project test/unit/macos/foo_input_ncm_tests.xcodeproj -scheme gtest_main -configuration ${{ matrix.config }} -derivedDataPath .
+ xcodebuild clean build -workspace foo_input_ncm.xcworkspace -scheme foo_input_ncm_tests -configuration ${{ matrix.config }} -derivedDataPath .
 
  - name: Run Tests
  working-directory: ${{ github.workspace }}/Build/Products/${{ matrix.config }}
  run: |
- ./foo_input_ncm_tests  --gtest_repeat=10 --gtest_shuffle
+ ./foo_input_ncm_tests --gtest_repeat=10 --gtest_shuffle
 
  matrix-build:
  name: Matrix Build
@@ -120,7 +122,7 @@ jobs:
 
  - name: Versioning
  run: |
- TAG=$(git describe --tags)
+ TAG=$(git describe git describe --long --all)
  VERSION=${TAG:1}
  echo "#define CURRENT_VERSION $VERSION" >> src/stdafx.h
  echo "----------------------------"
@@ -150,7 +152,7 @@ jobs:
  env:
  scheme: foo_input_ncm-${{ matrix.config }}
  run: |
- xcodebuild clean build -workspace ${{ matrix.workspace }} -scheme "$scheme" -configuration ${{ matrix.config }}
+ xcodebuild clean build -workspace ${{ matrix.workspace }} -scheme "$scheme" -configuration ${{ matrix.config }} -sdk macosx -mmacosx-version-min=13.3
  if: matrix.os == 'macos-14'
 
  - name: (MSBuild) Setup MSBuild

diff --git a/foo_input_ncm.vcxproj b/foo_input_ncm.vcxproj
@@ -90,7 +90,7 @@
  <ConformanceMode>true</ConformanceMode>
  <LanguageStandard>stdcpp20</LanguageStandard>
  <LanguageStandard_C>stdc11</LanguageStandard_C>
- <AdditionalIncludeDirectories>src; vendor; vendor/sdk</AdditionalIncludeDirectories>
+ <AdditionalIncludeDirectories>src;vendor/spdlog/include;vendor/rapidjson/include;vendor/sdk;vendor/sdk/foobar2000</AdditionalIncludeDirectories>
  <AdditionalOptions>/Zc:__cplusplus %(AdditionalOptions)</AdditionalOptions>
  <PrecompiledHeader>Use</PrecompiledHeader>
  </ClCompile>
@@ -116,7 +116,7 @@
  <ConformanceMode>true</ConformanceMode>
  <LanguageStandard>stdcpp20</LanguageStandard>
  <LanguageStandard_C>stdc11</LanguageStandard_C>
- <AdditionalIncludeDirectories>src; vendor; vendor/sdk</AdditionalIncludeDirectories>
+ <AdditionalIncludeDirectories>src;vendor/spdlog/include;vendor/rapidjson/include;vendor/sdk;vendor/sdk/foobar2000</AdditionalIncludeDirectories>
  <AdditionalOptions>/Zc:__cplusplus %(AdditionalOptions)</AdditionalOptions>
  <PrecompiledHeader>Use</PrecompiledHeader>
  </ClCompile>
@@ -140,7 +140,7 @@
  <ConformanceMode>true</ConformanceMode>
  <LanguageStandard>stdcpp20</LanguageStandard>
  <LanguageStandard_C>stdc11</LanguageStandard_C>
- <AdditionalIncludeDirectories>src;vendor/spdlog/include;vendor/rapidjson/include;vendor/sdk</AdditionalIncludeDirectories>
+ <AdditionalIncludeDirectories>src;vendor/spdlog/include;vendor/rapidjson/include;vendor/sdk;vendor/sdk/foobar2000</AdditionalIncludeDirectories>
  <AdditionalOptions>/Zc:__cplusplus %(AdditionalOptions)</AdditionalOptions>
  <PrecompiledHeader>Use</PrecompiledHeader>
  </ClCompile>
@@ -168,7 +168,7 @@ copy $(TargetPath) $(ProjectDir)dist\$(Platform)\</Command>
  <ConformanceMode>true</ConformanceMode>
  <LanguageStandard>stdcpp20</LanguageStandard>
  <LanguageStandard_C>stdc11</LanguageStandard_C>
- <AdditionalIncludeDirectories>src; vendor; vendor/sdk</AdditionalIncludeDirectories>
+ <AdditionalIncludeDirectories>src;vendor/spdlog/include;vendor/rapidjson/include;vendor/sdk;vendor/sdk/foobar2000</AdditionalIncludeDirectories>
  <AdditionalOptions>/Zc:__cplusplus %(AdditionalOptions)</AdditionalOptions>
  <PrecompiledHeader>Use</PrecompiledHeader>
  </ClCompile>

diff --git a/src/cipher/exception.hpp b/src/cipher/exception.hpp
@@ -14,7 +14,7 @@ namespace fb2k_ncm::cipher
  explicit cipher_error(std::string reason, STATUS status) : std::runtime_error(""), status_(status), reason_(std::move(reason)) {
  last_error_ = GetLastError();
  std::stringstream err_ss;
- msg_ = fmt::format("cipher runtime error because of [{}] \n Last Error: 0x{:08X}\n STATUS: 0x{:08X}",
+ msg_ = fmtlib::format("cipher runtime error because of [{}] \n Last Error: 0x{:08X}\n STATUS: 0x{:08X}",
  reason_,
  last_error_,
  static_cast<uint32_t>(status_));

diff --git a/src/common/platform.hpp b/src/common/platform.hpp
@@ -23,10 +23,18 @@ typedef DWORD DWORD;
 #define KEYSIZE_ERROR STATUS_INVALID_PARAMETER
 #define COMMON_ERROR STATUS_UNSUCCESSFUL
 
+#include <format>
+namespace fmtlib = std;
+
 #elif defined __APPLE__
 #include <CommonCrypto/CommonCryptoError.h>
 #include <cerrno>
 
+// since we are using spdlog, we can use fmt::format instead of std::format,
+// which is still unavailable (as of libc++15, within macos-14 github action runner)
+#include "spdlog/fmt/fmt.h"
+namespace fmtlib = fmt;
+
 #if 0
 namespace std
 {
@@ -48,7 +56,3 @@ inline auto GetLastError() {
 #define COMMON_ERROR kCCUnspecifiedError
 
 #endif
-
-// since we are using spdlog, we can use fmt::format instead of std::format,
-// which is still unavailable (as of libc++15, within macos-14 github action runner)
-#include "spdlog/fmt/bundled/format.h"
diff --git a/src/ncm_file.cpp b/src/ncm_file.cpp
@@ -148,7 +148,7 @@ void ncm_file::parse(uint16_t to_parse /* = 0xff*/) {
  uint64_t magic = 0;
  source_->read(&magic, sizeof(uint64_t), p_abort);
  if (magic != parsed_file_.magic) [[unlikely]] {
- throw_format_error(fmt::format("magic number mismatch: {}", magic));
+ throw_format_error(fmtlib::format("magic number mismatch: {}", magic));
  }
 
  // skip gap

diff --git a/test/unit/macos/foo_input_ncm_tests.xcodeproj/project.pbxproj b/test/unit/macos/foo_input_ncm_tests.xcodeproj/project.pbxproj
@@ -8,10 +8,10 @@
 
 /* Begin PBXBuildFile section */
  A35F94992BE17E7100ABAABA /* test_crypto_functionality.cpp in Sources */ = {isa = PBXBuildFile; fileRef = A35F94972BE17E7100ABAABA /* test_crypto_functionality.cpp */; };
- A35F94BA2BE17F0B00ABAABA /* libgtest_main.a in Frameworks */ = {isa = PBXBuildFile; fileRef = A35F94B82BE17F0300ABAABA /* libgtest_main.a */; };
- A35F94BB2BE17F0B00ABAABA /* libgtest.a in Frameworks */ = {isa = PBXBuildFile; fileRef = A35F94B62BE17F0300ABAABA /* libgtest.a */; };
  A35F94C32BE1813800ABAABA /* aes_common.cpp in Sources */ = {isa = PBXBuildFile; fileRef = A35F94C12BE1813800ABAABA /* aes_common.cpp */; };
  A35F94C42BE1813800ABAABA /* aes_macos.cpp in Sources */ = {isa = PBXBuildFile; fileRef = A35F94C22BE1813800ABAABA /* aes_macos.cpp */; };
+ A35F94CE2BE31FBE00ABAABA /* libgtest_main.a in Frameworks */ = {isa = PBXBuildFile; fileRef = A35F94B82BE17F0300ABAABA /* libgtest_main.a */; };
+ A35F94CF2BE31FBE00ABAABA /* libgtest.a in Frameworks */ = {isa = PBXBuildFile; fileRef = A35F94B62BE17F0300ABAABA /* libgtest.a */; };
 /* End PBXBuildFile section */
 
 /* Begin PBXContainerItemProxy section */
@@ -57,8 +57,8 @@
  isa = PBXFrameworksBuildPhase;
  buildActionMask = 2147483647;
  files = (
- A35F94BA2BE17F0B00ABAABA /* libgtest_main.a in Frameworks */,
- A35F94BB2BE17F0B00ABAABA /* libgtest.a in Frameworks */,
+ A35F94CE2BE31FBE00ABAABA /* libgtest_main.a in Frameworks */,
+ A35F94CF2BE31FBE00ABAABA /* libgtest.a in Frameworks */,
  );
  runOnlyForDeploymentPostprocessing = 0;
  };
@@ -244,7 +244,6 @@
  GCC_WARN_UNUSED_FUNCTION = YES;
  GCC_WARN_UNUSED_VARIABLE = YES;
  LOCALIZATION_PREFERS_STRING_CATALOGS = YES;
- MACOSX_DEPLOYMENT_TARGET = 13.3;
  MTL_ENABLE_DEBUG_INFO = INCLUDE_SOURCE;
  MTL_FAST_MATH = YES;
  ONLY_ACTIVE_ARCH = YES;
@@ -299,7 +298,6 @@
  GCC_WARN_UNUSED_FUNCTION = YES;
  GCC_WARN_UNUSED_VARIABLE = YES;
  LOCALIZATION_PREFERS_STRING_CATALOGS = YES;
- MACOSX_DEPLOYMENT_TARGET = 13.3;
  MTL_ENABLE_DEBUG_INFO = NO;
  MTL_FAST_MATH = YES;
  SDKROOT = macosx;

diff --git a/test/unit/win32/foo_input_ncm_tests.vcxproj b/test/unit/win32/foo_input_ncm_tests.vcxproj
@@ -83,7 +83,7 @@
  <BasicRuntimeChecks>EnableFastChecks</BasicRuntimeChecks>
  <RuntimeLibrary>MultiThreadedDebugDLL</RuntimeLibrary>
  <WarningLevel>Level3</WarningLevel>
- <AdditionalIncludeDirectories>$(SolutionDir)src;%(AdditionalIncludeDirectories)</AdditionalIncludeDirectories>
+ <AdditionalIncludeDirectories>$(SolutionDir)src;$(SolutionDir)vendor/spdlog/include;$(SolutionDir)vendor/rapidjson/include;$(SolutionDir)vendor/sdk;$(SolutionDir)vendor/sdk/foobar2000;%(AdditionalIncludeDirectories)</AdditionalIncludeDirectories>
  <LanguageStandard>stdcpp20</LanguageStandard>
  <AdditionalOptions>/Zc:__cplusplus %(AdditionalOptions)</AdditionalOptions>
  <DebugInformationFormat>ProgramDatabase</DebugInformationFormat>
@@ -106,7 +106,7 @@
  <BasicRuntimeChecks>EnableFastChecks</BasicRuntimeChecks>
  <RuntimeLibrary>MultiThreadedDebugDLL</RuntimeLibrary>
  <WarningLevel>Level3</WarningLevel>
- <AdditionalIncludeDirectories>$(SolutionDir)src;%(AdditionalIncludeDirectories)</AdditionalIncludeDirectories>
+ <AdditionalIncludeDirectories>$(SolutionDir)src;$(SolutionDir)vendor/spdlog/include;$(SolutionDir)vendor/rapidjson/include;$(SolutionDir)vendor/sdk;$(SolutionDir)vendor/sdk/foobar2000;%(AdditionalIncludeDirectories)</AdditionalIncludeDirectories>
  <LanguageStandard>stdcpp20</LanguageStandard>
  <AdditionalOptions>/Zc:__cplusplus %(AdditionalOptions)</AdditionalOptions>
  <DebugInformationFormat>ProgramDatabase</DebugInformationFormat>
@@ -129,7 +129,7 @@
  <BasicRuntimeChecks>EnableFastChecks</BasicRuntimeChecks>
  <RuntimeLibrary>MultiThreadedDebugDLL</RuntimeLibrary>
  <WarningLevel>Level3</WarningLevel>
- <AdditionalIncludeDirectories>$(SolutionDir)src;%(AdditionalIncludeDirectories)</AdditionalIncludeDirectories>
+ <AdditionalIncludeDirectories>$(SolutionDir)src;$(SolutionDir)vendor/spdlog/include;$(SolutionDir)vendor/rapidjson/include;$(SolutionDir)vendor/sdk;$(SolutionDir)vendor/sdk/foobar2000;%(AdditionalIncludeDirectories)</AdditionalIncludeDirectories>
  <LanguageStandard>stdcpp20</LanguageStandard>
  <AdditionalOptions>/Zc:__cplusplus %(AdditionalOptions)</AdditionalOptions>
  <DebugInformationFormat>ProgramDatabase</DebugInformationFormat>
@@ -151,7 +151,7 @@
  <RuntimeLibrary>MultiThreadedDLL</RuntimeLibrary>
  <WarningLevel>Level3</WarningLevel>
  <DebugInformationFormat>ProgramDatabase</DebugInformationFormat>
- <AdditionalIncludeDirectories>$(SolutionDir)src;%(AdditionalIncludeDirectories)</AdditionalIncludeDirectories>
+ <AdditionalIncludeDirectories>$(SolutionDir)src;$(SolutionDir)vendor/spdlog/include;$(SolutionDir)vendor/rapidjson/include;$(SolutionDir)vendor/sdk;$(SolutionDir)vendor/sdk/foobar2000;%(AdditionalIncludeDirectories)</AdditionalIncludeDirectories>
  <LanguageStandard>stdcpp20</LanguageStandard>
  <AdditionalOptions>/Zc:__cplusplus %(AdditionalOptions)</AdditionalOptions>
  <LanguageStandard_C>stdc11</LanguageStandard_C>
@@ -174,7 +174,7 @@
  <RuntimeLibrary>MultiThreadedDLL</RuntimeLibrary>
  <WarningLevel>Level3</WarningLevel>
  <DebugInformationFormat>ProgramDatabase</DebugInformationFormat>
- <AdditionalIncludeDirectories>$(SolutionDir)src;%(AdditionalIncludeDirectories)</AdditionalIncludeDirectories>
+ <AdditionalIncludeDirectories>$(SolutionDir)src;$(SolutionDir)vendor/spdlog/include;$(SolutionDir)vendor/rapidjson/include;$(SolutionDir)vendor/sdk;$(SolutionDir)vendor/sdk/foobar2000;%(AdditionalIncludeDirectories)</AdditionalIncludeDirectories>
  <LanguageStandard>stdcpp20</LanguageStandard>
  <AdditionalOptions>/Zc:__cplusplus %(AdditionalOptions)</AdditionalOptions>
  <LanguageStandard_C>stdc11</LanguageStandard_C>
@@ -197,7 +197,7 @@
  <RuntimeLibrary>MultiThreadedDLL</RuntimeLibrary>
  <WarningLevel>Level3</WarningLevel>
  <DebugInformationFormat>ProgramDatabase</DebugInformationFormat>
- <AdditionalIncludeDirectories>$(SolutionDir)src;%(AdditionalIncludeDirectories)</AdditionalIncludeDirectories>
+ <AdditionalIncludeDirectories>$(SolutionDir)src;$(SolutionDir)vendor/spdlog/include;$(SolutionDir)vendor/rapidjson/include;$(SolutionDir)vendor/sdk;$(SolutionDir)vendor/sdk/foobar2000;%(AdditionalIncludeDirectories)</AdditionalIncludeDirectories>
  <LanguageStandard>stdcpp20</LanguageStandard>
  <AdditionalOptions>/Zc:__cplusplus %(AdditionalOptions)</AdditionalOptions>
  <LanguageStandard_C>stdc11</LanguageStandard_C>