Duplicate LALSUITE_CHECK_PYTHON([3.5]) in lalburst's configure.ac (2.0.3)
This dates back to lalburst
1.4.4 from 2017, but the effect hasn't shown up in full bloom until now:
configure.ac
contains two lines LALSUITE_CHECK_PYTHON([...])
, unlike all other LALsuite components. This results in a massive chunk of code being duplicated in configure
- and issues when attempting to patch that file for Debian builds (i.e. add 'deb_system'
scheme to move Python modules to their Debian default).
The change seems to be commit f2520fc2 from Dec 1, 2016 by @kipp.cannon , that duplicates the addition of that CHECK
from commit a99edbc0 (Jul 24, 2014) by @karl-wette .
IMHO the second occurrence should be removed, to get back in sync with the other packages in LALsuite. Opinions, in particular from the contributors?