[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#728368: ITP: cpl-plugin-xsh -- ESO data reduction pipeline for XSHOOTER



Hi Julian,

after some code checking, I found out the following:

Only xsh_subtract_sky_single.c uses lapack, (functions SPBTRF and
SPBTRS). They directly call the Fortran routines, without any conversion
(see line 354f.), so in principle only lapack is needed; neither clapack
nor lapacke. The call is however not portable and is bound to the
specific Fortran convention (lower characters, one underscore etc),
which is ugly but works for Debian (and Ubuntu).

So, in principle one could just link to lapack; however I could not find
out how to do this (liblapack does not provide a liblapack.so). Linking
to liblapacke works as a workaround on Debian unstable

On Ubuntu saucy, I have to add a "--no-as-needed" to the linker flags to
get it linked. Since you also contributed to the lapack package: do you
have an idea what happens there on Ubuntu? Is it wise to just add this flag?

Best regards

Ole

Am 31.10.2013 20:01, schrieb Ole Streicher:
> Hi Julian,
> 
> Am 31.10.2013 19:49, schrieb Julian Taylor:
>> On 31.10.2013 11:44, Ole Streicher wrote:
>>> * Package name    : cpl-plugin-xsh
>>
>> this plugin requires clapack which is not in debian nor properly
>> supported upstream anymore, it would need porting to the fortran
>> interface or lapacke. This is not planned upstream, are you going to do it?
> 
> As far as I could see, clapack is used only for the
> xsh_subtract_sky_single recipe -- at least, this is the only one that
> has #includes clapack.h. Replacing this with lapacke (and linking to
> lapacke) works without dangerous warnings, and the tests for this recipe
> pass as well (and also for all other recipes).
> 
> So I would think that the problems here are minor. You can get a test
> version from my personal PPA.
> 
> http://ppa.launchpad.net/olebole/astro-saucy/ubuntu
> 
> (it is in the compiling queue in the moment).
> 
> Best regards
> 
> Ole
> 


Reply to: