Bug#718893: RFS: coinutils/2.9.4-1 [ITA] -- CoinOR base library
I've uploaded my current work (without autotools fix) to the debian-science git repo as coinutils.git. Please review. Thanks, Miles On Mon, Aug 19, 2013 at 11:08 PM, Miles Lubin miles.lu...@gmail.com wrote: Besides the obvious LT name, I've done some more digging that leads me to suspect that the issue is the version of libtool. I was able to install autoconf 2.59 and automake 1.9.6, but libtool 1.5.22 (8 years old) is not available. How should we proceed at this point? On Fri, Aug 16, 2013 at 10:28 AM, Miles Lubin miles.lu...@gmail.com wrote: I've been able to address all of the lintian issues except the outdated autotools files. The coinor packages use a highly nonstandard autotools work flow that's not compatible with autoreconf. They script they use to run autotools appears to be: https://projects.coin-or.org/BuildTools/browser/stable/0.7/run_autotools. I've tried to hack this script by installing the required versions of autoconf and automake, changing the executable names in the script, and changing the AUTOTOOLS_DIR variable from $HOME to /usr (!), but I'm not able to successfully run the script. The error (at the step of running autoconf) is: configure:323: error: possibly undefined macro: LT_LDFLAGS This variable is defined in CoinUtils/coin.m4. I would appreciate some tips to fix this, since I'm not very familiar with autotools. -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Bug#718893: RFS: coinutils/2.9.4-1 [ITA] -- CoinOR base library
Besides the obvious LT name, I've done some more digging that leads me to suspect that the issue is the version of libtool. I was able to install autoconf 2.59 and automake 1.9.6, but libtool 1.5.22 (8 years old) is not available. How should we proceed at this point? On Fri, Aug 16, 2013 at 10:28 AM, Miles Lubin miles.lu...@gmail.com wrote: I've been able to address all of the lintian issues except the outdated autotools files. The coinor packages use a highly nonstandard autotools work flow that's not compatible with autoreconf. They script they use to run autotools appears to be: https://projects.coin-or.org/BuildTools/browser/stable/0.7/run_autotools. I've tried to hack this script by installing the required versions of autoconf and automake, changing the executable names in the script, and changing the AUTOTOOLS_DIR variable from $HOME to /usr (!), but I'm not able to successfully run the script. The error (at the step of running autoconf) is: configure:323: error: possibly undefined macro: LT_LDFLAGS This variable is defined in CoinUtils/coin.m4. I would appreciate some tips to fix this, since I'm not very familiar with autotools. -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Bug#718893: RFS: coinutils/2.9.4-1 [ITA] -- CoinOR base library
I've been able to address all of the lintian issues except the outdated autotools files. The coinor packages use a highly nonstandard autotools work flow that's not compatible with autoreconf. They script they use to run autotools appears to be: https://projects.coin-or.org/BuildTools/browser/stable/0.7/run_autotools. I've tried to hack this script by installing the required versions of autoconf and automake, changing the executable names in the script, and changing the AUTOTOOLS_DIR variable from $HOME to /usr (!), but I'm not able to successfully run the script. The error (at the step of running autoconf) is: configure:323: error: possibly undefined macro: LT_LDFLAGS This variable is defined in CoinUtils/coin.m4. I would appreciate some tips to fix this, since I'm not very familiar with autotools. Thanks, Miles On Tue, Aug 13, 2013 at 9:40 AM, Sébastien Villemot sebast...@debian.org wrote: After a quick review of the package, I found the following issues that I would like you to fix before an upload: - make the package lintian clean, which means fix the 4 warnings. A few hints follow: + for the embedded javascript file, you should make the package depend on libjs-jquery, and replace the jquery.js file in your package with a symlink to the same file provided by libjs-jquery + for the debian/copyright file, the indentation of the paragraph describing the EPL is not correct; you should shift it right by one column, and replace empty lines by a space followed by a dot + for the outdated autotools files, you should regenerate them at build time using the dh-autoreconf helper - put the source of the package in a git repository on alioth under the debian-science tree, as explained in the Debian Science policy, and add the corresponding Vcs-* fields in the debian/control file. You need to create an alioth account and ask to join the Debian Science group there before being able to create the repository. - remove the obsolete README.source -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Bug#718893: RFS: coinutils/2.9.4-1 [ITA] -- CoinOR base library
Le mardi 13 août 2013 à 01:22 -0400, Miles Lubin a écrit : I've uploaded a new version with a best-effort attempt to conform to the debian-science guidelines. Please let me know of any issues. After a quick review of the package, I found the following issues that I would like you to fix before an upload: - make the package lintian clean, which means fix the 4 warnings. A few hints follow: + for the embedded javascript file, you should make the package depend on libjs-jquery, and replace the jquery.js file in your package with a symlink to the same file provided by libjs-jquery + for the debian/copyright file, the indentation of the paragraph describing the EPL is not correct; you should shift it right by one column, and replace empty lines by a space followed by a dot + for the outdated autotools files, you should regenerate them at build time using the dh-autoreconf helper - put the source of the package in a git repository on alioth under the debian-science tree, as explained in the Debian Science policy, and add the corresponding Vcs-* fields in the debian/control file. You need to create an alioth account and ask to join the Debian Science group there before being able to create the repository. - remove the obsolete README.source Don't hesitate to ask if you need help on any of these points. Best, -- .''`.Sébastien Villemot : :' :Debian Developer `. `' http://www.dynare.org/sebastien `- GPG Key: 4096R/381A7594 signature.asc Description: This is a digitally signed message part
Bug#718893: RFS: coinutils/2.9.4-1 [ITA] -- CoinOR base library
* Miles Lubin miles.lu...@gmail.com [130812 10:08]: Hi Etienne, Sébastien Villemot has kindly offered to sponsor the package as part of debian-science. Is that okay with you? Hi, Sure, please go ahead! -- Etienne Millon -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Bug#718893: RFS: coinutils/2.9.4-1 [ITA] -- CoinOR base library
Hi Sébastien, I've uploaded a new version with a best-effort attempt to conform to the debian-science guidelines. Please let me know of any issues. Thanks! On Mon, Aug 12, 2013 at 4:18 AM, Etienne Millon etienne.mil...@gmail.com wrote: * Miles Lubin miles.lu...@gmail.com [130812 10:08]: Hi Etienne, Sébastien Villemot has kindly offered to sponsor the package as part of debian-science. Is that okay with you? Hi, Sure, please go ahead! -- Etienne Millon -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Bug#718893: RFS: coinutils/2.9.4-1 [ITA] -- CoinOR base library
Hi Etienne, Sébastien Villemot has kindly offered to sponsor the package as part of debian-science. Is that okay with you? -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Bug#718893: RFS: coinutils/2.9.4-1 [ITA] -- CoinOR base library
* Sune Vuorela s...@debian.org [130807 16:36]: And in this specific case, the coinor build systems in general is so full of weirdnesses that just adapting it in smaller bits is much more feasable than adapting all of it. Oh I had almost forgotten that. That's a good point, thanks. Small packages seem the way to go. (It is however important not to use the convenience copies of code.) -- Etienne Millon -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Bug#718893: RFS: coinutils/2.9.4-1 [ITA] -- CoinOR base library
* Miles Lubin miles.lu...@gmail.com [130807 10:32]: Hi Etienne, Thanks for reviewing the package. I've uploaded a new version that should address your comments, and modified the changelog to target experimental. I'd like to push forward with the transition, if you're interested. Sure! A few of the coinor packages in the nice depgraph you generated are not essential to clp, cbc, or symphony, and unless someone has interest in maintaining them, it might be simpler to drop them in the transition. These are: coinor-vol, coinor-dylp, and coinor-flopc++. OSI can simply be built without depending on vol and dylp; in fact, these dependencies are not used at all by cbc or symphony. While we're talking about dependencies: it seems that every source tarball contains its dependencies. For example, cgl's tarball contains coinutils' source tree. There are tarballs containing the full distribution at: http://www.coin-or.org/download/source/CoinAll/ It seems to be a a little older (the last release is from january 2012), but it would be massively easier to maintain than having N source packages. What do you think? Do we need to file the transition request now, or may we start by updating everything in experimental? I believe it would be OK to upload in experimental. -- Etienne Millon -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Bug#718893: RFS: coinutils/2.9.4-1 [ITA] -- CoinOR base library
On Wednesday 07 August 2013 15:45:37 Etienne Millon wrote: There are tarballs containing the full distribution at: http://www.coin-or.org/download/source/CoinAll/ It seems to be a a little older (the last release is from january 2012), but it would be massively easier to maintain than having N source packages. What do you think? In the general case, it is normally much easier to maintain N smaller packages than one giant that combines everything. It makes 'release early, release often' much simpler since you can just push a fix to e.g. coinutils without having to also push a fix to cgl, cbc and others. Like the bugfix I pushed to cbc: http://packages.qa.debian.org/c/coinor-cbc/news/20110701T100215Z.html - much simpler just to push that than to push a entire new CoinAll. And in this specific case, the coinor build systems in general is so full of weirdnesses that just adapting it in smaller bits is much more feasable than adapting all of it. And given the different parts of coinor also have different release schedules it is also easier to provide the actual newest fixed bits with separate packages rather than large combined packages. So from my experience, both with coinor and with other large source packages, (KDE*, Qt*), separate sources is just the thing to do. /Sune -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Bug#718893: RFS: coinutils/2.9.4-1 [ITA] -- CoinOR base library
On Wed, Aug 7, 2013 at 10:34 AM, Sune Vuorela s...@debian.org wrote: And in this specific case, the coinor build systems in general is so full of weirdnesses that just adapting it in smaller bits is much more feasable than adapting all of it. And given the different parts of coinor also have different release schedules it is also easier to provide the actual newest fixed bits with separate packages rather than large combined packages. So from my experience, both with coinor and with other large source packages, (KDE*, Qt*), separate sources is just the thing to do. Thanks for sharing your experience with this. I agree that keeping the source packages separate is the better approach, especially given how different the release schedules are and how infrequently updated CoinAll is. -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Bug#718893: RFS: coinutils/2.9.4-1 [ITA] -- CoinOR base library
Package: sponsorship-requests Severity: normal Dear mentors, I am looking for a sponsor for my package coinutils * Package name: coinutils Version : 2.9.4-1 Upstream Author : J.P. Fasano, John J. Forrest, Lou Hafer, Laszlo Ladanyi, Francois Margot, Matt Saltzman, Ted Ralphs, Andreas Waechter * URL : https://projects.coin-or.org/CoinUtils * License : Eclipse Public License v 1.0 Section : science It builds those binary packages: coinor-libcoinutils-dev - Coin-or collection of utility classes coinor-libcoinutils-doc - Coin-or collection of utility classes coinor-libcoinutils3 - Coin-or collection of utility classes coinor-libcoinutils3-dbg - Coin-or collection of utility classes To access further information about this package, please visit the following URL: http://mentors.debian.net/package/coinutils Alternatively, one can download the package with dget using this command: dget -x http://mentors.debian.net/debian/pool/main/c/coinutils/coinutils_2.9.4-1.dsc This is a library of utilities used by various coinor-* packages for mathematical optimization (linear programming, etc.), many of which are RFA or O. Updating this package is the first step in updating its reverse dependencies, and as a first-time submitter I wanted feedback on this package before working on the others. Changes since the last upload: * New upstream version. - License changed to EPL. - Library version is now 3. * New maintainer (Closes: #645077) * Rewrite rules using dh, bump to compat 9. * Add patch to fix format security. Regards, Miles Lubin -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Bug#718893: RFS: coinutils/2.9.4-1 [ITA] -- CoinOR base library
Hello, Thanks for taking care of this package. Last year I tried to sort all these packages. In the process I adopted -cbc and ITA'd -cgl but I haven't had enough time to update them. At that time I documented the relations between the packages, and what's necessary to do on the following page: https://wiki.debian.org/CoinOr As you noted, the SONAME of libcoinutils changed from 0 to 3. This requires a transition, which needs to be coordinated with the release team (https://wiki.debian.org/Teams/ReleaseTeam/Transitions). This also means that the reverse-dependencies (-symphony, -flopc++, -osi, clp, -cgl and -cbc) need to be adapted so that they can use the new version. This can be done in experimental. As for the package itself (note that because of the SONAME bump it can't be uploaded to unstable as is), I have a few remarks: --- coinutils-2.6.4/debian/coinor-libcoinutils-dev.install2008-09-14 20:37:56.0 +0200 +++ coinutils-2.9.4/debian/coinor-libcoinutils-dev.install2013-08-06 05:10:15.0 +0200 @@ -1,4 +1,6 @@ debian/tmp/usr/include/coin -debian/tmp/usr/lib/lib*.a -debian/tmp/usr/lib/lib*.la -debian/tmp/usr/lib/lib*.so +debian/tmp/usr/share/coin +debian/tmp/usr/lib/*/lib*.a +debian/tmp/usr/lib/*/lib*.la +debian/tmp/usr/lib/*/lib*.so +debian/tmp/usr/lib/*/pkgconfig/*.pc You seem to be adding multiarch support. This should be mentioned in the changelog. Shipping *.la files is now deprecated, but it has to be fixed from the leaves of the graph so you can't do anything about it in this upload. https://wiki.debian.org/ReleaseGoals/LAFileRemoval -Standards-Version: 3.9.0 +Standards-Version: 3.9.4 This should be mentioned in the changelog along with the changes you made specifically for this (or no changes needed if appropriate). +export DH_VERBOSE=1 It shouldn't be set it directly in the rules file. I didn't try to build the package nor run lintian, since IMHO the most important thing is to solve the transition problem. Thanks again for putting energy in Coin-OR! -- Etienne Millon -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Bug#718893: RFS: coinutils/2.9.4-1 [ITA] -- CoinOR base library
Hi Etienne, Thanks for reviewing the package. I've uploaded a new version that should address your comments, and modified the changelog to target experimental. I'd like to push forward with the transition, if you're interested. A few of the coinor packages in the nice depgraph you generated are not essential to clp, cbc, or symphony, and unless someone has interest in maintaining them, it might be simpler to drop them in the transition. These are: coinor-vol, coinor-dylp, and coinor-flopc++. OSI can simply be built without depending on vol and dylp; in fact, these dependencies are not used at all by cbc or symphony. Do we need to file the transition request now, or may we start by updating everything in experimental? -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
