Merge branch 'main' into settings

.github/workflows/build.yml

@@ -154,23 +154,20 @@ jobs:
         has-sccache: ${{ inputs.use-ccache }}
       if: inputs.build-debug-info
 
-    - name: Setup caches
+    # due to very strange performance issues, sccache is currently disabled for macos. sorry
-      uses: ./.github/actions/setup-cache
+
-      with:
+    # - name: Setup caches
-        host: mac
+    #   uses: ./.github/actions/setup-cache
-        target: mac
+    #   with:
-        use-ccache: ${{ github.event_name != 'workflow_dispatch' || inputs.use-ccache }}
+    #     host: mac
+    #     target: mac
+    #     use-ccache: ${{ github.event_name != 'workflow_dispatch' || inputs.use-ccache }}
 
     - name: Setup Ninja
       uses: ./.github/actions/setup-ninja
       with:
         host: mac
 
-    - name: Install LLVM
-      run: |
-        brew install llvm
-        echo "/opt/homebrew/opt/llvm/bin" >> $GITHUB_PATH
-
     - name: Setup CLI
       uses: geode-sdk/cli/.github/actions/setup@main
 
@@ -180,9 +177,8 @@ jobs:
         -DCMAKE_C_COMPILER=clang
         -DCMAKE_CXX_COMPILER=clang++
         -DCMAKE_OSX_DEPLOYMENT_TARGET=10.15
-        -DCMAKE_AR='/usr/bin/ar'
+        -DCMAKE_OSX_ARCHITECTURES="arm64;x86_64"
-        -DCMAKE_RANLIB='/usr/bin/ranlib'
+        -DGEODE_DISABLE_PRECOMPILED_HEADERS=OFF
-        -DGEODE_DISABLE_PRECOMPILED_HEADERS=Off
         ${{ steps.build-debug-info.outputs.extra-configure }}
 
     - name: Build

CMakeLists.txt

@@ -133,18 +133,29 @@ endif()
 if (DEFINED GEODE_CCACHE_VARIANT)
 	if (NOT DEFINED GEODE_DISABLE_PRECOMPILED_HEADERS)
 		if (${GEODE_CCACHE_VARIANT} STREQUAL "sccache" AND CMAKE_CXX_COMPILER_ID STREQUAL "Clang")
+			if (DEFINED CMAKE_OSX_ARCHITECTURES AND (CMAKE_OSX_ARCHITECTURES STREQUAL "arm64;x86_64" OR CMAKE_OSX_ARCHITECTURES STREQUAL "x86_64;arm64"))
+				message(NOTICE "Using ${GEODE_CCACHE_VARIANT} with ${CMAKE_CXX_COMPILER_ID} while building multiple architectures, PCH will be disabled due to issues with sccache.")
+				set(GEODE_DISABLE_PRECOMPILED_HEADERS ON)
+			else()
 				message(NOTICE "Using ${GEODE_CCACHE_VARIANT} with ${CMAKE_CXX_COMPILER_ID}, PCH will be enabled.")
 				set(GEODE_DISABLE_PRECOMPILED_HEADERS OFF)
+			endif()
 		else()
 			message(NOTICE "Using ${GEODE_CCACHE_VARIANT} with ${CMAKE_CXX_COMPILER_ID}, PCH will be disabled.")
 			set(GEODE_DISABLE_PRECOMPILED_HEADERS ON)
 		endif()
 	endif()
+else()
+	if (${CMAKE_SYSTEM_NAME} STREQUAL "Darwin")
+		# when building for multiple architectures, a caching compiler is not recommended
+		message(NOTICE "Not using a caching compiler (ccache/sccache).")
 	else()
 		message(NOTICE "Not using a caching compiler (ccache/sccache). "
 			"It is recommended to install one to improve build times.")
 		message(NOTICE "We recommend sccache, check its README for installation instructions, "
 			"normally you can just use your usual package manager (e.g. 'scoop install sccache').")
+	endif()
+
 	if (NOT DEFINED GEODE_DISABLE_PRECOMPILED_HEADERS)
 		message(NOTICE "Because of this, PCH will be enabled.")
 		set(GEODE_DISABLE_PRECOMPILED_HEADERS OFF)

loader/src/server/DownloadManager.cpp

@@ -156,6 +156,7 @@ public:
         });
 
         auto req = web::WebRequest();
+        req.userAgent(getServerUserAgent());
         m_downloadListener.setFilter(req.get(version.downloadURL));
         ModDownloadEvent(m_id).post();
     }

loader/src/ui/mods/popups/FiltersPopup.cpp

@@ -103,6 +103,7 @@ bool FiltersPopup::setup(ModListSource* src) {
 
         m_developerNameInput = TextInput::create(inputContainer->getContentWidth(), "Developer Name");
         m_developerNameInput->setTextAlign(TextInputAlign::Left);
+        m_developerNameInput->setFilter("abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789-");
         m_developerNameInput->setString(src->getQuery().developer.value_or(""));
         inputContainer->addChildAtPosition(m_developerNameInput, Anchor::Center);