Bug#884974: sdpa: hardcoded mumps runtime dependency

[Makoto Yamashita]

Dear Gianfranco Costamagna,

Thank you very much for your support.
>From the subsequent message, I understand you kindly applied the
modification.
I really appreciate your work.

Best regards,
Makoto Yamashita



2019年8月2日(金) 20:54 Gianfranco Costamagna :

> control: tags -1 patch pending
>
> uploaded in deferred/2
>
> G.
>

Bug#884974: sdpa: hardcoded mumps runtime dependency

[Gianfranco Costamagna]

control: tags -1 patch pending

uploaded in deferred/2

G.
diff -Nru sdpa-7.3.11+dfsg/debian/changelog sdpa-7.3.11+dfsg/debian/changelog
--- sdpa-7.3.11+dfsg/debian/changelog   2017-12-05 15:46:32.0 +0100
+++ sdpa-7.3.11+dfsg/debian/changelog   2019-08-02 13:46:12.0 +0200
@@ -1,3 +1,10 @@
+sdpa (7.3.11+dfsg-1.1) unstable; urgency=medium
+
+  * Non-maintainer upload.
+  * Drop runtime dependency on mups (Closes: #884974)
+
+ -- Gianfranco Costamagna   Fri, 02 Aug 2019 
13:46:12 +0200
+
 sdpa (7.3.11+dfsg-1) unstable; urgency=low
 
 * New upstream
diff -Nru sdpa-7.3.11+dfsg/debian/control sdpa-7.3.11+dfsg/debian/control
--- sdpa-7.3.11+dfsg/debian/control 2017-12-05 15:46:32.0 +0100
+++ sdpa-7.3.11+dfsg/debian/control 2019-08-02 13:46:10.0 +0200
@@ -11,7 +11,7 @@
 Package: sdpa
 Architecture: any
 Depends: ${shlibs:Depends}, ${misc:Depends},
-   libopenblas-dev, libmumps-seq-5.1.1
+   libopenblas-dev
 Description: High-performance package for SemiDefinite Programs
  The software SDPA (SemiDefinite Programming Algorithm) is one of the most
  efficient and stable  software packages for solving SDPs based on the
@@ -38,7 +38,7 @@
 Package: sdpam
 Architecture: any
 Depends: ${shlibs:Depends}, ${misc:Depends},
-   libopenblas-dev, libmumps-seq-5.1.1, 
+   libopenblas-dev,
octave, libsdpa-dev
 Description: Matlab/Octave interface of SDPA
  This package provides SDPA-M, Matlab/Octave interface 

Bug#884974: sdpa: hardcoded mumps runtime dependency

[Gianfranco Costamagna]

Hello,
> 
> >When I prepared the debian package, I needed  libmumps-seq-4.10.0.
> >Without  libmumps-seq-4.10.0, the SDPA package installed by apt-get 
> >could not work, since apt-get could not know that SDPA requires the mumps 
> >library.
> 
> 
> this seems to be not true, usually runtime dependencies are evaluated, just 
> assuming somebody dynamically
> links to them.
> Did you link them statically and then got that problem? if you link 
> dinamically, and you build the package,
> you should get the runtime dependencies automatically evaluated for you.
> 
> >And, as for OpenBLAS, as you already know that we switched from ATLAS to 
> >OpenBLAS.
> >A main reason is that OpenBLAS is much faster than ATLAS.
> >It would be great that OpenBLAS would work on more plathomes.
> 
> 
> this is not a problem, just make sure you open a bug against ftp.debian.org 
> with reportbug and ask for their removal.
> 

ping!

G.

> 
> 
> cheers,
> 
> G.
> 
> 
>  
> 
> 
> 2017-12-24 21:38 GMT+09:00 Gianfranco Costamagna :
> 
> Hello,
> >
> >>I already applied the patch you sent before.
> >>Could you give me more details on what you are indicating?
> >>It is now too ambiguous to do a next step correctly.
> >>Now, I cannot find any trouble.
> >
> >
> >trouble is:
> >1) you hardcode a runtime dependency on mumps, this makes impossible to do a 
> >new mumps
> >transition without having to full source upload sdpa each times.
> >2) You are including it statically, without no good reason
> >
> >see the patch here to know how to properly do it
> >https://bugs.debian.org/cgi- bin/bugreport.cgi?att=1;bug= 
> >868505;filename=debdiff2;msg= 27
> >
> >other issues:
> >a) it depends on openblas where it is not available. This is making the 
> >package not built where it has been
> >in previous releases.
> >You should drop that dependency, or ask ftpmasters to remove such 
> >architectures
> >https://buildd.debian.org/ status/package.php?p=sdpa& suite=unstable
> >
> >hope this clarifies a bit
> >
> >G.
> >
> 
> 

