search

BoutrosLaboratory / bamql

Query language for filtering SAM/BAM reads
http://labs.oicr.on.ca/boutros-lab/software/BAMQL
Other
31 stars 6 forks source link

configure forces -c++11 but bamql-1.6 and 1.6.1 evidently require -c++14 #26

Closed mmokrejs closed 4 weeks ago

mmokrejs commented 3 months ago

I am not able to compile current git trunk so I downloaded last two tagged releases. In 1.6 you dropped support for older llvm due to new syntax used. The configure fails for me complaining the code requires -c++14 or -gnu++14 but configure.ac forces -c++11 everywhere. Was this overlooked in the past? After editing generated configure and replacing -c++11 with -c++14 I could make configure happy and generate config.status. Still compilation of the package fails for me later but it was a good move ahead.

Practically same happens with 1.6.1 source tree.

Provided >=1.6 require llvm-14 I assume the -c++11 is a leftover from the past?

apmasell commented 4 weeks ago

This has been fixed in the main branch. LLVM use to do really frustrating things with its flags, but they have updated.