Soeren Sonnenburg a écrit :
On Tue, 2010-06-08 at 15:56 +0200, Jean-Christian de Rivaz wrote:
This simple change improve the situation a little:
diff --git a/debian/rules b/debian/rules
index b5301a4..44e5bca 100755
--- a/debian/rules
+++ b/debian/rules
@@ -4,7 +4,7 @@ include /usr/share/cdbs/1/rules/debhelper.mk
include /usr/share/cdbs/1/class/autotools.mk
LDFLAGS += -llapack
-DEB_CONFIGURE_EXTRA_FLAGS += --enable-static
+DEB_CONFIGURE_EXTRA_FLAGS += --enable-static --enable-gnu-packages
wait we cannot do that. coinor is licensed under the CPL. That means it
is in conflict to the GPL and thus products linked to gnu gpl based
backages is not allowed ...
However bzip2/gzip are not gpl but bsd licenses so a different patch
force enabling gzip/bzip2 would work.
(Should I keep your private email for the reply or only the bug email ?
I am not familiar with the Debian reporting convention. Sorry.)
Thanks for pointing out this license issue, I have completely missed it.
From the followings files:
/usr/share/doc/zlib1g-dev/copyright
/usr/share/doc/libbz2-dev/copyright
It seem to me that both libraries have there own licenses, nor GPL, nor BSD.
The /usr/share/doc/coinor-libcoinutils-dev/copyright license is more
complicated, but I get the feeling (to be formally verified) that it is
compatible with the license of both zlib1g and libbz2.
So, from my understanding, the bug is to allow the zlib1g and libbz2
test in the CoinUtils/configure only if the --enable-gnu-packages is
provided. It should alway test them because there are not GNU package at
all and do not use the GPL license. I successfully tested this idea with
the following patch (also in attachment because I worry that the mail
split some lines):
diff --git a/CoinUtils/configure b/CoinUtils/configure
index 4e7660d..67fc675 100755
--- a/CoinUtils/configure
+++ b/CoinUtils/configure
@@ -32870,7 +32870,6 @@ fi;
coin_has_zlib=no
-if test $coin_enable_gnu = yes; then
#if test x"" = x; then
# hdr="#include <zlib.h>"
#else
@@ -33117,7 +33116,6 @@ cat >>confdefs.h <<\_ACEOF
_ACEOF
fi
-fi
@@ -33137,7 +33135,6 @@ fi
coin_has_bzlib=no
-if test $coin_enable_gnu = yes; then
#if test x"" = x; then
# hdr="#include <bzlib.h>"
#else
@@ -33384,7 +33381,6 @@ cat >>confdefs.h <<\_ACEOF
_ACEOF
fi
-fi
##############################################################################
diff --git a/debian/rules b/debian/rules
index b5301a4..d577b37 100755
--- a/debian/rules
+++ b/debian/rules
@@ -3,7 +3,7 @@
include /usr/share/cdbs/1/rules/debhelper.mk
include /usr/share/cdbs/1/class/autotools.mk
-LDFLAGS += -llapack
+LDFLAGS += -llapack -lz -lbz2
DEB_CONFIGURE_EXTRA_FLAGS += --enable-static
build/coinor-libcoinutils-doc:: debian/stamp-build-coinor-libcoinutils-doc
Could this satisfy the license problem ?
Jean-Christian
diff --git a/CoinUtils/configure b/CoinUtils/configure
index 4e7660d..67fc675 100755
--- a/CoinUtils/configure
+++ b/CoinUtils/configure
@@ -32870,7 +32870,6 @@ fi;
coin_has_zlib=no
-if test $coin_enable_gnu = yes; then
#if test x"" = x; then
# hdr="#include <zlib.h>"
#else
@@ -33117,7 +33116,6 @@ cat >>confdefs.h <<\_ACEOF
_ACEOF
fi
-fi
@@ -33137,7 +33135,6 @@ fi
coin_has_bzlib=no
-if test $coin_enable_gnu = yes; then
#if test x"" = x; then
# hdr="#include <bzlib.h>"
#else
@@ -33384,7 +33381,6 @@ cat >>confdefs.h <<\_ACEOF
_ACEOF
fi
-fi
##############################################################################
diff --git a/debian/rules b/debian/rules
index b5301a4..d577b37 100755
--- a/debian/rules
+++ b/debian/rules
@@ -3,7 +3,7 @@
include /usr/share/cdbs/1/rules/debhelper.mk
include /usr/share/cdbs/1/class/autotools.mk
-LDFLAGS += -llapack
+LDFLAGS += -llapack -lz -lbz2
DEB_CONFIGURE_EXTRA_FLAGS += --enable-static
build/coinor-libcoinutils-doc:: debian/stamp-build-coinor-libcoinutils-doc