Bug#884974: sdpa: hardcoded mumps runtime dependency

[Gianfranco Costamagna]

Hello,

>When I prepared the debian package, I needed  libmumps-seq-4.10.0.
>Without  libmumps-seq-4.10.0, the SDPA package installed by apt-get 
>could not work, since apt-get could not know that SDPA requires the mumps 
>library.


this seems to be not true, usually runtime dependencies are evaluated, just 
assuming somebody dynamically
links to them.
Did you link them statically and then got that problem? if you link 
dinamically, and you build the package,
you should get the runtime dependencies automatically evaluated for you.

>And, as for OpenBLAS, as you already know that we switched from ATLAS to 
>OpenBLAS.
>A main reason is that OpenBLAS is much faster than ATLAS.
>It would be great that OpenBLAS would work on more plathomes.


this is not a problem, just make sure you open a bug against ftp.debian.org 
with reportbug and ask for their removal.



cheers,

G.


 


2017-12-24 21:38 GMT+09:00 Gianfranco Costamagna :

Hello,
>
>>I already applied the patch you sent before.
>>Could you give me more details on what you are indicating?
>>It is now too ambiguous to do a next step correctly.
>>Now, I cannot find any trouble.
>
>
>trouble is:
>1) you hardcode a runtime dependency on mumps, this makes impossible to do a 
>new mumps
>transition without having to full source upload sdpa each times.
>2) You are including it statically, without no good reason
>
>see the patch here to know how to properly do it
>https://bugs.debian.org/cgi- bin/bugreport.cgi?att=1;bug= 
>868505;filename=debdiff2;msg= 27
>
>other issues:
>a) it depends on openblas where it is not available. This is making the 
>package not built where it has been
>in previous releases.
>You should drop that dependency, or ask ftpmasters to remove such architectures
>https://buildd.debian.org/ status/package.php?p=sdpa& suite=unstable
>
>hope this clarifies a bit
>
>G.
>

Bug#884974: sdpa: hardcoded mumps runtime dependency

[Drew Parsons]

On Sun, 24 Dec 2017 22:01:33 +0900 Makoto Yamashita  wrote:
> 
> However, unfortunately for us, I could not resolve the issues.
> (I tried them before, but I could not.)
> 
> When I prepared the debian package, I needed  libmumps-seq-4.10.0.
> Without  libmumps-seq-4.10.0, the SDPA package installed by apt-get
> could not work, since apt-get could not know that SDPA requires the
mumps
> library.

I am confused by this.  In 7.3.9+dfsg-1.1 the dependency of sdpa and
sdpam on libmumps-seq-5.1.1 was computed automatically by the debian
build scripts and included in shlibs:Depends, so it did not need to be
hardwired.  The dependency was detected from sdpa's /usr/bin/sdpa and
from  sdpam's /usr/lib/sdpa/mex/*.mex, pulled in via dmumps used in
sdpa_chordal.cpp. How has that behaviour since changed?

> And, as for OpenBLAS, as you already know that we switched from ATLAS
to
> OpenBLAS.
> A main reason is that OpenBLAS is much faster than ATLAS.
> It would be great that OpenBLAS would work on more plathomes.

I think having sdpa available with a slower BLAS on those platforms
without OpenBLAS is better than not having sdpa available at all.

The BLAS libraries are interchangeable at runtime (i.e. any of them can
be used in place of any other, they all provide libblas3 |libblas.so.3)
so it does not matter for runtime performance if the build is made
using the generic reference BLAS.  Users can still install the best
performing BLAS for their system. sdpa does not need to be built
against OpenBLAS in order for it to run with OpenBLAS.  libblas-dev
should be used as the preferred Build-Depends since it is the only BLAS
available on all platforms (the buildds will only try to install the
first alternative).

Build-Depends: libblas-dev | libopenblas-dev | libatlas-base-dev | libblas.so

Bug#884974: sdpa: hardcoded mumps runtime dependency

[Makoto Yamashita]

Dear Mr. Gianfranco Costamagna,

Thank you very much for your mail.

Your mail makes me understand your points, and I agree them.

However, unfortunately for us, I could not resolve the issues.
(I tried them before, but I could not.)

When I prepared the debian package, I needed  libmumps-seq-4.10.0.
Without  libmumps-seq-4.10.0, the SDPA package installed by apt-get
could not work, since apt-get could not know that SDPA requires the mumps
library.

And, as for OpenBLAS, as you already know that we switched from ATLAS to
OpenBLAS.
A main reason is that OpenBLAS is much faster than ATLAS.
It would be great that OpenBLAS would work on more plathomes.

Thank you very much again.
Makoto Yamashita




2017-12-24 21:38 GMT+09:00 Gianfranco Costamagna :

> Hello,
>
> >I already applied the patch you sent before.
> >Could you give me more details on what you are indicating?
> >It is now too ambiguous to do a next step correctly.
> >Now, I cannot find any trouble.
>
>
> trouble is:
> 1) you hardcode a runtime dependency on mumps, this makes impossible to do
> a new mumps
> transition without having to full source upload sdpa each times.
> 2) You are including it statically, without no good reason
>
> see the patch here to know how to properly do it
> https://bugs.debian.org/cgi-bin/bugreport.cgi?att=1;bug=
> 868505;filename=debdiff2;msg=27
>
> other issues:
> a) it depends on openblas where it is not available. This is making the
> package not built where it has been
> in previous releases.
> You should drop that dependency, or ask ftpmasters to remove such
> architectures
> https://buildd.debian.org/status/package.php?p=sdpa=unstable
>
> hope this clarifies a bit
>
> G.
>

Bug#884974: sdpa: hardcoded mumps runtime dependency

[Gianfranco Costamagna]

Hello,

>I already applied the patch you sent before.
>Could you give me more details on what you are indicating?
>It is now too ambiguous to do a next step correctly.
>Now, I cannot find any trouble.


trouble is:
1) you hardcode a runtime dependency on mumps, this makes impossible to do a 
new mumps
transition without having to full source upload sdpa each times.
2) You are including it statically, without no good reason

see the patch here to know how to properly do it
https://bugs.debian.org/cgi-bin/bugreport.cgi?att=1;bug=868505;filename=debdiff2;msg=27

other issues:
a) it depends on openblas where it is not available. This is making the package 
not built where it has been
in previous releases.
You should drop that dependency, or ask ftpmasters to remove such architectures
https://buildd.debian.org/status/package.php?p=sdpa=unstable

hope this clarifies a bit

G.

Bug#884974: sdpa: hardcoded mumps runtime dependency

[Makoto Yamashita]

Dear Mr. Gianfranco Costamagna,

Thank you very much for your report.

I already applied the patch you sent before.
Could you give me more details on what you are indicating?
It is now too ambiguous to do a next step correctly.
Now, I cannot find any trouble.

Thank you very much.
Makoto Yamashita

2017-12-22 17:18 GMT+08:00 Gianfranco Costamagna :

> Package: sdpa
> Severity: serious
> Version: 7.3.11+dfsg-1
> Tags: patch
> Justification: hardcoded cruft library
>
> Hello, *please* *please* *please* drop the runtime dependency, and maybe
> stop using static mumps libraries, unless *really* necessary.
>
> I provided a patch in [1] but it seems to have been applied very
> differently, and I don't understand the reasons.
>
> [1] https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=868505;msg=27
>
>
> thanks
>
> G.
>
>

Bug#884974: sdpa: hardcoded mumps runtime dependency

[Gianfranco Costamagna]

Package: sdpa
Severity: serious
Version: 7.3.11+dfsg-1
Tags: patch
Justification: hardcoded cruft library

Hello, *please* *please* *please* drop the runtime dependency, and maybe stop 
using static mumps libraries, unless *really* necessary.

I provided a patch in [1] but it seems to have been applied very differently, 
and I don't understand the reasons.

[1] https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=868505;msg=27


thanks

G.



signature.asc
Description: OpenPGP digital signature