search

kgreenek / rules_pcl

Bazel rules for building PCL libraries (PointCloudLibrary)
Apache License 2.0
15 stars 15 forks source link

Remove @qhull//:libqhull from surface and add surface tests #13

Closed franciszhi closed 1 year ago

franciszhi commented 1 year ago

When "@qhull//:libqhull" is used in :surface, the following error will occur:

QH6248 qh_lib_check: Incorrect qhull library called.  Caller uses reentrant Qhull while qhull library is non-reentrant
QH6249 qh_lib_check: Incorrect qhull library called.  Size of qhT for caller is 8752, but for qhull library is 3000.
QH6256 qhull internal error (qh_lib_check): Cannot continue due to QH6249..  Library 'qhull 7.3.2 (2019.1 2019/06/21)' should use a static qhT (e.g., libqhull.so).  It may be out-of-date.  Exit with 49
kgreenek commented 1 year ago

Thank you for finding this and for sending the fix! This is also a great example of how to pass args to a cc_test, which I actually wasn't aware of. I will use this example to add the rest of the tests which take a pcd file as an arg.