dropEst icon indicating copy to clipboard operation
dropEst copied to clipboard

CMake is looking for BAMTOOLS_ROOT/lib but BAMTOOLS creates lib64

Open jjv5 opened this issue 4 years ago • 3 comments

In Cmake, FindBamTools.cmake:

find_library(BAMTOOLS_LIBRARY bamtools PATHS ${BAMTOOLS_ROOT}/lib PATH_SUFFIXES bamtools ../lib)

so finding BAMTOOLS fails no matter what env vars are set to.

bamtools 2.5.0 installs in library directory named lib64, not lib.

jjv5 avatar Jul 12 '21 18:07 jjv5

Need a new pull request?

diff --git a/CMake/FindBamTools.cmake b/CMake/FindBamTools.cmake
index 333ac17..aff5c47 100644
--- a/CMake/FindBamTools.cmake
+++ b/CMake/FindBamTools.cmake
@@ -9,7 +9,7 @@
 #   BAMTOOLS_FOUND - true if BAM has been found and can be used
 
 find_path(BAMTOOLS_INCLUDE_DIR api/BamReader.h PATHS ${BAMTOOLS_ROOT}/include PATH_SUFFIXES bamtools ../include ../include/bamtools)
-find_library(BAMTOOLS_LIBRARY bamtools PATHS ${BAMTOOLS_ROOT}/lib PATH_SUFFIXES bamtools ../lib)
+find_library(BAMTOOLS_LIBRARY bamtools PATHS ${BAMTOOLS_ROOT}/lib64 PATH_SUFFIXES bamtools ../lib)
 
 include(FindPackageHandleStandardArgs)
 FIND_PACKAGE_HANDLE_STANDARD_ARGS(BAMTOOLS

hailingfang avatar Jun 22 '22 01:06 hailingfang

Hi @benjaminfang

Yes, please add to the existing PR and update. There are several things here to update, and I never had the bandwidth to do it properly.

I think the latest Docker image is here btw: https://hub.docker.com/u/pkharchenkolab

evanbiederstedt avatar Jun 22 '22 01:06 evanbiederstedt

OK, done. please review.

hailingfang avatar Jun 22 '22 02:06 hailingfang