Bug 37 – ApiExtractor needs the binary libboost-graph library to be built

PySide Bugzilla Closed for New Bugs

PySide is now a Qt Add-on and uses the Qt Project's JIRA Bug Tracker instead of this Bugzilla instance. This Bugzilla is left for reference purposes.

Bug 37 - ApiExtractor needs the binary libboost-graph library to be built

Summary:

ApiExtractor needs the binary libboost-graph library to be built

Status:	CLOSED FIXED

Product:	PySide
Classification:	Unclassified
Component:	ApiExtractor
Version:	0.3.0
Platform:	All Linux

Importance:	P5 normal
Assigned To:	renato filho

URL:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2009-09-07 06:29 EEST by Didier Raboud
Modified:	2009-10-14 14:20 EEST (History)
CC List:	5 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Didier Raboud 2009-09-07 06:29:37 EEST

Hi, 

while trying to build apiextractor 0.3 in pbuilder (minimal schroot, with only
build-depends packages installed), I fell on the issue that the
/usr/lib/libboost_graph1.39.0.so is needed on the system to be correctly built.

To avoid this (which for Debian means a great hassle for maintainance), the
following patch is sufficient :

Index: apiextractor-0.3/CMakeLists.txt
===================================================================
--- apiextractor-0.3.orig/CMakeLists.txt        2009-09-07 10:28:46.000000000
+0200
+++ apiextractor-0.3/CMakeLists.txt     2009-09-07 10:29:13.000000000 +0200
 -69,13 +69,13 @@
                     ${CMAKE_CURRENT_BINARY_DIR}
                     ${CMAKE_CURRENT_SOURCE_DIR}/parser
                     ${CMAKE_CURRENT_SOURCE_DIR}/parser/rpp
-                    ${QT_INCLUDE_DIR} ${Boost_INCLUDE_DIR}
+                    ${QT_INCLUDE_DIR}
                     ${LIBXSLT_INCLUDE_DIRS}
                     ${LIBXML2_INCLUDE_DIRS}
                     )

 add_library(apiextractor SHARED ${apiextractor_SRC} ${apiextractor_RCCS_SRC})
-target_link_libraries(apiextractor ${Boost_GRAPH_LIBRARY} ${LIBXSLT_LIBRARIES}
${LIBXML2_LIBRARIES} ${QT_QTCORE_LIBRARY} ${QT_QTXMLPATTERNS_LIBRARY}
${QT_QTXML_LIBRARY})
+target_link_libraries(apiextractor ${LIBXSLT_LIBRARIES} ${LIBXML2_LIBRARIES}
${QT_QTCORE_LIBRARY} ${QT_QTXMLPATTERNS_LIBRARY} ${QT_QTXML_LIBRARY})
 set_target_properties(apiextractor PROPERTIES VERSION ${apiextractor_VERSION}
SOVERSION ${apiextractor_MAJOR_VERSION})

 # create pkg-config file






Best regards, 

OdyX

Comment 1 Lauro Moura 2009-09-07 17:58:40 EEST

Should the include be removed as well? APIExtractor requires the boost::graph
headers and couldn't removing it break systems where boost is installed outside
the default search path?

For the library requirement removal, I'm fine with it.

Comment 2 Hugo Parente Lima 2009-09-08 11:58:42 EEST

Fixed in commit 3247a6f50cd0b50fb046de3845a52e4860f2fbfb.

APIExtractor still needs boost::graph library, but JUST to be built, not to
run, because boost::graph is a header-only library.