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!

Dirk

-- 
http://dirk.eddelbuettel.com | @eddelbuettel | e...@debian.org

Reply via email to