-
Notifications
You must be signed in to change notification settings - Fork 6.1k
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
[etcd-cpp-apiv3] Add etcd-cpp-apiv3 ports. #29363
Conversation
Signed-off-by: Tao He <sighingnow@gmail.com>
Signed-off-by: Tao He <sighingnow@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a new experimental fast check for PR issues. Please let us know if this bot is helpful!
PRs must add only one version and must not modify any published versions
When making any changes to a library, the version or port-version in vcpkg.json
or CONTROL
must be modified.
error: checked-in files for etcd-cpp-apiv3 have changed but the version was not updated
version: 0.2.12
old SHA: 1c700da2d3e7ad82e97bca5ecc5b24129519b18d
new SHA: 8ac6dd66dbc8eb4349b1df236e96898e7de654d7
Did you remember to update the version or port version?
Use --overwrite-version to bypass this check
***No files were updated***
You have modified or added at least one portfile where deprecated functions are used.
If you feel able to do so, please consider migrating them to the new functions:
vcpkg_install_cmake
-> vcpkg_cmake_install
(from port vcpkg-cmake
)
vcpkg_build_cmake
-> vcpkg_cmake_build
(from port vcpkg-cmake
)
vcpkg_configure_cmake
-> vcpkg_cmake_configure
(Please remove the option PREFER_NINJA
) (from port vcpkg-cmake
)
vcpkg_fixup_cmake_targets
-> vcpkg_cmake_config_fixup
(from port vcpkg-cmake-config
)
vcpkg_extract_source_archive_ex
-> vcpkg_extract_source_archive
vcpkg_build_msbuild
-> vcpkg_install_msbuild
vcpkg_copy_tool_dependencies
-> vcpkg_copy_tools
vcpkg_apply_patches
should be replaced by the PATCHES
arguments to the "extract" helpers (e.g. vcpkg_from_github()
)
In the ports that use the new function, you have to add the corresponding dependencies:
{
"name": "vcpkg-cmake",
"host": true
},
{
"name": "vcpkg-cmake-config",
"host": true
}
The following files are affected:
ports/etcd-cpp-apiv3/portfile.cmake
You have modified or added at least one vcpkg.json where you should check the license
field.
If you feel able to do so, please consider adding a "license" field to the following files:
ports/etcd-cpp-apiv3/vcpkg.json
Valid values for the license field can be found in the documentation
Signed-off-by: Tao He <sighingnow@gmail.com>
Hi @MonicaLiu0311, Could you please help to take a look at the CI errors? The "Create Diff" failed should be caused by this pull request. All comments above has been addressed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a new experimental fast check for PR issues. Please let us know if this bot is helpful!
All manifest files must be formatted
./vcpkg format-manifest ports/*/vcpkg.json
Diff
diff --git a/ports/etcd-cpp-apiv3/vcpkg.json b/ports/etcd-cpp-apiv3/vcpkg.json
index 359d9f8..3ad3971 100644
--- a/ports/etcd-cpp-apiv3/vcpkg.json
+++ b/ports/etcd-cpp-apiv3/vcpkg.json
@@ -5,14 +5,6 @@
"homepage": "https://github.com/etcd-cpp-apiv3/etcd-cpp-apiv3",
"license": "BSD-3-Clause",
"dependencies": [
- {
- "name": "vcpkg-cmake",
- "host": true
- },
- {
- "name": "vcpkg-cmake-config",
- "host": true
- },
"boost-asio",
"boost-random",
"boost-system",
@@ -20,6 +12,14 @@
"cpprestsdk",
"grpc",
"openssl",
- "protobuf"
+ "protobuf",
+ {
+ "name": "vcpkg-cmake",
+ "host": true
+ },
+ {
+ "name": "vcpkg-cmake-config",
+ "host": true
+ }
]
}
PRs must add only one version and must not modify any published versions
When making any changes to a library, the version or port-version in vcpkg.json
or CONTROL
must be modified.
error: checked-in files for etcd-cpp-apiv3 have changed but the version was not updated
version: 0.2.12
old SHA: 8ac6dd66dbc8eb4349b1df236e96898e7de654d7
new SHA: d11dfdbdc719392013a05c613a68271b167b7ec2
Did you remember to update the version or port version?
Use --overwrite-version to bypass this check
***No files were updated***
The error log is as above, please run the following command and push.
Explanation of After the first modification, please run |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a new experimental fast check for PR issues. Please let us know if this bot is helpful!
PRs must add only one version and must not modify any published versions
When making any changes to a library, the version or port-version in vcpkg.json
or CONTROL
must be modified.
error: checked-in files for etcd-cpp-apiv3 have changed but the version was not updated
version: 0.2.12
old SHA: 8ac6dd66dbc8eb4349b1df236e96898e7de654d7
new SHA: d11dfdbdc719392013a05c613a68271b167b7ec2
Did you remember to update the version or port version?
Use --overwrite-version to bypass this check
***No files were updated***
Signed-off-by: Tao He <sighingnow@gmail.com>
CI passed. Can @MonicaLiu0311 or @LilyWangLL help to get this port merged? Thanks! |
Polite ping @MonicaLiu0311 @LilyWangLL |
|
Signed-off-by: Tao He <sighingnow@gmail.com>
1f7847c
Usage text updated. Sorry for missing that. |
@sighingnow please follow instructions #29363 (comment) to fix the versioning issue. |
Signed-off-by: Tao He <sighingnow@gmail.com>
Fixed. |
Signed-off-by: Tao He <sighingnow@gmail.com>
Signed-off-by: Tao He <sighingnow@gmail.com>
Hi @MonicaLiu0311, Comment addressed. Can we get this pr accepted now? Thanks! |
find_package
calls are REQUIRED, are satisfied byvcpkg.json
's declared dependencies, or disabled with CMAKE_DISABLE_FIND_PACKAGE_Xxxvcpkg.json
matches what upstream says.vcpkg.json
matches what upstream says../vcpkg x-add-version --all
and committing the result.