On 30 October 2018 at 19:47, Dirk Eddelbuettel wrote: | | Hi, | | On 31 October 2018 at 00:15, D Haley wrote: | | Hi, | | | | I've attached a patch that might suit - | | Thanks for that. | | | however I am a bit unsure about modifying a non-leaf package, such as GSL. | | "Yes but" this is new-ish functionality in 2.5 only. So few depends on this. | | I'd say if 'make; make check' passes it is worth a try. Could you try 'make | check' on your end? | | | Most of the references to delete were of the form xxx_delete, which are | | not relevant to the current problem (as this does not conflict with the | | C++ operator). The only externally visible (in header files) clash that | | I could find was in the gsl_movstat.h file. This is referenced in a few | | files in movstat/ internally, and needed to be adjusted. | | I saw just that when I glanced at it, but that was while I was at work and | preoccupied with a million other things. | | It is clearly just a field in a struct we can renamed but ... someone | somewhere must be referencing the struct, no? | | We can also get Patrick involved who is ... pretty much carrying upstream GNU | GSL by himself from what I can tell. | | | This fixes the problem for me, and can be used as a patch in the debian | | gsl git, c3eee7ef (as a file in debian/patches/). I've rebuilt the .deb | | on a VM, and made a few quick tests using g++. | | It's a good start so thanks for that!
Sorry for taking so long on this -- I just uploaded a new version with the corresponding upstream patch. Thanks again for reporting this. Cheers, Dirk | Dirk | | -- | http://dirk.eddelbuettel.com | @eddelbuettel | e...@debian.org -- http://dirk.eddelbuettel.com | @eddelbuettel | e...@debian.org
