Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Remove 2d library (contains no symbols) #1679

Merged
merged 1 commit into from
Aug 18, 2016

Conversation

jspricke
Copy link
Member

No description provided.

@SergioRAgostinho
Copy link
Member

just confirmed it 😮 👍

@taketwo taketwo merged commit 40b5cd8 into PointCloudLibrary:master Aug 18, 2016
@jspricke jspricke deleted the 2d_remove_lib branch August 18, 2016 14:04
PCL_ADD_LIBRARY("${LIB_NAME}" "${SUBSYS_NAME}" ${srcs} ${incs} ${impl_incs})
link_directories(${VTK_LINK_DIRECTORIES})
target_link_libraries("${LIB_NAME}" ${VTK_LIBRARIES} pcl_io)
PCL_MAKE_PKGCONFIG("${LIB_NAME}" "${SUBSYS_NAME}" "${SUBSYS_DESC}" "${SUBSYS_DEPS}" "" "" "" "")
Copy link
Member

Choose a reason for hiding this comment

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

This should be PCL_MAKE_PKGCONFIG_HEADER_ONLY

Copy link
Member Author

Choose a reason for hiding this comment

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

Can you send a new pull request? This is merged ;)

Copy link
Member

Choose a reason for hiding this comment

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

OK

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants