statgen / bamUtil

http://genome.sph.umich.edu/wiki/BamUtil
87 stars 29 forks source link

make error #11

Closed songsy closed 10 years ago

songsy commented 10 years ago

make[1]: Entering directory /home/songsy/softwares/bamUtil/src' make -C ../../libStatGen --no-print-directory opt ar -cru ../libStatGen.a obj/bgzf.o obj/knetfile.o g++ -O4 -pipe -Wall -Werror -Wno-strict-overflow -I../include -I. -D__ZLIB_AVAILABLE__ -D_FILE_OFFSET_BITS=64 -D__STDC_LIMIT_MACROS -o obj/GzipHeader.o -c GzipHeader.cpp -DVERSION="\"1.0.0\"" cc1plus: warnings being treated as errors GzipHeader.cpp: In member function ‘bool GzipHeader::readHeader(UncompressedFileType&)’: GzipHeader.cpp:83: error: comparison between signed and unsigned integer expressions make[3]: *** [obj/GzipHeader.o] Error 1 make[2]: *** [general] Error 2 make[1]: *** [../../libStatGen/libStatGen.a] Error 2 make[1]: Leaving directory/home/songsy/softwares/bamUtil/src' make: *\ [src] Error 2

I have no idea what's the problem! Thanks!!

mktrost commented 10 years ago

libStatGen had a place where it was comparing a signed integer with an unsigned integer. Can you upgrade to the latest git version of libStatGen? (https://github.com/statgen/libStatGen) If that is not convenient, it is a simple change to: libStatGen/general/GzipHeader.cpp Edit line 83, add "(int)" before "GZIP_HEADER_SIZE". See this link for what the edit should look like: https://github.com/statgen/libStatGen/commit/80e4e27bcd42c2bf8c716e34942a9841e800a152

Let me know if that fixes your problem or if you have additional issues. Mary Kate