Skip to content

Commit adcb69e

Browse files
leoparenteclaude
andcommitted
fix: set cppstd=17 in default Conan profile for Windows builds
conan_provider.cmake passes --profile:build=default to conan install. The auto-detected default on MSVC always sets compiler.cppstd=14, which fails validation for protobuf/6.33.5 (requires C++17) when used as a build/tool requirement. Pre-creating the default profile with cppstd=17 before CMake runs ensures both the host and build profiles agree on the C++ standard. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent dffb8bb commit adcb69e

2 files changed

Lines changed: 12 additions & 0 deletions

File tree

.github/workflows/build-develop.yml

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -121,6 +121,12 @@ jobs:
121121
id: conan
122122
uses: turtlebrowser/get-conan@c171f295f3f507360ee018736a6608731aa2109d #v1.2
123123

124+
- name: Setup Conan Profile
125+
shell: bash
126+
run: |
127+
conan profile detect --force
128+
sed -i 's/compiler.cppstd=14/compiler.cppstd=17/' "$(conan profile path default)"
129+
124130
- name: Setup Conan Cache
125131
uses: actions/cache@v4
126132
with:

.github/workflows/build-release.yml

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -168,6 +168,12 @@ jobs:
168168
id: conan
169169
uses: turtlebrowser/get-conan@c171f295f3f507360ee018736a6608731aa2109d #v1.2
170170

171+
- name: Setup Conan Profile
172+
shell: bash
173+
run: |
174+
conan profile detect --force
175+
sed -i 's/compiler.cppstd=14/compiler.cppstd=17/' "$(conan profile path default)"
176+
171177
- name: Setup Conan Cache
172178
uses: actions/cache@v4
173179
with:

0 commit comments

Comments
 (0)