Bug#891750: marked as done (comet-ms: please add libraries to LIBS and not to CXXFLAGS)
Your message dated Fri, 02 Mar 2018 13:49:29 + with message-idand subject line Bug#891750: fixed in comet-ms 2017014-2 has caused the Debian Bug report #891750, regarding comet-ms: please add libraries to LIBS and not to CXXFLAGS 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 ow...@bugs.debian.org immediately.) -- 891750: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=891750 Debian Bug Tracking System Contact ow...@bugs.debian.org with problems --- Begin Message --- Package: comet-ms Version: 2017012-1 Severity: normal tags: patch Dear maintainer, Your Makefile has a little typo, that prevents the package from linking correctly when wl-asneeded flag is used (because libraries are stripped, since the .o file are at the bottom of the link line) please accept this patch, to solve the issue --- comet-ms-2017012/debian/Makefile2017-12-03 21:15:46.0 +0100 +++ comet-ms-2017012/debian/Makefile2018-02-28 15:02:03.0 +0100 @@ -1,8 +1,9 @@ CXX = g++ GCC = gcc -CXXFLAGS = -g -O3 -Wall -Wextra -Wno-char-subscripts -D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64 -D__LINUX__ -I/usr/include/libmstoolkit -lpthread -lmstoolkitlite +CXXFLAGS = -g -O3 -Wall -Wextra -Wno-char-subscripts -D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64 -D__LINUX__ -I/usr/include/libmstoolkit EXECNAME = comet-ms +LIBS = -lmstoolkitlite -lpthread COMETSEARCH = Comet.o Threading.o CometInterfaces.o CometSearch.o CometPreprocess.o CometPostAnalysis.o CometMassSpecUtils.o CometWriteOut.o CometWriteSqt.o CometWritePepXML.o CometWritePercolator.o CometWriteTxt.o CometSearchManager.o you can also grab the patch from here: http://launchpadlibrarian.net/358943993/comet-ms_2017012-1_2017012-1ubuntu1.diff.gz thanks Gianfranco signature.asc Description: OpenPGP digital signature --- End Message --- --- Begin Message --- Source: comet-ms Source-Version: 2017014-2 We believe that the bug you reported is fixed in the latest version of comet-ms, which is due to be installed in the Debian FTP archive. A summary of the changes between this version and the previous one is attached. Thank you for reporting the bug, which will now be closed. If you have further comments please address them to 891...@bugs.debian.org, and the maintainer will reopen the bug report if appropriate. Debian distribution maintenance software pp. Filippo Rusconi (supplier of updated comet-ms package) (This message was generated automatically at their request; if you believe that there is a problem with it please contact the archive administrators by mailing ftpmas...@ftp-master.debian.org) -BEGIN PGP SIGNED MESSAGE- Hash: SHA256 Format: 1.8 Date: Fri, 02 Mar 2018 11:34:35 +0100 Source: comet-ms Binary: comet-ms Architecture: source amd64 Version: 2017014-2 Distribution: unstable Urgency: medium Maintainer: The Debichem Group Changed-By: Filippo Rusconi Description: comet-ms - Tandem mass spectrometry (MS/MS) search engine Closes: 891750 Changes: comet-ms (2017014-2) unstable; urgency=medium . * Fix problem with library linking in the Makefile. Thanks to Gianfranco Costamagna for re-reporting that bug (Closes: #891750). In fact, he saw that the rules duplicated code and the Makefile such that the Makefile did not direct compilation, since that compilation was local. Checksums-Sha1: 1ca792a0555787f512678a089a0148e67c9f8b42 2053 comet-ms_2017014-2.dsc ebc05ba3872266008f4cac1e5f6efe627cc74211 5932 comet-ms_2017014-2.debian.tar.xz 6a2ddcf80d4fce3e6264232595589e192793e8f0 1046504 comet-ms-dbgsym_2017014-2_amd64.deb 91a38933433a55c902aba24967c56eaa71bd1088 6164 comet-ms_2017014-2_amd64.buildinfo 886055d2ce6667fe10975c540731afa8b0b2b39b 851548 comet-ms_2017014-2_amd64.deb Checksums-Sha256: 203f9a56db6188c52a0216f68a5443f622d3bdb0f77b704cc1068b0cf7ace955 2053 comet-ms_2017014-2.dsc 32eeab4f078ec1c553fa1b0974e5d3ee43013f4399b5736c55ad83f49421c708 5932 comet-ms_2017014-2.debian.tar.xz e2777870c86c44cdca65eab404abc1f0482ab3e72fdb88c4f8c01c6d8ff9e9f6 1046504 comet-ms-dbgsym_2017014-2_amd64.deb 8c246319d98beb3be96d052becc83bb66f7edfa9e9a000a5ee3ac027d78432b6 6164 comet-ms_2017014-2_amd64.buildinfo bb575ea348a15b50383b8a87333dc12bed5b01b74169883a9abe42d153585ac6 851548 comet-ms_2017014-2_amd64.deb Files: abaac9e7bb4aa88837accafd502d5b0c 2053 science optional comet-ms_2017014-2.dsc 086b457d119e6bbed2af2b9d26214605 5932 science optional comet-ms_2017014-2.debian.tar.xz 0e7b3b76636ed2267dca18c74d81c442 1046504
Bug#891750: marked as done (comet-ms: please add libraries to LIBS and not to CXXFLAGS)
Your message dated Thu, 01 Mar 2018 12:19:29 + with message-idand subject line Bug#891750: fixed in comet-ms 2017014-1 has caused the Debian Bug report #891750, regarding comet-ms: please add libraries to LIBS and not to CXXFLAGS 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 ow...@bugs.debian.org immediately.) -- 891750: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=891750 Debian Bug Tracking System Contact ow...@bugs.debian.org with problems --- Begin Message --- Package: comet-ms Version: 2017012-1 Severity: normal tags: patch Dear maintainer, Your Makefile has a little typo, that prevents the package from linking correctly when wl-asneeded flag is used (because libraries are stripped, since the .o file are at the bottom of the link line) please accept this patch, to solve the issue --- comet-ms-2017012/debian/Makefile2017-12-03 21:15:46.0 +0100 +++ comet-ms-2017012/debian/Makefile2018-02-28 15:02:03.0 +0100 @@ -1,8 +1,9 @@ CXX = g++ GCC = gcc -CXXFLAGS = -g -O3 -Wall -Wextra -Wno-char-subscripts -D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64 -D__LINUX__ -I/usr/include/libmstoolkit -lpthread -lmstoolkitlite +CXXFLAGS = -g -O3 -Wall -Wextra -Wno-char-subscripts -D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64 -D__LINUX__ -I/usr/include/libmstoolkit EXECNAME = comet-ms +LIBS = -lmstoolkitlite -lpthread COMETSEARCH = Comet.o Threading.o CometInterfaces.o CometSearch.o CometPreprocess.o CometPostAnalysis.o CometMassSpecUtils.o CometWriteOut.o CometWriteSqt.o CometWritePepXML.o CometWritePercolator.o CometWriteTxt.o CometSearchManager.o you can also grab the patch from here: http://launchpadlibrarian.net/358943993/comet-ms_2017012-1_2017012-1ubuntu1.diff.gz thanks Gianfranco signature.asc Description: OpenPGP digital signature --- End Message --- --- Begin Message --- Source: comet-ms Source-Version: 2017014-1 We believe that the bug you reported is fixed in the latest version of comet-ms, which is due to be installed in the Debian FTP archive. A summary of the changes between this version and the previous one is attached. Thank you for reporting the bug, which will now be closed. If you have further comments please address them to 891...@bugs.debian.org, and the maintainer will reopen the bug report if appropriate. Debian distribution maintenance software pp. Filippo Rusconi (supplier of updated comet-ms package) (This message was generated automatically at their request; if you believe that there is a problem with it please contact the archive administrators by mailing ftpmas...@ftp-master.debian.org) -BEGIN PGP SIGNED MESSAGE- Hash: SHA256 Format: 1.8 Date: Thu, 01 Mar 2018 11:37:59 +0100 Source: comet-ms Binary: comet-ms Architecture: source amd64 Version: 2017014-1 Distribution: unstable Urgency: medium Maintainer: The Debichem Group Changed-By: Filippo Rusconi Description: comet-ms - Tandem mass spectrometry (MS/MS) search engine Closes: 891750 Changes: comet-ms (2017014-1) unstable; urgency=medium . * New upstream version. . * Updated the patch that modifies the two Makefile_s such that libmstoolkitlite is used from the Debian package. Tested the comet-ms binary ok. . * Fix problem with library linking in the Makefile. Thanks to Gianfranco Costamagna for reporting that bug (Closes: #891750). Checksums-Sha1: 927ad9069a6a8ca04f8fd4ad8ebba635e45b6ee0 2053 comet-ms_2017014-1.dsc 2d1aceb99b7cf644b583867afeba402beb96a581 1168205 comet-ms_2017014.orig.tar.gz 8d8889af74b291821bce8a4b522c8b8d864d3159 5864 comet-ms_2017014-1.debian.tar.xz b3156da28ee1546dbafee5e1c69133f8744bf0a4 1008804 comet-ms-dbgsym_2017014-1_amd64.deb 80b29d993525b3a4f49c863bf4c743fc1a1b014e 6200 comet-ms_2017014-1_amd64.buildinfo 15cd09e7a6edb17e11cecc18398a0e072b77e6e5 841816 comet-ms_2017014-1_amd64.deb Checksums-Sha256: 4c40b11f1fd62c91cc84a8f38b10ecc9d2b880b8d58e5aab3e1c3ab2c9070a8a 2053 comet-ms_2017014-1.dsc 214a0bc53358ebd07a473ab98e9bcfb47903dc756faa006aaf8c63a29e548428 1168205 comet-ms_2017014.orig.tar.gz 523393a10d5e924d84cfc4250953f9b2f87dd821442043261691358fe778ce3a 5864 comet-ms_2017014-1.debian.tar.xz c916762e20c10d611d00b4e61e87035f06348350ecb47070047f439ef319a3be 1008804 comet-ms-dbgsym_2017014-1_amd64.deb 799cd6856d711edde1c184260825683903f1330681bd785cef64267a0ce76238 6200 comet-ms_2017014-1_amd64.buildinfo e2b7f9194c4d2dc4d73a26ef0abd00e4aad1de04d061ca4d79f580f000c8e352 841816 comet-ms_2017014-1_amd64.deb Files: