Skip to content

[google-cloud-cpp] update to latest release (v2.13.0) - #32391

Merged
Javier Matos Denizac (JavierMatosD) merged 1 commit into
microsoft:masterfrom
coryan:google-cloud-cpp-update-to-v2.13.0
Jul 7, 2023
Merged

Javier Matos Denizac (JavierMatosD) merged 1 commit into
microsoft:masterfrom
coryan:google-cloud-cpp-update-to-v2.13.0

Conversation

@coryan

Copy link
Copy Markdown
Contributor

Updates google-cloud-cpp to the latest release (v2.13.0)

Tested locally (on x64-linux) with:

for feature in datafusion metastore migrationcenter networksecurity rapidmigrationassessment; do ./vcpkg remove google-cloud-cpp; ./vcpkg install "google-cloud-cpp[core,${feature}]" || break; done

and

./vcpkg remove google-cloud-cpp && ./vcpkg install 'google-cloud-cpp[*]'
  • Changes comply with the maintainer guide
  • SHA512s are updated for each updated download
  • The "supports" clause reflects platforms that may be fixed by this new version
  • Any fixed CI baseline entries are removed from that file.
  • Any patches that are no longer applied are deleted from the port's directory.
  • The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
  • Only one version is added to each modified port's versions file.

@coryan
Carlos O'Ryan (coryan) marked this pull request as ready for review July 4, 2023 16:56
@MonicaLiu0311 MonicaLiu (MonicaLiu0311) added the category:port-update The issue is with a library, which is requesting update new revision label Jul 5, 2023
@jimwang118

Copy link
Copy Markdown
Contributor

I have the following issue when testing x64-windows.

F:\google-cloud-cpp\buildtrees\google-cloud-cpp\x64-windows-dbg\google\cloud\asset\google/cloud/asset/v1/assets.pb.h(4183): error C2059: syntax error: 'constant'
F:\google-cloud-cpp\buildtrees\google-cloud-cpp\x64-windows-dbg\google\cloud\asset\google/cloud/asset/v1/assets.pb.h(4184): error C2238: unexpected token(s) preceding ';'
F:\google-cloud-cpp\buildtrees\google-cloud-cpp\x64-windows-dbg\google\cloud\asset\google/cloud/asset/v1/assets.pb.h(4185): error C2059: syntax error: 'constant'
F:\google-cloud-cpp\buildtrees\google-cloud-cpp\x64-windows-dbg\google\cloud\asset\google/cloud/asset/v1/assets.pb.h(4186): error C2238: unexpected token(s) preceding ';'

@jimwang118

Copy link
Copy Markdown
Contributor

Note: I will be converting your PR to draft status. When you respond, please revert to "ready for review". That way, I can be aware that you've responded since you can't modify the tags.

@jimwang118
jim wang (jimwang118) marked this pull request as draft July 6, 2023 08:24
@coryan

Carlos O'Ryan (coryan) commented Jul 6, 2023 •

Copy link
Copy Markdown
Contributor Author

I have the following issue when testing x64-windows.

F:\google-cloud-cpp\buildtrees\google-cloud-cpp\x64-windows-dbg\google\cloud\asset\google/cloud/asset/v1/assets.pb.h(4183): error C2059: syntax error: 'constant'
F:\google-cloud-cpp\buildtrees\google-cloud-cpp\x64-windows-dbg\google\cloud\asset\google/cloud/asset/v1/assets.pb.h(4184): error C2238: unexpected token(s) preceding ';'
F:\google-cloud-cpp\buildtrees\google-cloud-cpp\x64-windows-dbg\google\cloud\asset\google/cloud/asset/v1/assets.pb.h(4185): error C2059: syntax error: 'constant'
F:\google-cloud-cpp\buildtrees\google-cloud-cpp\x64-windows-dbg\google\cloud\asset\google/cloud/asset/v1/assets.pb.h(4186): error C2238: unexpected token(s) preceding ';'

asset is already marked as not supported on Windows:

"asset": {
"description": "Cloud Asset API C++ Client Library",
"supports": "!windows",
"dependencies": [
{
"name": "google-cloud-cpp",
"default-features": false,
"features": [
"accesscontextmanager",
"grpc-common",
"osconfig"
]
}
]
},

This is not a new problem, the previous version would not compile on Windows either. Should I fix something with the feature annotations?

FWIW, the underlying problem is in Protobuf, and fixed in Protobuf >= 4.23.x (the generated code uses a symbol that is a macro on Windows). I am planning to mark the library as supported when vcpkg gets a recent enough version of Protobuf.

@coryan
Carlos O'Ryan (coryan) marked this pull request as ready for review July 6, 2023 12:09
@jimwang118

Copy link
Copy Markdown
Contributor

I have the following issue when testing x64-windows.

F:\google-cloud-cpp\buildtrees\google-cloud-cpp\x64-windows-dbg\google\cloud\asset\google/cloud/asset/v1/assets.pb.h(4183): error C2059: syntax error: 'constant'
F:\google-cloud-cpp\buildtrees\google-cloud-cpp\x64-windows-dbg\google\cloud\asset\google/cloud/asset/v1/assets.pb.h(4184): error C2238: unexpected token(s) preceding ';'
F:\google-cloud-cpp\buildtrees\google-cloud-cpp\x64-windows-dbg\google\cloud\asset\google/cloud/asset/v1/assets.pb.h(4185): error C2059: syntax error: 'constant'
F:\google-cloud-cpp\buildtrees\google-cloud-cpp\x64-windows-dbg\google\cloud\asset\google/cloud/asset/v1/assets.pb.h(4186): error C2238: unexpected token(s) preceding ';'

asset is already marked as not supported on Windows:

"asset": {
"description": "Cloud Asset API C++ Client Library",
"supports": "!windows",
"dependencies": [
{
"name": "google-cloud-cpp",
"default-features": false,
"features": [
"accesscontextmanager",
"grpc-common",
"osconfig"
]
}
]
},

This is not a new problem, the previous version would not compile on Windows either. Should I fix something with the feature annotations?

FWIW, the underlying problem is in Protobuf, and fixed in Protobuf >= 4.23.x (the generated code uses a symbol that is a macro on Windows). I am planning to mark the library as supported when vcpkg gets a recent enough version of Protobuf.

Ok, then I have no problem.

@jimwang118 jim wang (jimwang118) added the info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this. label Jul 7, 2023
@JavierMatosD
Javier Matos Denizac (JavierMatosD) merged commit dafef74 into microsoft:master Jul 7, 2023
@coryan
Carlos O'Ryan (coryan) deleted the google-cloud-cpp-update-to-v2.13.0 branch July 7, 2023 17:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category:port-update The issue is with a library, which is requesting update new revision info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants