[Protocol] Remove duplicate event_game_state_changed.proto entry (#7234 )

The .proto file appeared twice in the PROTO_FILES list, causing protoc to process it twice on every build. Keep a single entry. Co-authored-by: Lukas Brübach <Bruebach.Lukas@bdosecurity.de>
[Build] Use pipes for GCC/Clang compilation (#7233 )
2026-09-23 18:06:26 -07:00 · 2026-09-04 05:20:53 +02:00 · 2026-09-04 05:20:53 +02:00 · 2026-09-04 05:20:52 +02:00
2 changed files with 8 additions and 3 deletions
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@ -5,8 +5,8 @@
 # This file sets all the variables shared between the projects
 # like the installation path, compilation flags etc..

-# cmake 3.16 is required if using qt6
-cmake_minimum_required(VERSION 3.10)
+# 3.16 required for Qt6 and target_precompile_headers()
+cmake_minimum_required(VERSION 3.16)

 # Use compiler cache (ccache)
 option(USE_CCACHE "Cache the build results with ccache" OFF)
@ -184,6 +184,9 @@ elseif(CMAKE_COMPILER_IS_GNUCXX)
      set(CMAKE_CXX_FLAGS_DEBUG "${CMAKE_CXX_FLAGS_DEBUG} ${FLAG}")
    endif()
  endforeach()
+
+  # Reduce compiler I/O by using pipes between stages instead of temp files
+  set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -pipe")
 else()
  # other: osx/llvm, bsd/llvm
  set(CMAKE_CXX_FLAGS_RELEASE "-O2")
@ -192,6 +195,9 @@ else()
  else()
    set(CMAKE_CXX_FLAGS_DEBUG "-g -O0 -Wall -Wextra")
  endif()
+
+  # Reduce compiler I/O by using pipes between stages instead of temp files
+  set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -pipe")
 endif()

 # GNU systems need to define the Mersenne exponent for the RNG to compile w/o warning
--- a/libcockatrice_protocol/libcockatrice/protocol/pb/CMakeLists.txt
+++ b/libcockatrice_protocol/libcockatrice/protocol/pb/CMakeLists.txt
@ -90,7 +90,6 @@ set(PROTO_FILES
    event_game_log_notice.proto
    event_game_say.proto
    event_game_state_changed.proto
-    event_game_state_changed.proto
    event_join.proto
    event_join_room.proto
    event_kicked.proto
Author	SHA1	Message	Date
BruebachL	3ec62df3e7	[Protocol] Remove duplicate event_game_state_changed.proto entry (#7234 ) Some checks are pending CodeQL / Analyze (cpp) (push) Waiting to run Details CodeQL / Analyze (actions) (push) Waiting to run Details Build Desktop / Configure (push) Waiting to run Details Build Desktop / Debian 13 (push) Blocked by required conditions Details Build Desktop / Debian 12 (push) Blocked by required conditions Details Build Desktop / Fedora 44 (push) Blocked by required conditions Details Build Desktop / Fedora 43 (push) Blocked by required conditions Details Build Desktop / Servatrice_Debian 12 (push) Blocked by required conditions Details Build Desktop / Ubuntu 26.04 (push) Blocked by required conditions Details Build Desktop / Ubuntu 24.04 (push) Blocked by required conditions Details Build Desktop / Arch (push) Blocked by required conditions Details Build Desktop / macOS 13 Intel (push) Blocked by required conditions Details Build Desktop / macOS 14 (push) Blocked by required conditions Details Build Desktop / macOS 15 (push) Blocked by required conditions Details Build Desktop / macOS 26 Debug (push) Blocked by required conditions Details Build Desktop / Windows 10 (push) Blocked by required conditions Details Build Docker / Servatrice (arm) (push) Waiting to run Details Build Docker / Servatrice (x86) (push) Waiting to run Details Build Docker / Publish multi-platform Servatrice image (push) Blocked by required conditions Details The .proto file appeared twice in the PROTO_FILES list, causing protoc to process it twice on every build. Keep a single entry. Co-authored-by: Lukas Brübach <Bruebach.Lukas@bdosecurity.de>	2026-09-04 05:20:53 +02:00
BruebachL	fcfb14cf56	[Build] Use pipes for GCC/Clang compilation (#7233 ) Pass -pipe so GCC/Clang transfer intermediate representation between compiler stages over pipes instead of temporary files, reducing build I/O. Co-authored-by: Lukas Brübach <Bruebach.Lukas@bdosecurity.de>	2026-09-04 05:20:53 +02:00
BruebachL	35ebae8d7f	[Build] Bump cmake_minimum_required from 3.10 to 3.16 (#7232 ) 3.16 is already required by Qt6 (and enforced at find_package time). This unlocks native target_precompile_headers(), better AUTOMOC/AUTORCC handling, and qt6_finalize_project() without a version guard. Co-authored-by: Lukas Brübach <Bruebach.Lukas@bdosecurity.de>	2026-09-04 05:20:52 +02:00