-
Notifications
You must be signed in to change notification settings - Fork 15.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
CMake files use string(JOIN)
while only requesting CMake >= 3.5
#12672
Labels
Comments
fowles
added
packaging & distribution
and removed
untriaged
auto added to all issues by default when created.
labels
May 4, 2023
Somewhere around here: Line 9 in dbb9f9b
One needs to write a loop
|
copybara-service bot
pushed a commit
that referenced
this issue
May 5, 2023
This also downgrades our tests to use cmake 3.10, compatible with our support window. Closes #12672 PiperOrigin-RevId: 529751568
copybara-service bot
pushed a commit
that referenced
this issue
May 5, 2023
This also downgrades our tests to use cmake 3.10, compatible with our support window. Closes #12672 PiperOrigin-RevId: 529751568
copybara-service bot
pushed a commit
that referenced
this issue
May 5, 2023
This also downgrades our tests to use cmake 3.10, compatible with our support window. Closes #12672 PiperOrigin-RevId: 529751568
copybara-service bot
pushed a commit
that referenced
this issue
May 5, 2023
This also downgrades our tests to use cmake 3.10, compatible with our support window. Closes #12672 PiperOrigin-RevId: 529751568
copybara-service bot
pushed a commit
that referenced
this issue
May 5, 2023
This also downgrades our tests to use cmake 3.10, compatible with our support window. Closes #12672 PiperOrigin-RevId: 529751568
copybara-service bot
pushed a commit
that referenced
this issue
May 5, 2023
This also downgrades our tests to use cmake 3.10, compatible with our support window. Closes #12672 PiperOrigin-RevId: 529751568
copybara-service bot
pushed a commit
that referenced
this issue
May 5, 2023
This also downgrades our tests to use cmake 3.10, compatible with our support window. Closes #12672 PiperOrigin-RevId: 529751568
mkruskal-google
added a commit
that referenced
this issue
May 6, 2023
This also downgrades our tests to use cmake 3.10, compatible with our support window. Closes #12672 PiperOrigin-RevId: 529840763
mkruskal-google
added a commit
that referenced
this issue
May 6, 2023
This also downgrades our tests to use cmake 3.10, compatible with our support window. Closes #12672 PiperOrigin-RevId: 529840763
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
What version of protobuf and what language are you using?
Version: 22.4
Language: C++
What operating system (Linux, Windows, ...) and version?
Ubuntu 18.04
What runtime / compiler are you using (e.g., python version or gcc version)
GCC 7.5
What did you do?
I used this Dockerfile:
https://github.com/coryan/google-cloud-cpp/blob/414854fb53439c3d34ccb17766312c518afa9e3d/ci/cloudbuild/dockerfiles/demo-ubuntu-bionic.Dockerfile#L66-L76
What did you expect to see
A successful build.
What did you see instead?
Anything else we should know about your project / environment
string(JOIN)
ships with CMake >= 3.12:https://cmake.org/cmake/help/latest/command/string.html#join
However Protobuf 22.4 requires CMake >= 3.5:
protobuf/CMakeLists.txt
Line 2 in dbb9f9b
You could bump the CMake requirements but notice that the overall foundations matrix still says CMake >= 3.10:
https://github.com/google/oss-policies-info/blob/6587f128170c23282eb40949c914403a4010079a/foundational-cxx-support-matrix.md?plain=1#L6
The text was updated successfully, but these errors were encountered: