Skip to content

[librabbitmq] Fix cannot include header file when using. #25224

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

Merged
dan-shaw merged 12 commits into from
Jun 17, 2022
Merged

[librabbitmq] Fix cannot include header file when using. #25224

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
8e71b48
fix usage include files
Jun 14, 2022
d4308db
update version
Jun 14, 2022
bd02fb3
add license
Jun 14, 2022
7514ce8
overwrite version
Jun 14, 2022
c60cc02
update patch and options
Jun 15, 2022
06f3094
update version
Jun 15, 2022
d809f06
update patch
Jun 17, 2022
d3ee87d
update version
Jun 17, 2022
a8c8df9
update patch
Jun 17, 2022
6934d56
update version
Jun 17, 2022
15e3fdc
delete blank lines
Jun 17, 2022
3f0b57f
overwrite version
Jun 17, 2022
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
41 changes: 41 additions & 0 deletions ports/librabbitmq/fix-link-header-files.patch
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,41 @@
diff --git a/librabbitmq/CMakeLists.txt b/librabbitmq/CMakeLists.txt
index 671840b..b46614c 100644
--- a/librabbitmq/CMakeLists.txt
+++ b/librabbitmq/CMakeLists.txt
@@ -112,11 +112,25 @@ add_definitions(-DAMQP_BUILD -D_CRT_SECURE_NO_WARNINGS)

set(RMQ_LIBRARIES ${AMQP_SSL_LIBS} ${SOCKET_LIBRARIES} ${LIBRT} ${CMAKE_THREAD_LIBS_INIT})

+set(PUBLIC_INCLUDE_DIRS
+ $<BUILD_INTERFACE:${CMAKE_CURRENT_BINARY_DIR}>
+ $<BUILD_INTERFACE:${CMAKE_CURRENT_SOURCE_DIR}>
+ $<INSTALL_INTERFACE:include>
+)
+
+set(PRIVATE_INCLUDE_DIRS
+ ${CMAKE_CURRENT_BINARY_DIR}
+)
+
if (BUILD_SHARED_LIBS)
add_library(rabbitmq SHARED ${RABBITMQ_SOURCES})

target_link_libraries(rabbitmq ${RMQ_LIBRARIES})

+ target_include_directories(rabbitmq
+ PUBLIC ${PUBLIC_INCLUDE_DIRS}
+ PRIVATE ${PRIVATE_INCLUDE_DIRS}
+ )
if (WIN32)
set_target_properties(rabbitmq PROPERTIES VERSION ${RMQ_VERSION} OUTPUT_NAME rabbitmq.${RMQ_SOVERSION})
else (WIN32)
@@ -137,6 +151,10 @@ if (BUILD_STATIC_LIBS)

target_link_libraries(rabbitmq-static ${RMQ_LIBRARIES})

+ target_include_directories(rabbitmq-static
+ PUBLIC ${PUBLIC_INCLUDE_DIRS}
+ PRIVATE ${PRIVATE_INCLUDE_DIRS}
+ )
set_target_properties(rabbitmq-static PROPERTIES COMPILE_DEFINITIONS AMQP_STATIC)
if (WIN32)
set_target_properties(rabbitmq-static PROPERTIES
3 changes: 2 additions & 1 deletion ports/librabbitmq/portfile.cmake
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,17 +6,18 @@ vcpkg_from_github(
HEAD_REF master
PATCHES
fix-uwpwarning.patch
fix-link-header-files.patch #Remove this patch in the next version
)

string(COMPARE EQUAL "${VCPKG_LIBRARY_LINKAGE}" "static" BUILD_STATIC)
string(COMPARE EQUAL "${VCPKG_LIBRARY_LINKAGE}" "dynamic" BUILD_SHARED)

vcpkg_cmake_configure(
SOURCE_PATH "${SOURCE_PATH}"
OPTIONS
-DBUILD_EXAMPLES=OFF
-DBUILD_TESTS=OFF
-DBUILD_TOOLS=OFF
-DBUILD_STATIC_LIBS=${BUILD_STATIC}
)

vcpkg_cmake_install()
Expand Down
3 changes: 2 additions & 1 deletion ports/librabbitmq/vcpkg.json
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,9 +1,10 @@
{
"name": "librabbitmq",
"version": "0.11.0",
"port-version": 1,
"port-version": 2,
"description": "A C-language AMQP client library for use with v2.0+ of the RabbitMQ broker.",
"homepage": "https://github.com/alanxz/rabbitmq-c",
"license": "MIT",
"dependencies": [
"openssl",
{
Expand Down
2 changes: 1 addition & 1 deletion versions/baseline.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3962,7 +3962,7 @@
},
"librabbitmq": {
"baseline": "0.11.0",
"port-version": 1
"port-version": 2
},
"libraqm": {
"baseline": "0.9.0",
Expand Down
5 changes: 5 additions & 0 deletions versions/l-/librabbitmq.json
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,5 +1,10 @@
{
"versions": [
{
"git-tree": "eaa23b49b84d0ec45e28645d78adc6c0b2f8faed",
"version": "0.11.0",
"port-version": 2
},
{
"git-tree": "3131805ba3b7940db19df08d930a13d2fc1c707c",
"version": "0.11.0",
Expand Down