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.

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

[librabbitmq] Update to use rabbitmq-config.cmake #11733

Merged
merged 3 commits into from
Jun 12, 2020

Conversation

trollham
Copy link
Contributor

@trollham trollham commented Jun 3, 2020

The rabbitmq-c library recently merged a pull request that implemented config-file generation. Vcpkg can now better integrate librabbitmq into it's ecosystem. I tested this works on the following triplets: x64-linux, x64-windows, and a custom x64-windows-static which just changes library linkage from "dynamic" to "static".

@ghost
Copy link

ghost commented Jun 3, 2020

CLA assistant check
All CLA requirements met.

@trollham trollham marked this pull request as ready for review June 3, 2020 04:45
@NancyLi1013 NancyLi1013 self-assigned this Jun 3, 2020
ports/librabbitmq/CONTROL Outdated Show resolved Hide resolved
ports/librabbitmq/portfile.cmake Show resolved Hide resolved
ports/librabbitmq/portfile.cmake Show resolved Hide resolved
ports/librabbitmq/portfile.cmake Outdated Show resolved Hide resolved
@NancyLi1013 NancyLi1013 added category:port-update The issue is with a library, which is requesting update new revision requires:author-response labels Jun 3, 2020
@NancyLi1013 NancyLi1013 changed the title Update librabbitmq portfile to use rabbitmq-config.cmake [librabbitmq] Update to use rabbitmq-config.cmake Jun 3, 2020
@trollham
Copy link
Contributor Author

trollham commented Jun 3, 2020

@NancyLi1013 Thank you for your feedback! I've made the proposed changes.

@JackBoosY
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@NancyLi1013 NancyLi1013 added the info:reviewed Pull Request changes follow basic guidelines label Jun 10, 2020
@dan-shaw
Copy link
Contributor

Thanks for the PR!

@dan-shaw dan-shaw merged commit ca52d42 into microsoft:master Jun 12, 2020
JangBoo pushed a commit to JangBoo/vcpkg that referenced this pull request Jun 18, 2020
* Update librabbitmq portfile to use rabbitmq-config.cmake

* Change version and update outdated vcpkg functions

* Capitalization error

Co-authored-by: Eric Huff <thegreaterici@gmail.com>
penumbra23 pushed a commit to codespace-dev/vcpkg that referenced this pull request Aug 5, 2020
* Update librabbitmq portfile to use rabbitmq-config.cmake

* Change version and update outdated vcpkg functions

* Capitalization error

Co-authored-by: Eric Huff <thegreaterici@gmail.com>
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:reviewed Pull Request changes follow basic guidelines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants