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

Bug#781021: marked as done (jessie-pu: package cvxopt/1.1.4-1.2+deb8u1)

Your message dated Sat, 28 Mar 2015 19:45:59 +0100
with message-id <5516F6E7.2070400@thykier.net>
and subject line Re: Bug#781021: jessie-pu: package cvxopt/1.1.4-1.2+deb8u1
has caused the Debian Bug report #781021,
regarding jessie-pu: package cvxopt/1.1.4-1.2+deb8u1
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact owner@bugs.debian.org
immediately.)


-- 
781021: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=781021
Debian Bug Tracking System
Contact owner@bugs.debian.org with problems
--- Begin Message --- 
Package: release.debian.org
User: release.debian.org@packages.debian.org
Usertags: pu
Tags: jessie
Severity: normal

Hello

cvxopt fails to import glpk, with error "undefined symbol" (#780251).
This is due to libglpk0 to libglpk36 ABI change.

There already is a patch in jessie: An emulation of the functions that
disappeared. However, the patch was not linked!

The proposed diff fixes the link, and fixes a #if typo.

I believe it's suitable for future Jessie release point, but not in 8.0
because of the freeze. However, the fix is also in sid, and would you
like to have it sooner, it would be possible to:
unblock cvxopt/1.1.4-1.3

Thank you.


-- System Information:
Debian Release: 8.0
  APT prefers testing
  APT policy: (990, 'testing')
Architecture: amd64 (x86_64)
Foreign Architectures: i386

Kernel: Linux 3.16.0-4-amd64 (SMP w/2 CPU cores)
Locale: LANG=en_GB.utf8, LC_CTYPE=en_GB.utf8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash
Init: sysvinit (via /sbin/init)

-- 
Nirgal

diff -Nru cvxopt-1.1.4/debian/changelog cvxopt-1.1.4/debian/changelog
--- cvxopt-1.1.4/debian/changelog	2013-12-06 17:38:32.000000000 +0100
+++ cvxopt-1.1.4/debian/changelog	2015-03-23 12:02:20.000000000 +0100
@@ -1,3 +1,11 @@
+cvxopt (1.1.4-1.2+deb8u1) unstable; urgency=medium
+
+  * Non-maintainer upload.
+  * Fix glpk-4.49.diff: Link the lpx emulation file, fix the #if activation
+    test. This fixes "undefined symbol: lpx_create_prob" bugs. (Closes: #780251)
+
+ -- Jean-Michel Nirgal Vourgère <jmv_deb@nirgal.com>  Wed, 11 Mar 2015 23:00:40 +0100
+
 cvxopt (1.1.4-1.2) unstable; urgency=medium
 
   * Non-maintainer upload.
diff -Nru cvxopt-1.1.4/debian/patches/glpk-4.49.diff cvxopt-1.1.4/debian/patches/glpk-4.49.diff
--- cvxopt-1.1.4/debian/patches/glpk-4.49.diff	2013-12-06 17:39:16.000000000 +0100
+++ cvxopt-1.1.4/debian/patches/glpk-4.49.diff	2015-03-14 21:15:54.000000000 +0100
@@ -3,13 +3,16 @@
  patch adds compatibility routines that were provided by the upstream author of
  GLPK.
 Author: Sébastien Villemot <sebastien@debian.org>
+Author: Jean-Michel Nirgal Vourgère <jmv_deb@nirgal.com>
 Bug-Debian: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=714368
-Forwarded: no
-Last-Update: 2013-08-15
+Forwarded: not-needed
+Last-Update: 2015-03-11
 ---
 This patch header follows DEP-3: http://dep.debian.net/deps/dep3/
---- a/src/C/glpk.c
-+++ b/src/C/glpk.c
+Index: cvxopt-1.1.4/src/C/glpk.c
+===================================================================
+--- cvxopt-1.1.4.orig/src/C/glpk.c
++++ cvxopt-1.1.4/src/C/glpk.c
 @@ -20,7 +20,7 @@
  
  #include "cvxopt.h"
@@ -19,8 +22,10 @@
  
  PyDoc_STRVAR(glpk__doc__,
      "Interface to the simplex and mixed integer LP algorithms in GLPK.\n\n"
+Index: cvxopt-1.1.4/src/C/lpx.c
+===================================================================
 --- /dev/null
-+++ b/src/C/lpx.c
++++ cvxopt-1.1.4/src/C/lpx.c
 @@ -0,0 +1,1516 @@
 +/* lpx.c (old GLPK API) */
 +
@@ -35,12 +40,12 @@
 +*  Please note that you may mix calls to old and new GLPK API routines
 +*  (except calls to glp_create_prob and glp_delete_prob). */
 +
-+#if (GLP_VERSION_MAJOR == 4 && GLP_MINOR_VERSION >= 49) || GLP_VERSION_MAJOR > 4
-+
 +#include <float.h>
 +#include <limits.h>
 +#include "lpx.h"
 +
++#if (GLP_MAJOR_VERSION == 4 && GLP_MINOR_VERSION >= 49) || GLP_MAJOR_VERSION > 4
++
 +#define xassert glp_assert
 +#define xerror  glp_error
 +
@@ -1538,8 +1543,10 @@
 +
 +/* eof */
 +
+Index: cvxopt-1.1.4/src/C/lpx.h
+===================================================================
 --- /dev/null
-+++ b/src/C/lpx.h
++++ cvxopt-1.1.4/src/C/lpx.h
 @@ -0,0 +1,568 @@
 +/* lpx.h (old GLPK API) */
 +
@@ -2109,3 +2116,16 @@
 +
 +#endif
 +/* eof */
+Index: cvxopt-1.1.4/src/setup.py
+===================================================================
+--- cvxopt-1.1.4.orig/src/setup.py
++++ cvxopt-1.1.4/src/setup.py
+@@ -63,7 +63,7 @@ if BUILD_GLPK:
+     glpk = Extension('glpk', libraries = ['glpk'],
+         include_dirs = [ GLPK_INC_DIR ],
+         library_dirs = [ GLPK_LIB_DIR ],
+-        sources = ['C/glpk.c'] )
++        sources = ['C/glpk.c', 'C/lpx.c'] )
+     extmods += [glpk];
+ 
+ if BUILD_DSDP:

Attachment: signature.asc
Description: OpenPGP digital signature


--- End Message ---
--- Begin Message --- 
On 2015-03-23 12:42, Jean-Michel Nirgal Vourgère wrote:
> Package: release.debian.org
> User: release.debian.org@packages.debian.org
> Usertags: pu
> Tags: jessie
> Severity: normal
> 
> Hello
> 
> cvxopt fails to import glpk, with error "undefined symbol" (#780251).
> This is due to libglpk0 to libglpk36 ABI change.
> 
> There already is a patch in jessie: An emulation of the functions that
> disappeared. However, the patch was not linked!
> 
> The proposed diff fixes the link, and fixes a #if typo.
> 
> I believe it's suitable for future Jessie release point, but not in 8.0
> because of the freeze. However, the fix is also in sid, and would you
> like to have it sooner, it would be possible to:
> unblock cvxopt/1.1.4-1.3
> 
> Thank you.
> 
> 
> [...]

Unblocked, thanks.

For reference, please use an "unblock" request in the future for
requests targeting an unreleased release. :)  The "<dist>-pu" are
intended for updates to an (released) stable release. :)

Thanks,
~Niels

--- End Message ---
Reply to: