You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
build_google_cloud_cpp_storage() calls resolve_dependency(OpenSSL) but it should not be here. We should have only one resolve_dependency(OpenSSL) for easy to maintain.
Component(s)
C++
The text was updated successfully, but these errors were encountered:
### Rationale for this change
`build_google_cloud_cpp_storage()` calls `resolve_dependency(OpenSSL)` but it should not be here. We should have only one `resolve_dependency(OpenSSL)` for easy to maintain.
### What changes are included in this PR?
Don't call `resolve_dependency(OpenSSL)` from `build_google_cloud_cpp_storage()`.
### Are these changes tested?
Yes.
### Are there any user-facing changes?
No.
* Closes: #36944
Authored-by: Sutou Kouhei <kou@clear-code.com>
Signed-off-by: Sutou Kouhei <kou@clear-code.com>
…e#36945)
### Rationale for this change
`build_google_cloud_cpp_storage()` calls `resolve_dependency(OpenSSL)` but it should not be here. We should have only one `resolve_dependency(OpenSSL)` for easy to maintain.
### What changes are included in this PR?
Don't call `resolve_dependency(OpenSSL)` from `build_google_cloud_cpp_storage()`.
### Are these changes tested?
Yes.
### Are there any user-facing changes?
No.
* Closes: apache#36944
Authored-by: Sutou Kouhei <kou@clear-code.com>
Signed-off-by: Sutou Kouhei <kou@clear-code.com>
…e#36945)
### Rationale for this change
`build_google_cloud_cpp_storage()` calls `resolve_dependency(OpenSSL)` but it should not be here. We should have only one `resolve_dependency(OpenSSL)` for easy to maintain.
### What changes are included in this PR?
Don't call `resolve_dependency(OpenSSL)` from `build_google_cloud_cpp_storage()`.
### Are these changes tested?
Yes.
### Are there any user-facing changes?
No.
* Closes: apache#36944
Authored-by: Sutou Kouhei <kou@clear-code.com>
Signed-off-by: Sutou Kouhei <kou@clear-code.com>
Describe the enhancement requested
build_google_cloud_cpp_storage()
callsresolve_dependency(OpenSSL)
but it should not be here. We should have only oneresolve_dependency(OpenSSL)
for easy to maintain.Component(s)
C++
The text was updated successfully, but these errors were encountered: