Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[rmqcpp] Update to latest revision #40101

Merged
merged 8 commits into from
Sep 8, 2024
Merged

[rmqcpp] Update to latest revision #40101

Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
50a5175
[rmqcpp] Update to latest revision — Fixes #38736
SirWrexes Jul 26, 2024
dc8a58d
[rmqcpp] Update version metadata to add 1.0.0#1
SirWrexes Jul 26, 2024
2565863
[rmqcpp] Remove `host: true` on dependencie where it doesn't make sense
SirWrexes Jul 28, 2024
a80fce0
[rmqcpp] Update baseline `git-tree`
SirWrexes Jul 28, 2024
36e9884
[rmqcpp] Add patch to skip building examples and tests
SirWrexes Aug 8, 2024
5293891
[rmqcpp] Update baseline and versions file
SirWrexes Aug 9, 2024
46315b4
revert and add port-verison
FrankXie05 Sep 4, 2024
6a6a0be
update version
FrankXie05 Sep 4, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
21 changes: 4 additions & 17 deletions ports/rmqcpp/vcpkg.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,25 +6,12 @@
"dependencies": [
{
"name": "bde",
"host": true,
"version>=": "3.123.0.0"
},
{
"name": "boost-asio",
"host": true
},
{
"name": "boost-iostreams",
"host": true
},
{
"name": "gtest",
"host": true
},
{
"name": "openssl",
"host": true
},
"boost-asio",
"boost-iostreams",
"gtest",
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Despite only being needed for tests, gtest is set as a non-host dependency since there is no way to disable tests in the port for now.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The test dirs are removed from the build. What is left?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Did you try building the port without this dependency? the patch above that deletes add_subdirectory(tests) from CMakeLists.txt should make it so that this dependency is no longer needed.

"openssl",
{
"name": "vcpkg-cmake",
"host": true
Expand Down
Loading