From: Mantis B. T. <no...@bu...> - 2012-05-19 12:34:52
|
The following issue has been SUBMITTED. ====================================================================== http://bugs.bacula.org/view.php?id=1871 ====================================================================== Reported By: lu...@pc... Assigned To: ====================================================================== Project: bacula Issue ID: 1871 Category: configure/build process Reproducibility: always Severity: minor Priority: normal Status: new ====================================================================== Date Submitted: 2012-05-19 13:34 BST Last Modified: 2012-05-19 13:34 BST ====================================================================== Summary: readline support not available without TERM_LIB Description: This bug has been reported in Debian by Sven Joachim [1]. [1] <http://bugs.debian.org/646730> It is impossible to build the bconsole readline support in Bacula_5.0.2+ without the TERM_LIB. I know what the Bacula manual says about readline: - System Requirements [2] * If you want to enable command line editing and history, you will need to have /usr/include/termcap.h and either the termcap or the ncurses library loaded (libtermcap-devel or ncurses-devel). [2] <http://www.bacula.org/5.2.x-manuals/en/main/main/System_Requirements.html> - Building Bacula from Source [3] The --enable-conio or --enable-readline options are useful because they provide a command line history and editing capability for the Console program. If you have included either option in the build, either the termcap or the ncurses package will be needed to link. [...] readline is no longer supported after version 1.34. The code within Bacula remains, so it should be usable, and if users submit patches for it, we will be happy to apply them. However, due to the fact that each version of readline seems to be incompatible with previous versions, and that there are significant differences between systems, we can no longer afford to support it. [3] <http://www.bacula.org/manuals/en/install/install/Installing_Bacula.html> However, the readline support is never available if you do not have TERM_LIB because of [4], which basically put conio and readline at the same level in terms of dependencies. I was not able to find any reason for this change in the ChangeLog neither in the BTS and similar bugs were either too old (http://bugs.bacula.org/view.php?id=743 [5]) or not related (http://bugs.bacula.org/view.php?id=1226 [6]). [4] <http://www.bacula.org/git/cgit.cgi/bacula/commit/?h=Branch-5.0&id=967d4334e69d2ab739a2f270f2df1bcfb0ffc7d4> [5] <http://bugs.bacula.org/view.php?id=743> [6] <http://bugs.bacula.org/view.php?id=1226> However, it seems that the readline support does not need any TERM_LIB to be built, given that according to build logs [7][8] /usr/sbin/bacula-console (this is upstream bconsole, see [9] for the future fix in Debian) does not use any symbols from libhistory.so.6, nor libtinfo.so.5 neither libncurses.so.5. [7] <http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=646730#31> [8] <https://buildd.debian.org/status/fetch.php?pkg=bacula&arch=amd64&ver=5.0.3%2Bdfsg-0.1&stamp=1335779247> [9] <http://anonscm.debian.org/gitweb/?p=pkg-bacula/bacula.git;a=commitdiff;h=2cdfcfec4764ae6cde77c44f474bd521f7562e1d> With the attached patch provided by Alexander Golovko [10] the Debian packages are built with: ===== $ cd /tmp/buildd/bacula-5.0.3+dfsgreal/debian/tmp-build-sqlite3 && \ QMAKE=/usr/bin/qmake-qt4 ./configure --config-cache \ --host=x86_64-linux-gnu --build=x86_64-linux-gnu \ --prefix=/usr \ --with-archivedir=/nonexistant/path/to/file/archive/dir \ --sysconfdir=/etc/bacula --with-scriptdir=/etc/bacula/scripts \ --sharedstatedir=/var/lib/bacula \ --localstatedir=/var/lib/bacula \ --with-pid-dir=/var/run/bacula --with-smtp-host=localhost \ --with-working-dir=/var/lib/bacula \ --with-subsys-dir=/var/lock \ --mandir=\${prefix}/share/man \ --infodir=\${prefix}/share/info \ --enable-smartalloc --with-python --with-tcp-wrappers --with-openssl --with-libiconv-prefix=/usr/include --with-readline=/usr/include/readline --disable-conio --with-libintl-prefix=/usr/include --with-x --docdir=\${prefix}/share/doc/bacula-common --htmldir=\${prefix}/share/doc/bacula-common/html --libdir=\${prefix}/lib/bacula --enable-batch-insert --disable-bwx-console --without-qwt --enable-ipv6 --with-dir-password=XXX_DIRPASSWORD_XXX --with-fd-password=XXX_FDPASSWORD_XXX --with-sd-password=XXX_SDPASSWORD_XXX --with-mon-dir-password=XXX_MONDIRPASSWORD_XXX --with-mon-fd-password=XXX_MONFDPASSWORD_XXX --with-mon-sd-password=XXX_MONSDPASSWORD_XXX --with-db-name=XXX_DBNAME_XXX --with-db-user=XXX_DBUSER_XXX --with-db-password=XXX_DBPASSWORD_XXX --with-sqlite3 --without-mysql --without-postgresql --without-sqlite --enable-tray-monitor --enable-bat [...] Configuration on Fri May 18 19:54:40 UTC 2012: Host: x86_64-pc-linux-gnu -- debian wheezy/sid Bacula version: Bacula 5.0.3 (04 August 2010) [...] SQL binaries Directory /usr/bin Large file support: yes Bacula conio support: no -lreadline -lhistory readline support: yes TCP Wrappers support: yes -lwrap TLS support: yes Encryption support: yes ZLIB support: yes [...] ===== [10] <http://anonscm.debian.org/gitweb/?p=pkg-bacula/bacula.git;a=commitdiff;h=488241e3d2c90af4011597c652670d3c74e95cbe> dpkg-shlibdeps still complains about unused symbols, but this time only for libhistory.so.6: ===== dpkg-shlibdeps: warning: package could avoid a useless dependency if debian/bacula-console/usr/sbin/bacula-console was not linked against libdl.so.2 (it uses none of the library's symbols). dpkg-shlibdeps: warning: package could avoid a useless dependency if debian/bacula-console/usr/sbin/bacula-console was not linked against libz.so.1 (it uses none of the library's symbols). dpkg-shlibdeps: warning: package could avoid a useless dependency if debian/bacula-console/usr/sbin/bacula-console was not linked against libhistory.so.6 (it uses none of the library's symbols). dpkg-shlibdeps: warning: package could avoid a useless dependency if debian/bacula-console/usr/sbin/bacula-console was not linked against libcrypto.so.1.0.0 (it uses none of the library's symbols). dpkg-shlibdeps: warning: package could avoid a useless dependency if debian/bacula-console/usr/sbin/bacula-console was not linked against libwrap.so.0 (it uses none of the library's symbols). dpkg-shlibdeps: warning: package could avoid a useless dependency if debian/bacula-console/usr/sbin/bacula-console was not linked against libssl.so.1.0.0 (it uses none of the library's symbols). ===== The resulting bacula-console binary works as expected (CTRL-UP/DOWN and TAB-COMPLETION) against a 5.0.2 Director. Is there something I am missing? ====================================================================== Issue History Date Modified Username Field Change ====================================================================== 2012-05-19 13:34 lu...@pc... New Issue 2012-05-19 13:34 lu...@pc... File Added: fix-readline-ncurses-depends.patch ====================================================================== |
From: Mantis B. T. <no...@bu...> - 2012-05-24 06:34:12
|
The following issue has been ASSIGNED. ====================================================================== http://bugs.bacula.org/view.php?id=1871 ====================================================================== Reported By: lu...@pc... Assigned To: kern ====================================================================== Project: bacula Issue ID: 1871 Category: configure/build process Reproducibility: always Severity: minor Priority: normal Status: acknowledged ====================================================================== Date Submitted: 2012-05-19 13:34 BST Last Modified: 2012-05-24 07:34 BST ====================================================================== Summary: readline support not available without TERM_LIB Description: This bug has been reported in Debian by Sven Joachim [1]. [1] <http://bugs.debian.org/646730> It is impossible to build the bconsole readline support in Bacula_5.0.2+ without the TERM_LIB. I know what the Bacula manual says about readline: - System Requirements [2] * If you want to enable command line editing and history, you will need to have /usr/include/termcap.h and either the termcap or the ncurses library loaded (libtermcap-devel or ncurses-devel). [2] <http://www.bacula.org/5.2.x-manuals/en/main/main/System_Requirements.html> - Building Bacula from Source [3] The --enable-conio or --enable-readline options are useful because they provide a command line history and editing capability for the Console program. If you have included either option in the build, either the termcap or the ncurses package will be needed to link. [...] readline is no longer supported after version 1.34. The code within Bacula remains, so it should be usable, and if users submit patches for it, we will be happy to apply them. However, due to the fact that each version of readline seems to be incompatible with previous versions, and that there are significant differences between systems, we can no longer afford to support it. [3] <http://www.bacula.org/manuals/en/install/install/Installing_Bacula.html> However, the readline support is never available if you do not have TERM_LIB because of [4], which basically put conio and readline at the same level in terms of dependencies. I was not able to find any reason for this change in the ChangeLog neither in the BTS and similar bugs were either too old (http://bugs.bacula.org/view.php?id=743 [5]) or not related (http://bugs.bacula.org/view.php?id=1226 [6]). [4] <http://www.bacula.org/git/cgit.cgi/bacula/commit/?h=Branch-5.0&id=967d4334e69d2ab739a2f270f2df1bcfb0ffc7d4> [5] <http://bugs.bacula.org/view.php?id=743> [6] <http://bugs.bacula.org/view.php?id=1226> However, it seems that the readline support does not need any TERM_LIB to be built, given that according to build logs [7][8] /usr/sbin/bacula-console (this is upstream bconsole, see [9] for the future fix in Debian) does not use any symbols from libhistory.so.6, nor libtinfo.so.5 neither libncurses.so.5. [7] <http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=646730#31> [8] <https://buildd.debian.org/status/fetch.php?pkg=bacula&arch=amd64&ver=5.0.3%2Bdfsg-0.1&stamp=1335779247> [9] <http://anonscm.debian.org/gitweb/?p=pkg-bacula/bacula.git;a=commitdiff;h=2cdfcfec4764ae6cde77c44f474bd521f7562e1d> With the attached patch provided by Alexander Golovko [10] the Debian packages are built with: ===== $ cd /tmp/buildd/bacula-5.0.3+dfsgreal/debian/tmp-build-sqlite3 && \ QMAKE=/usr/bin/qmake-qt4 ./configure --config-cache \ --host=x86_64-linux-gnu --build=x86_64-linux-gnu \ --prefix=/usr \ --with-archivedir=/nonexistant/path/to/file/archive/dir \ --sysconfdir=/etc/bacula --with-scriptdir=/etc/bacula/scripts \ --sharedstatedir=/var/lib/bacula \ --localstatedir=/var/lib/bacula \ --with-pid-dir=/var/run/bacula --with-smtp-host=localhost \ --with-working-dir=/var/lib/bacula \ --with-subsys-dir=/var/lock \ --mandir=\${prefix}/share/man \ --infodir=\${prefix}/share/info \ --enable-smartalloc --with-python --with-tcp-wrappers --with-openssl --with-libiconv-prefix=/usr/include --with-readline=/usr/include/readline --disable-conio --with-libintl-prefix=/usr/include --with-x --docdir=\${prefix}/share/doc/bacula-common --htmldir=\${prefix}/share/doc/bacula-common/html --libdir=\${prefix}/lib/bacula --enable-batch-insert --disable-bwx-console --without-qwt --enable-ipv6 --with-dir-password=XXX_DIRPASSWORD_XXX --with-fd-password=XXX_FDPASSWORD_XXX --with-sd-password=XXX_SDPASSWORD_XXX --with-mon-dir-password=XXX_MONDIRPASSWORD_XXX --with-mon-fd-password=XXX_MONFDPASSWORD_XXX --with-mon-sd-password=XXX_MONSDPASSWORD_XXX --with-db-name=XXX_DBNAME_XXX --with-db-user=XXX_DBUSER_XXX --with-db-password=XXX_DBPASSWORD_XXX --with-sqlite3 --without-mysql --without-postgresql --without-sqlite --enable-tray-monitor --enable-bat [...] Configuration on Fri May 18 19:54:40 UTC 2012: Host: x86_64-pc-linux-gnu -- debian wheezy/sid Bacula version: Bacula 5.0.3 (04 August 2010) [...] SQL binaries Directory /usr/bin Large file support: yes Bacula conio support: no -lreadline -lhistory readline support: yes TCP Wrappers support: yes -lwrap TLS support: yes Encryption support: yes ZLIB support: yes [...] ===== [10] <http://anonscm.debian.org/gitweb/?p=pkg-bacula/bacula.git;a=commitdiff;h=488241e3d2c90af4011597c652670d3c74e95cbe> dpkg-shlibdeps still complains about unused symbols, but this time only for libhistory.so.6: ===== dpkg-shlibdeps: warning: package could avoid a useless dependency if debian/bacula-console/usr/sbin/bacula-console was not linked against libdl.so.2 (it uses none of the library's symbols). dpkg-shlibdeps: warning: package could avoid a useless dependency if debian/bacula-console/usr/sbin/bacula-console was not linked against libz.so.1 (it uses none of the library's symbols). dpkg-shlibdeps: warning: package could avoid a useless dependency if debian/bacula-console/usr/sbin/bacula-console was not linked against libhistory.so.6 (it uses none of the library's symbols). dpkg-shlibdeps: warning: package could avoid a useless dependency if debian/bacula-console/usr/sbin/bacula-console was not linked against libcrypto.so.1.0.0 (it uses none of the library's symbols). dpkg-shlibdeps: warning: package could avoid a useless dependency if debian/bacula-console/usr/sbin/bacula-console was not linked against libwrap.so.0 (it uses none of the library's symbols). dpkg-shlibdeps: warning: package could avoid a useless dependency if debian/bacula-console/usr/sbin/bacula-console was not linked against libssl.so.1.0.0 (it uses none of the library's symbols). ===== The resulting bacula-console binary works as expected (CTRL-UP/DOWN and TAB-COMPLETION) against a 5.0.2 Director. Is there something I am missing? ====================================================================== ---------------------------------------------------------------------- (0006326) kern (administrator) - 2012-05-24 07:34 http://bugs.bacula.org/view.php?id=1871#c6326 ---------------------------------------------------------------------- Thanks for the patch. This patch looks quite reasonable to me. We will test it and unless some problems come up, highly unlikely given your testing, we will include it in the next release. I will leave the bug report open until the patch is committed. Issue History Date Modified Username Field Change ====================================================================== 2012-05-19 13:34 lu...@pc... New Issue 2012-05-19 13:34 lu...@pc... File Added: fix-readline-ncurses-depends.patch 2012-05-24 07:34 kern Note Added: 0006326 2012-05-24 07:34 kern Assigned To => kern 2012-05-24 07:34 kern Status new => acknowledged ====================================================================== |
From: Mantis B. T. <no...@bu...> - 2012-06-10 08:00:01
|
The following issue has been CLOSED ====================================================================== http://bugs.bacula.org/view.php?id=1871 ====================================================================== Reported By: lu...@pc... Assigned To: kern ====================================================================== Project: bacula Issue ID: 1871 Category: configure/build process Reproducibility: always Severity: minor Priority: normal Status: closed Resolution: fixed Fixed in Version: 5.2.8 ====================================================================== Date Submitted: 2012-05-19 13:34 BST Last Modified: 2012-06-10 08:59 BST ====================================================================== Summary: readline support not available without TERM_LIB Description: This bug has been reported in Debian by Sven Joachim [1]. [1] <http://bugs.debian.org/646730> It is impossible to build the bconsole readline support in Bacula_5.0.2+ without the TERM_LIB. I know what the Bacula manual says about readline: - System Requirements [2] * If you want to enable command line editing and history, you will need to have /usr/include/termcap.h and either the termcap or the ncurses library loaded (libtermcap-devel or ncurses-devel). [2] <http://www.bacula.org/5.2.x-manuals/en/main/main/System_Requirements.html> - Building Bacula from Source [3] The --enable-conio or --enable-readline options are useful because they provide a command line history and editing capability for the Console program. If you have included either option in the build, either the termcap or the ncurses package will be needed to link. [...] readline is no longer supported after version 1.34. The code within Bacula remains, so it should be usable, and if users submit patches for it, we will be happy to apply them. However, due to the fact that each version of readline seems to be incompatible with previous versions, and that there are significant differences between systems, we can no longer afford to support it. [3] <http://www.bacula.org/manuals/en/install/install/Installing_Bacula.html> However, the readline support is never available if you do not have TERM_LIB because of [4], which basically put conio and readline at the same level in terms of dependencies. I was not able to find any reason for this change in the ChangeLog neither in the BTS and similar bugs were either too old (http://bugs.bacula.org/view.php?id=743 [5]) or not related (http://bugs.bacula.org/view.php?id=1226 [6]). [4] <http://www.bacula.org/git/cgit.cgi/bacula/commit/?h=Branch-5.0&id=967d4334e69d2ab739a2f270f2df1bcfb0ffc7d4> [5] <http://bugs.bacula.org/view.php?id=743> [6] <http://bugs.bacula.org/view.php?id=1226> However, it seems that the readline support does not need any TERM_LIB to be built, given that according to build logs [7][8] /usr/sbin/bacula-console (this is upstream bconsole, see [9] for the future fix in Debian) does not use any symbols from libhistory.so.6, nor libtinfo.so.5 neither libncurses.so.5. [7] <http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=646730#31> [8] <https://buildd.debian.org/status/fetch.php?pkg=bacula&arch=amd64&ver=5.0.3%2Bdfsg-0.1&stamp=1335779247> [9] <http://anonscm.debian.org/gitweb/?p=pkg-bacula/bacula.git;a=commitdiff;h=2cdfcfec4764ae6cde77c44f474bd521f7562e1d> With the attached patch provided by Alexander Golovko [10] the Debian packages are built with: ===== $ cd /tmp/buildd/bacula-5.0.3+dfsgreal/debian/tmp-build-sqlite3 && \ QMAKE=/usr/bin/qmake-qt4 ./configure --config-cache \ --host=x86_64-linux-gnu --build=x86_64-linux-gnu \ --prefix=/usr \ --with-archivedir=/nonexistant/path/to/file/archive/dir \ --sysconfdir=/etc/bacula --with-scriptdir=/etc/bacula/scripts \ --sharedstatedir=/var/lib/bacula \ --localstatedir=/var/lib/bacula \ --with-pid-dir=/var/run/bacula --with-smtp-host=localhost \ --with-working-dir=/var/lib/bacula \ --with-subsys-dir=/var/lock \ --mandir=\${prefix}/share/man \ --infodir=\${prefix}/share/info \ --enable-smartalloc --with-python --with-tcp-wrappers --with-openssl --with-libiconv-prefix=/usr/include --with-readline=/usr/include/readline --disable-conio --with-libintl-prefix=/usr/include --with-x --docdir=\${prefix}/share/doc/bacula-common --htmldir=\${prefix}/share/doc/bacula-common/html --libdir=\${prefix}/lib/bacula --enable-batch-insert --disable-bwx-console --without-qwt --enable-ipv6 --with-dir-password=XXX_DIRPASSWORD_XXX --with-fd-password=XXX_FDPASSWORD_XXX --with-sd-password=XXX_SDPASSWORD_XXX --with-mon-dir-password=XXX_MONDIRPASSWORD_XXX --with-mon-fd-password=XXX_MONFDPASSWORD_XXX --with-mon-sd-password=XXX_MONSDPASSWORD_XXX --with-db-name=XXX_DBNAME_XXX --with-db-user=XXX_DBUSER_XXX --with-db-password=XXX_DBPASSWORD_XXX --with-sqlite3 --without-mysql --without-postgresql --without-sqlite --enable-tray-monitor --enable-bat [...] Configuration on Fri May 18 19:54:40 UTC 2012: Host: x86_64-pc-linux-gnu -- debian wheezy/sid Bacula version: Bacula 5.0.3 (04 August 2010) [...] SQL binaries Directory /usr/bin Large file support: yes Bacula conio support: no -lreadline -lhistory readline support: yes TCP Wrappers support: yes -lwrap TLS support: yes Encryption support: yes ZLIB support: yes [...] ===== [10] <http://anonscm.debian.org/gitweb/?p=pkg-bacula/bacula.git;a=commitdiff;h=488241e3d2c90af4011597c652670d3c74e95cbe> dpkg-shlibdeps still complains about unused symbols, but this time only for libhistory.so.6: ===== dpkg-shlibdeps: warning: package could avoid a useless dependency if debian/bacula-console/usr/sbin/bacula-console was not linked against libdl.so.2 (it uses none of the library's symbols). dpkg-shlibdeps: warning: package could avoid a useless dependency if debian/bacula-console/usr/sbin/bacula-console was not linked against libz.so.1 (it uses none of the library's symbols). dpkg-shlibdeps: warning: package could avoid a useless dependency if debian/bacula-console/usr/sbin/bacula-console was not linked against libhistory.so.6 (it uses none of the library's symbols). dpkg-shlibdeps: warning: package could avoid a useless dependency if debian/bacula-console/usr/sbin/bacula-console was not linked against libcrypto.so.1.0.0 (it uses none of the library's symbols). dpkg-shlibdeps: warning: package could avoid a useless dependency if debian/bacula-console/usr/sbin/bacula-console was not linked against libwrap.so.0 (it uses none of the library's symbols). dpkg-shlibdeps: warning: package could avoid a useless dependency if debian/bacula-console/usr/sbin/bacula-console was not linked against libssl.so.1.0.0 (it uses none of the library's symbols). ===== The resulting bacula-console binary works as expected (CTRL-UP/DOWN and TAB-COMPLETION) against a 5.0.2 Director. Is there something I am missing? ====================================================================== ---------------------------------------------------------------------- (0006326) kern (administrator) - 2012-05-24 07:34 http://bugs.bacula.org/view.php?id=1871#c6326 ---------------------------------------------------------------------- Thanks for the patch. This patch looks quite reasonable to me. We will test it and unless some problems come up, highly unlikely given your testing, we will include it in the next release. I will leave the bug report open until the patch is committed. ---------------------------------------------------------------------- (0006383) kern (administrator) - 2012-06-10 08:59 http://bugs.bacula.org/view.php?id=1871#c6383 ---------------------------------------------------------------------- Thanks again for the patch. It is applied and will be available in version 5.2.8 to be released today or tomorrow. Issue History Date Modified Username Field Change ====================================================================== 2012-05-19 13:34 lu...@pc... New Issue 2012-05-19 13:34 lu...@pc... File Added: fix-readline-ncurses-depends.patch 2012-05-24 07:34 kern Note Added: 0006326 2012-05-24 07:34 kern Assigned To => kern 2012-05-24 07:34 kern Status new => acknowledged 2012-06-10 08:59 kern Note Added: 0006383 2012-06-10 08:59 kern Status acknowledged => closed 2012-06-10 08:59 kern Resolution open => fixed 2012-06-10 08:59 kern Fixed in Version => 5.2.8 ====================================================================== |
From: Mantis B. T. <no...@bu...> - 2012-06-11 11:57:01
|
A NOTE has been added to this issue. ====================================================================== http://bugs.bacula.org/view.php?id=1871 ====================================================================== Reported By: lu...@pc... Assigned To: kern ====================================================================== Project: bacula Issue ID: 1871 Category: configure/build process Reproducibility: always Severity: minor Priority: normal Status: feedback ====================================================================== Date Submitted: 2012-05-19 13:34 BST Last Modified: 2012-06-11 12:56 BST ====================================================================== Summary: readline support not available without TERM_LIB Description: This bug has been reported in Debian by Sven Joachim [1]. [1] <http://bugs.debian.org/646730> It is impossible to build the bconsole readline support in Bacula_5.0.2+ without the TERM_LIB. I know what the Bacula manual says about readline: - System Requirements [2] * If you want to enable command line editing and history, you will need to have /usr/include/termcap.h and either the termcap or the ncurses library loaded (libtermcap-devel or ncurses-devel). [2] <http://www.bacula.org/5.2.x-manuals/en/main/main/System_Requirements.html> - Building Bacula from Source [3] The --enable-conio or --enable-readline options are useful because they provide a command line history and editing capability for the Console program. If you have included either option in the build, either the termcap or the ncurses package will be needed to link. [...] readline is no longer supported after version 1.34. The code within Bacula remains, so it should be usable, and if users submit patches for it, we will be happy to apply them. However, due to the fact that each version of readline seems to be incompatible with previous versions, and that there are significant differences between systems, we can no longer afford to support it. [3] <http://www.bacula.org/manuals/en/install/install/Installing_Bacula.html> However, the readline support is never available if you do not have TERM_LIB because of [4], which basically put conio and readline at the same level in terms of dependencies. I was not able to find any reason for this change in the ChangeLog neither in the BTS and similar bugs were either too old (http://bugs.bacula.org/view.php?id=743 [5]) or not related (http://bugs.bacula.org/view.php?id=1226 [6]). [4] <http://www.bacula.org/git/cgit.cgi/bacula/commit/?h=Branch-5.0&id=967d4334e69d2ab739a2f270f2df1bcfb0ffc7d4> [5] <http://bugs.bacula.org/view.php?id=743> [6] <http://bugs.bacula.org/view.php?id=1226> However, it seems that the readline support does not need any TERM_LIB to be built, given that according to build logs [7][8] /usr/sbin/bacula-console (this is upstream bconsole, see [9] for the future fix in Debian) does not use any symbols from libhistory.so.6, nor libtinfo.so.5 neither libncurses.so.5. [7] <http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=646730#31> [8] <https://buildd.debian.org/status/fetch.php?pkg=bacula&arch=amd64&ver=5.0.3%2Bdfsg-0.1&stamp=1335779247> [9] <http://anonscm.debian.org/gitweb/?p=pkg-bacula/bacula.git;a=commitdiff;h=2cdfcfec4764ae6cde77c44f474bd521f7562e1d> With the attached patch provided by Alexander Golovko [10] the Debian packages are built with: ===== $ cd /tmp/buildd/bacula-5.0.3+dfsgreal/debian/tmp-build-sqlite3 && \ QMAKE=/usr/bin/qmake-qt4 ./configure --config-cache \ --host=x86_64-linux-gnu --build=x86_64-linux-gnu \ --prefix=/usr \ --with-archivedir=/nonexistant/path/to/file/archive/dir \ --sysconfdir=/etc/bacula --with-scriptdir=/etc/bacula/scripts \ --sharedstatedir=/var/lib/bacula \ --localstatedir=/var/lib/bacula \ --with-pid-dir=/var/run/bacula --with-smtp-host=localhost \ --with-working-dir=/var/lib/bacula \ --with-subsys-dir=/var/lock \ --mandir=\${prefix}/share/man \ --infodir=\${prefix}/share/info \ --enable-smartalloc --with-python --with-tcp-wrappers --with-openssl --with-libiconv-prefix=/usr/include --with-readline=/usr/include/readline --disable-conio --with-libintl-prefix=/usr/include --with-x --docdir=\${prefix}/share/doc/bacula-common --htmldir=\${prefix}/share/doc/bacula-common/html --libdir=\${prefix}/lib/bacula --enable-batch-insert --disable-bwx-console --without-qwt --enable-ipv6 --with-dir-password=XXX_DIRPASSWORD_XXX --with-fd-password=XXX_FDPASSWORD_XXX --with-sd-password=XXX_SDPASSWORD_XXX --with-mon-dir-password=XXX_MONDIRPASSWORD_XXX --with-mon-fd-password=XXX_MONFDPASSWORD_XXX --with-mon-sd-password=XXX_MONSDPASSWORD_XXX --with-db-name=XXX_DBNAME_XXX --with-db-user=XXX_DBUSER_XXX --with-db-password=XXX_DBPASSWORD_XXX --with-sqlite3 --without-mysql --without-postgresql --without-sqlite --enable-tray-monitor --enable-bat [...] Configuration on Fri May 18 19:54:40 UTC 2012: Host: x86_64-pc-linux-gnu -- debian wheezy/sid Bacula version: Bacula 5.0.3 (04 August 2010) [...] SQL binaries Directory /usr/bin Large file support: yes Bacula conio support: no -lreadline -lhistory readline support: yes TCP Wrappers support: yes -lwrap TLS support: yes Encryption support: yes ZLIB support: yes [...] ===== [10] <http://anonscm.debian.org/gitweb/?p=pkg-bacula/bacula.git;a=commitdiff;h=488241e3d2c90af4011597c652670d3c74e95cbe> dpkg-shlibdeps still complains about unused symbols, but this time only for libhistory.so.6: ===== dpkg-shlibdeps: warning: package could avoid a useless dependency if debian/bacula-console/usr/sbin/bacula-console was not linked against libdl.so.2 (it uses none of the library's symbols). dpkg-shlibdeps: warning: package could avoid a useless dependency if debian/bacula-console/usr/sbin/bacula-console was not linked against libz.so.1 (it uses none of the library's symbols). dpkg-shlibdeps: warning: package could avoid a useless dependency if debian/bacula-console/usr/sbin/bacula-console was not linked against libhistory.so.6 (it uses none of the library's symbols). dpkg-shlibdeps: warning: package could avoid a useless dependency if debian/bacula-console/usr/sbin/bacula-console was not linked against libcrypto.so.1.0.0 (it uses none of the library's symbols). dpkg-shlibdeps: warning: package could avoid a useless dependency if debian/bacula-console/usr/sbin/bacula-console was not linked against libwrap.so.0 (it uses none of the library's symbols). dpkg-shlibdeps: warning: package could avoid a useless dependency if debian/bacula-console/usr/sbin/bacula-console was not linked against libssl.so.1.0.0 (it uses none of the library's symbols). ===== The resulting bacula-console binary works as expected (CTRL-UP/DOWN and TAB-COMPLETION) against a 5.0.2 Director. Is there something I am missing? ====================================================================== ---------------------------------------------------------------------- (0006326) kern (administrator) - 2012-05-24 07:34 http://bugs.bacula.org/view.php?id=1871#c6326 ---------------------------------------------------------------------- Thanks for the patch. This patch looks quite reasonable to me. We will test it and unless some problems come up, highly unlikely given your testing, we will include it in the next release. I will leave the bug report open until the patch is committed. ---------------------------------------------------------------------- (0006383) kern (administrator) - 2012-06-10 08:59 http://bugs.bacula.org/view.php?id=1871#c6383 ---------------------------------------------------------------------- Thanks again for the patch. It is applied and will be available in version 5.2.8 to be released today or tomorrow. ---------------------------------------------------------------------- (0006385) marcovw (developer) - 2012-06-11 09:46 http://bugs.bacula.org/view.php?id=1871#c6385 ---------------------------------------------------------------------- Maybe this works on Linux but on Solaris its a disaster as readline uses a quite some symbols from the termcap library Undefined first referenced symbol in file tgoto /usr/lib/libreadline.so tputs /usr/lib/libreadline.so tgetent /usr/lib/libreadline.so tgetnum /usr/lib/libreadline.so tgetstr /usr/lib/libreadline.so tgetflag /usr/lib/libreadline.so I also see NO reason in changing this behaviour as on Linux this also never been a problem (at least not on OpenSuse/Fedora and SLES/RHEL/CentOS) ---------------------------------------------------------------------- (0006387) slaanesh (reporter) - 2012-06-11 12:56 http://bugs.bacula.org/view.php?id=1871#c6387 ---------------------------------------------------------------------- Hello, this also breaks compilation with RHEL 5 libraries: make[1]: Entering directory `/builddir/build/BUILD/bacula-5.2.8/src/console' Compiling console.c Compiling console_conf.c Compiling authenticate.c /builddir/build/BUILD/bacula-5.2.8/libtool --silent --tag=CXX --mode=link /usr/bin/g++ -L../lib -L../cats -o bconsole console.o console_conf.o authenticate.o \ -lreadline -lbaccfg -lbac -lm -lpthread -ldl -ldl \ -lssl -lcrypto /usr/lib/gcc/x86_64-redhat-linux/4.1.2/../../../../lib64/libreadline.so: undefined reference to `tgetnum' /usr/lib/gcc/x86_64-redhat-linux/4.1.2/../../../../lib64/libreadline.so: undefined reference to `tgetent' /usr/lib/gcc/x86_64-redhat-linux/4.1.2/../../../../lib64/libreadline.so: undefined reference to `tgetstr' /usr/lib/gcc/x86_64-redhat-linux/4.1.2/../../../../lib64/libreadline.so: undefined reference to `tgoto' /usr/lib/gcc/x86_64-redhat-linux/4.1.2/../../../../lib64/libreadline.so: undefined reference to `UP' /usr/lib/gcc/x86_64-redhat-linux/4.1.2/../../../../lib64/libreadline.so: undefined reference to `BC' /usr/lib/gcc/x86_64-redhat-linux/4.1.2/../../../../lib64/libreadline.so: undefined reference to `tputs' /usr/lib/gcc/x86_64-redhat-linux/4.1.2/../../../../lib64/libreadline.so: undefined reference to `PC' /usr/lib/gcc/x86_64-redhat-linux/4.1.2/../../../../lib64/libreadline.so: undefined reference to `tgetflag' collect2: ld returned 1 exit status make[1]: Leaving directory `/builddir/build/BUILD/bacula-5.2.8/src/console' make[1]: *** [bconsole] Error 1 Works as expected in RHEL 6 and Fedora. Issue History Date Modified Username Field Change ====================================================================== 2012-05-19 13:34 lu...@pc... New Issue 2012-05-19 13:34 lu...@pc... File Added: fix-readline-ncurses-depends.patch 2012-05-24 07:34 kern Note Added: 0006326 2012-05-24 07:34 kern Assigned To => kern 2012-05-24 07:34 kern Status new => acknowledged 2012-06-10 08:59 kern Note Added: 0006383 2012-06-10 08:59 kern Status acknowledged => closed 2012-06-10 08:59 kern Resolution open => fixed 2012-06-10 08:59 kern Fixed in Version => 5.2.8 2012-06-11 09:46 marcovw Note Added: 0006385 2012-06-11 09:46 marcovw Status closed => feedback 2012-06-11 09:46 marcovw Resolution fixed => reopened 2012-06-11 12:56 slaanesh Note Added: 0006387 ====================================================================== |
From: Mantis B. T. <no...@bu...> - 2012-06-11 08:46:22
|
The following issue has been REOPENED. ====================================================================== http://bugs.bacula.org/view.php?id=1871 ====================================================================== Reported By: lu...@pc... Assigned To: kern ====================================================================== Project: bacula Issue ID: 1871 Category: configure/build process Reproducibility: always Severity: minor Priority: normal Status: feedback ====================================================================== Date Submitted: 2012-05-19 13:34 BST Last Modified: 2012-06-11 09:46 BST ====================================================================== Summary: readline support not available without TERM_LIB Description: This bug has been reported in Debian by Sven Joachim [1]. [1] <http://bugs.debian.org/646730> It is impossible to build the bconsole readline support in Bacula_5.0.2+ without the TERM_LIB. I know what the Bacula manual says about readline: - System Requirements [2] * If you want to enable command line editing and history, you will need to have /usr/include/termcap.h and either the termcap or the ncurses library loaded (libtermcap-devel or ncurses-devel). [2] <http://www.bacula.org/5.2.x-manuals/en/main/main/System_Requirements.html> - Building Bacula from Source [3] The --enable-conio or --enable-readline options are useful because they provide a command line history and editing capability for the Console program. If you have included either option in the build, either the termcap or the ncurses package will be needed to link. [...] readline is no longer supported after version 1.34. The code within Bacula remains, so it should be usable, and if users submit patches for it, we will be happy to apply them. However, due to the fact that each version of readline seems to be incompatible with previous versions, and that there are significant differences between systems, we can no longer afford to support it. [3] <http://www.bacula.org/manuals/en/install/install/Installing_Bacula.html> However, the readline support is never available if you do not have TERM_LIB because of [4], which basically put conio and readline at the same level in terms of dependencies. I was not able to find any reason for this change in the ChangeLog neither in the BTS and similar bugs were either too old (http://bugs.bacula.org/view.php?id=743 [5]) or not related (http://bugs.bacula.org/view.php?id=1226 [6]). [4] <http://www.bacula.org/git/cgit.cgi/bacula/commit/?h=Branch-5.0&id=967d4334e69d2ab739a2f270f2df1bcfb0ffc7d4> [5] <http://bugs.bacula.org/view.php?id=743> [6] <http://bugs.bacula.org/view.php?id=1226> However, it seems that the readline support does not need any TERM_LIB to be built, given that according to build logs [7][8] /usr/sbin/bacula-console (this is upstream bconsole, see [9] for the future fix in Debian) does not use any symbols from libhistory.so.6, nor libtinfo.so.5 neither libncurses.so.5. [7] <http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=646730#31> [8] <https://buildd.debian.org/status/fetch.php?pkg=bacula&arch=amd64&ver=5.0.3%2Bdfsg-0.1&stamp=1335779247> [9] <http://anonscm.debian.org/gitweb/?p=pkg-bacula/bacula.git;a=commitdiff;h=2cdfcfec4764ae6cde77c44f474bd521f7562e1d> With the attached patch provided by Alexander Golovko [10] the Debian packages are built with: ===== $ cd /tmp/buildd/bacula-5.0.3+dfsgreal/debian/tmp-build-sqlite3 && \ QMAKE=/usr/bin/qmake-qt4 ./configure --config-cache \ --host=x86_64-linux-gnu --build=x86_64-linux-gnu \ --prefix=/usr \ --with-archivedir=/nonexistant/path/to/file/archive/dir \ --sysconfdir=/etc/bacula --with-scriptdir=/etc/bacula/scripts \ --sharedstatedir=/var/lib/bacula \ --localstatedir=/var/lib/bacula \ --with-pid-dir=/var/run/bacula --with-smtp-host=localhost \ --with-working-dir=/var/lib/bacula \ --with-subsys-dir=/var/lock \ --mandir=\${prefix}/share/man \ --infodir=\${prefix}/share/info \ --enable-smartalloc --with-python --with-tcp-wrappers --with-openssl --with-libiconv-prefix=/usr/include --with-readline=/usr/include/readline --disable-conio --with-libintl-prefix=/usr/include --with-x --docdir=\${prefix}/share/doc/bacula-common --htmldir=\${prefix}/share/doc/bacula-common/html --libdir=\${prefix}/lib/bacula --enable-batch-insert --disable-bwx-console --without-qwt --enable-ipv6 --with-dir-password=XXX_DIRPASSWORD_XXX --with-fd-password=XXX_FDPASSWORD_XXX --with-sd-password=XXX_SDPASSWORD_XXX --with-mon-dir-password=XXX_MONDIRPASSWORD_XXX --with-mon-fd-password=XXX_MONFDPASSWORD_XXX --with-mon-sd-password=XXX_MONSDPASSWORD_XXX --with-db-name=XXX_DBNAME_XXX --with-db-user=XXX_DBUSER_XXX --with-db-password=XXX_DBPASSWORD_XXX --with-sqlite3 --without-mysql --without-postgresql --without-sqlite --enable-tray-monitor --enable-bat [...] Configuration on Fri May 18 19:54:40 UTC 2012: Host: x86_64-pc-linux-gnu -- debian wheezy/sid Bacula version: Bacula 5.0.3 (04 August 2010) [...] SQL binaries Directory /usr/bin Large file support: yes Bacula conio support: no -lreadline -lhistory readline support: yes TCP Wrappers support: yes -lwrap TLS support: yes Encryption support: yes ZLIB support: yes [...] ===== [10] <http://anonscm.debian.org/gitweb/?p=pkg-bacula/bacula.git;a=commitdiff;h=488241e3d2c90af4011597c652670d3c74e95cbe> dpkg-shlibdeps still complains about unused symbols, but this time only for libhistory.so.6: ===== dpkg-shlibdeps: warning: package could avoid a useless dependency if debian/bacula-console/usr/sbin/bacula-console was not linked against libdl.so.2 (it uses none of the library's symbols). dpkg-shlibdeps: warning: package could avoid a useless dependency if debian/bacula-console/usr/sbin/bacula-console was not linked against libz.so.1 (it uses none of the library's symbols). dpkg-shlibdeps: warning: package could avoid a useless dependency if debian/bacula-console/usr/sbin/bacula-console was not linked against libhistory.so.6 (it uses none of the library's symbols). dpkg-shlibdeps: warning: package could avoid a useless dependency if debian/bacula-console/usr/sbin/bacula-console was not linked against libcrypto.so.1.0.0 (it uses none of the library's symbols). dpkg-shlibdeps: warning: package could avoid a useless dependency if debian/bacula-console/usr/sbin/bacula-console was not linked against libwrap.so.0 (it uses none of the library's symbols). dpkg-shlibdeps: warning: package could avoid a useless dependency if debian/bacula-console/usr/sbin/bacula-console was not linked against libssl.so.1.0.0 (it uses none of the library's symbols). ===== The resulting bacula-console binary works as expected (CTRL-UP/DOWN and TAB-COMPLETION) against a 5.0.2 Director. Is there something I am missing? ====================================================================== ---------------------------------------------------------------------- (0006326) kern (administrator) - 2012-05-24 07:34 http://bugs.bacula.org/view.php?id=1871#c6326 ---------------------------------------------------------------------- Thanks for the patch. This patch looks quite reasonable to me. We will test it and unless some problems come up, highly unlikely given your testing, we will include it in the next release. I will leave the bug report open until the patch is committed. ---------------------------------------------------------------------- (0006383) kern (administrator) - 2012-06-10 08:59 http://bugs.bacula.org/view.php?id=1871#c6383 ---------------------------------------------------------------------- Thanks again for the patch. It is applied and will be available in version 5.2.8 to be released today or tomorrow. ---------------------------------------------------------------------- (0006385) marcovw (developer) - 2012-06-11 09:46 http://bugs.bacula.org/view.php?id=1871#c6385 ---------------------------------------------------------------------- Maybe this works on Linux but on Solaris its a disaster as readline uses a quite some symbols from the termcap library Undefined first referenced symbol in file tgoto /usr/lib/libreadline.so tputs /usr/lib/libreadline.so tgetent /usr/lib/libreadline.so tgetnum /usr/lib/libreadline.so tgetstr /usr/lib/libreadline.so tgetflag /usr/lib/libreadline.so I also see NO reason in changing this behaviour as on Linux this also never been a problem (at least not on OpenSuse/Fedora and SLES/RHEL/CentOS) Issue History Date Modified Username Field Change ====================================================================== 2012-05-19 13:34 lu...@pc... New Issue 2012-05-19 13:34 lu...@pc... File Added: fix-readline-ncurses-depends.patch 2012-05-24 07:34 kern Note Added: 0006326 2012-05-24 07:34 kern Assigned To => kern 2012-05-24 07:34 kern Status new => acknowledged 2012-06-10 08:59 kern Note Added: 0006383 2012-06-10 08:59 kern Status acknowledged => closed 2012-06-10 08:59 kern Resolution open => fixed 2012-06-10 08:59 kern Fixed in Version => 5.2.8 2012-06-11 09:46 marcovw Note Added: 0006385 2012-06-11 09:46 marcovw Status closed => feedback 2012-06-11 09:46 marcovw Resolution fixed => reopened ====================================================================== |
From: Mantis B. T. <no...@bu...> - 2012-06-11 13:52:32
|
A NOTE has been added to this issue. ====================================================================== http://bugs.bacula.org/view.php?id=1871 ====================================================================== Reported By: lu...@pc... Assigned To: kern ====================================================================== Project: bacula Issue ID: 1871 Category: configure/build process Reproducibility: always Severity: minor Priority: normal Status: feedback ====================================================================== Date Submitted: 2012-05-19 13:34 BST Last Modified: 2012-06-11 14:52 BST ====================================================================== Summary: readline support not available without TERM_LIB Description: This bug has been reported in Debian by Sven Joachim [1]. [1] <http://bugs.debian.org/646730> It is impossible to build the bconsole readline support in Bacula_5.0.2+ without the TERM_LIB. I know what the Bacula manual says about readline: - System Requirements [2] * If you want to enable command line editing and history, you will need to have /usr/include/termcap.h and either the termcap or the ncurses library loaded (libtermcap-devel or ncurses-devel). [2] <http://www.bacula.org/5.2.x-manuals/en/main/main/System_Requirements.html> - Building Bacula from Source [3] The --enable-conio or --enable-readline options are useful because they provide a command line history and editing capability for the Console program. If you have included either option in the build, either the termcap or the ncurses package will be needed to link. [...] readline is no longer supported after version 1.34. The code within Bacula remains, so it should be usable, and if users submit patches for it, we will be happy to apply them. However, due to the fact that each version of readline seems to be incompatible with previous versions, and that there are significant differences between systems, we can no longer afford to support it. [3] <http://www.bacula.org/manuals/en/install/install/Installing_Bacula.html> However, the readline support is never available if you do not have TERM_LIB because of [4], which basically put conio and readline at the same level in terms of dependencies. I was not able to find any reason for this change in the ChangeLog neither in the BTS and similar bugs were either too old (http://bugs.bacula.org/view.php?id=743 [5]) or not related (http://bugs.bacula.org/view.php?id=1226 [6]). [4] <http://www.bacula.org/git/cgit.cgi/bacula/commit/?h=Branch-5.0&id=967d4334e69d2ab739a2f270f2df1bcfb0ffc7d4> [5] <http://bugs.bacula.org/view.php?id=743> [6] <http://bugs.bacula.org/view.php?id=1226> However, it seems that the readline support does not need any TERM_LIB to be built, given that according to build logs [7][8] /usr/sbin/bacula-console (this is upstream bconsole, see [9] for the future fix in Debian) does not use any symbols from libhistory.so.6, nor libtinfo.so.5 neither libncurses.so.5. [7] <http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=646730#31> [8] <https://buildd.debian.org/status/fetch.php?pkg=bacula&arch=amd64&ver=5.0.3%2Bdfsg-0.1&stamp=1335779247> [9] <http://anonscm.debian.org/gitweb/?p=pkg-bacula/bacula.git;a=commitdiff;h=2cdfcfec4764ae6cde77c44f474bd521f7562e1d> With the attached patch provided by Alexander Golovko [10] the Debian packages are built with: ===== $ cd /tmp/buildd/bacula-5.0.3+dfsgreal/debian/tmp-build-sqlite3 && \ QMAKE=/usr/bin/qmake-qt4 ./configure --config-cache \ --host=x86_64-linux-gnu --build=x86_64-linux-gnu \ --prefix=/usr \ --with-archivedir=/nonexistant/path/to/file/archive/dir \ --sysconfdir=/etc/bacula --with-scriptdir=/etc/bacula/scripts \ --sharedstatedir=/var/lib/bacula \ --localstatedir=/var/lib/bacula \ --with-pid-dir=/var/run/bacula --with-smtp-host=localhost \ --with-working-dir=/var/lib/bacula \ --with-subsys-dir=/var/lock \ --mandir=\${prefix}/share/man \ --infodir=\${prefix}/share/info \ --enable-smartalloc --with-python --with-tcp-wrappers --with-openssl --with-libiconv-prefix=/usr/include --with-readline=/usr/include/readline --disable-conio --with-libintl-prefix=/usr/include --with-x --docdir=\${prefix}/share/doc/bacula-common --htmldir=\${prefix}/share/doc/bacula-common/html --libdir=\${prefix}/lib/bacula --enable-batch-insert --disable-bwx-console --without-qwt --enable-ipv6 --with-dir-password=XXX_DIRPASSWORD_XXX --with-fd-password=XXX_FDPASSWORD_XXX --with-sd-password=XXX_SDPASSWORD_XXX --with-mon-dir-password=XXX_MONDIRPASSWORD_XXX --with-mon-fd-password=XXX_MONFDPASSWORD_XXX --with-mon-sd-password=XXX_MONSDPASSWORD_XXX --with-db-name=XXX_DBNAME_XXX --with-db-user=XXX_DBUSER_XXX --with-db-password=XXX_DBPASSWORD_XXX --with-sqlite3 --without-mysql --without-postgresql --without-sqlite --enable-tray-monitor --enable-bat [...] Configuration on Fri May 18 19:54:40 UTC 2012: Host: x86_64-pc-linux-gnu -- debian wheezy/sid Bacula version: Bacula 5.0.3 (04 August 2010) [...] SQL binaries Directory /usr/bin Large file support: yes Bacula conio support: no -lreadline -lhistory readline support: yes TCP Wrappers support: yes -lwrap TLS support: yes Encryption support: yes ZLIB support: yes [...] ===== [10] <http://anonscm.debian.org/gitweb/?p=pkg-bacula/bacula.git;a=commitdiff;h=488241e3d2c90af4011597c652670d3c74e95cbe> dpkg-shlibdeps still complains about unused symbols, but this time only for libhistory.so.6: ===== dpkg-shlibdeps: warning: package could avoid a useless dependency if debian/bacula-console/usr/sbin/bacula-console was not linked against libdl.so.2 (it uses none of the library's symbols). dpkg-shlibdeps: warning: package could avoid a useless dependency if debian/bacula-console/usr/sbin/bacula-console was not linked against libz.so.1 (it uses none of the library's symbols). dpkg-shlibdeps: warning: package could avoid a useless dependency if debian/bacula-console/usr/sbin/bacula-console was not linked against libhistory.so.6 (it uses none of the library's symbols). dpkg-shlibdeps: warning: package could avoid a useless dependency if debian/bacula-console/usr/sbin/bacula-console was not linked against libcrypto.so.1.0.0 (it uses none of the library's symbols). dpkg-shlibdeps: warning: package could avoid a useless dependency if debian/bacula-console/usr/sbin/bacula-console was not linked against libwrap.so.0 (it uses none of the library's symbols). dpkg-shlibdeps: warning: package could avoid a useless dependency if debian/bacula-console/usr/sbin/bacula-console was not linked against libssl.so.1.0.0 (it uses none of the library's symbols). ===== The resulting bacula-console binary works as expected (CTRL-UP/DOWN and TAB-COMPLETION) against a 5.0.2 Director. Is there something I am missing? ====================================================================== ---------------------------------------------------------------------- (0006326) kern (administrator) - 2012-05-24 07:34 http://bugs.bacula.org/view.php?id=1871#c6326 ---------------------------------------------------------------------- Thanks for the patch. This patch looks quite reasonable to me. We will test it and unless some problems come up, highly unlikely given your testing, we will include it in the next release. I will leave the bug report open until the patch is committed. ---------------------------------------------------------------------- (0006383) kern (administrator) - 2012-06-10 08:59 http://bugs.bacula.org/view.php?id=1871#c6383 ---------------------------------------------------------------------- Thanks again for the patch. It is applied and will be available in version 5.2.8 to be released today or tomorrow. ---------------------------------------------------------------------- (0006385) marcovw (developer) - 2012-06-11 09:46 http://bugs.bacula.org/view.php?id=1871#c6385 ---------------------------------------------------------------------- Maybe this works on Linux but on Solaris its a disaster as readline uses a quite some symbols from the termcap library Undefined first referenced symbol in file tgoto /usr/lib/libreadline.so tputs /usr/lib/libreadline.so tgetent /usr/lib/libreadline.so tgetnum /usr/lib/libreadline.so tgetstr /usr/lib/libreadline.so tgetflag /usr/lib/libreadline.so I also see NO reason in changing this behaviour as on Linux this also never been a problem (at least not on OpenSuse/Fedora and SLES/RHEL/CentOS) ---------------------------------------------------------------------- (0006387) slaanesh (reporter) - 2012-06-11 12:56 http://bugs.bacula.org/view.php?id=1871#c6387 ---------------------------------------------------------------------- Hello, this also breaks compilation with RHEL 5 libraries: make[1]: Entering directory `/builddir/build/BUILD/bacula-5.2.8/src/console' Compiling console.c Compiling console_conf.c Compiling authenticate.c /builddir/build/BUILD/bacula-5.2.8/libtool --silent --tag=CXX --mode=link /usr/bin/g++ -L../lib -L../cats -o bconsole console.o console_conf.o authenticate.o \ -lreadline -lbaccfg -lbac -lm -lpthread -ldl -ldl \ -lssl -lcrypto /usr/lib/gcc/x86_64-redhat-linux/4.1.2/../../../../lib64/libreadline.so: undefined reference to `tgetnum' /usr/lib/gcc/x86_64-redhat-linux/4.1.2/../../../../lib64/libreadline.so: undefined reference to `tgetent' /usr/lib/gcc/x86_64-redhat-linux/4.1.2/../../../../lib64/libreadline.so: undefined reference to `tgetstr' /usr/lib/gcc/x86_64-redhat-linux/4.1.2/../../../../lib64/libreadline.so: undefined reference to `tgoto' /usr/lib/gcc/x86_64-redhat-linux/4.1.2/../../../../lib64/libreadline.so: undefined reference to `UP' /usr/lib/gcc/x86_64-redhat-linux/4.1.2/../../../../lib64/libreadline.so: undefined reference to `BC' /usr/lib/gcc/x86_64-redhat-linux/4.1.2/../../../../lib64/libreadline.so: undefined reference to `tputs' /usr/lib/gcc/x86_64-redhat-linux/4.1.2/../../../../lib64/libreadline.so: undefined reference to `PC' /usr/lib/gcc/x86_64-redhat-linux/4.1.2/../../../../lib64/libreadline.so: undefined reference to `tgetflag' collect2: ld returned 1 exit status make[1]: Leaving directory `/builddir/build/BUILD/bacula-5.2.8/src/console' make[1]: *** [bconsole] Error 1 Works as expected in RHEL 6 and Fedora. ---------------------------------------------------------------------- (0006390) kern (administrator) - 2012-06-11 14:52 http://bugs.bacula.org/view.php?id=1871#c6390 ---------------------------------------------------------------------- I guess I should have tried building it on a few more systems. I will back out the patch and release a new version without it sometime tonight ... Issue History Date Modified Username Field Change ====================================================================== 2012-05-19 13:34 lu...@pc... New Issue 2012-05-19 13:34 lu...@pc... File Added: fix-readline-ncurses-depends.patch 2012-05-24 07:34 kern Note Added: 0006326 2012-05-24 07:34 kern Assigned To => kern 2012-05-24 07:34 kern Status new => acknowledged 2012-06-10 08:59 kern Note Added: 0006383 2012-06-10 08:59 kern Status acknowledged => closed 2012-06-10 08:59 kern Resolution open => fixed 2012-06-10 08:59 kern Fixed in Version => 5.2.8 2012-06-11 09:46 marcovw Note Added: 0006385 2012-06-11 09:46 marcovw Status closed => feedback 2012-06-11 09:46 marcovw Resolution fixed => reopened 2012-06-11 12:56 slaanesh Note Added: 0006387 2012-06-11 14:52 kern Note Added: 0006390 ====================================================================== |
From: Mantis B. T. <no...@bu...> - 2012-06-17 11:44:40
|
The following issue has been CLOSED ====================================================================== http://bugs.bacula.org/view.php?id=1871 ====================================================================== Reported By: lu...@pc... Assigned To: kern ====================================================================== Project: bacula Issue ID: 1871 Category: configure/build process Reproducibility: always Severity: minor Priority: normal Status: closed Resolution: won't fix Fixed in Version: ====================================================================== Date Submitted: 2012-05-19 13:34 BST Last Modified: 2012-06-17 12:44 BST ====================================================================== Summary: readline support not available without TERM_LIB Description: This bug has been reported in Debian by Sven Joachim [1]. [1] <http://bugs.debian.org/646730> It is impossible to build the bconsole readline support in Bacula_5.0.2+ without the TERM_LIB. I know what the Bacula manual says about readline: - System Requirements [2] * If you want to enable command line editing and history, you will need to have /usr/include/termcap.h and either the termcap or the ncurses library loaded (libtermcap-devel or ncurses-devel). [2] <http://www.bacula.org/5.2.x-manuals/en/main/main/System_Requirements.html> - Building Bacula from Source [3] The --enable-conio or --enable-readline options are useful because they provide a command line history and editing capability for the Console program. If you have included either option in the build, either the termcap or the ncurses package will be needed to link. [...] readline is no longer supported after version 1.34. The code within Bacula remains, so it should be usable, and if users submit patches for it, we will be happy to apply them. However, due to the fact that each version of readline seems to be incompatible with previous versions, and that there are significant differences between systems, we can no longer afford to support it. [3] <http://www.bacula.org/manuals/en/install/install/Installing_Bacula.html> However, the readline support is never available if you do not have TERM_LIB because of [4], which basically put conio and readline at the same level in terms of dependencies. I was not able to find any reason for this change in the ChangeLog neither in the BTS and similar bugs were either too old (http://bugs.bacula.org/view.php?id=743 [5]) or not related (http://bugs.bacula.org/view.php?id=1226 [6]). [4] <http://www.bacula.org/git/cgit.cgi/bacula/commit/?h=Branch-5.0&id=967d4334e69d2ab739a2f270f2df1bcfb0ffc7d4> [5] <http://bugs.bacula.org/view.php?id=743> [6] <http://bugs.bacula.org/view.php?id=1226> However, it seems that the readline support does not need any TERM_LIB to be built, given that according to build logs [7][8] /usr/sbin/bacula-console (this is upstream bconsole, see [9] for the future fix in Debian) does not use any symbols from libhistory.so.6, nor libtinfo.so.5 neither libncurses.so.5. [7] <http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=646730#31> [8] <https://buildd.debian.org/status/fetch.php?pkg=bacula&arch=amd64&ver=5.0.3%2Bdfsg-0.1&stamp=1335779247> [9] <http://anonscm.debian.org/gitweb/?p=pkg-bacula/bacula.git;a=commitdiff;h=2cdfcfec4764ae6cde77c44f474bd521f7562e1d> With the attached patch provided by Alexander Golovko [10] the Debian packages are built with: ===== $ cd /tmp/buildd/bacula-5.0.3+dfsgreal/debian/tmp-build-sqlite3 && \ QMAKE=/usr/bin/qmake-qt4 ./configure --config-cache \ --host=x86_64-linux-gnu --build=x86_64-linux-gnu \ --prefix=/usr \ --with-archivedir=/nonexistant/path/to/file/archive/dir \ --sysconfdir=/etc/bacula --with-scriptdir=/etc/bacula/scripts \ --sharedstatedir=/var/lib/bacula \ --localstatedir=/var/lib/bacula \ --with-pid-dir=/var/run/bacula --with-smtp-host=localhost \ --with-working-dir=/var/lib/bacula \ --with-subsys-dir=/var/lock \ --mandir=\${prefix}/share/man \ --infodir=\${prefix}/share/info \ --enable-smartalloc --with-python --with-tcp-wrappers --with-openssl --with-libiconv-prefix=/usr/include --with-readline=/usr/include/readline --disable-conio --with-libintl-prefix=/usr/include --with-x --docdir=\${prefix}/share/doc/bacula-common --htmldir=\${prefix}/share/doc/bacula-common/html --libdir=\${prefix}/lib/bacula --enable-batch-insert --disable-bwx-console --without-qwt --enable-ipv6 --with-dir-password=XXX_DIRPASSWORD_XXX --with-fd-password=XXX_FDPASSWORD_XXX --with-sd-password=XXX_SDPASSWORD_XXX --with-mon-dir-password=XXX_MONDIRPASSWORD_XXX --with-mon-fd-password=XXX_MONFDPASSWORD_XXX --with-mon-sd-password=XXX_MONSDPASSWORD_XXX --with-db-name=XXX_DBNAME_XXX --with-db-user=XXX_DBUSER_XXX --with-db-password=XXX_DBPASSWORD_XXX --with-sqlite3 --without-mysql --without-postgresql --without-sqlite --enable-tray-monitor --enable-bat [...] Configuration on Fri May 18 19:54:40 UTC 2012: Host: x86_64-pc-linux-gnu -- debian wheezy/sid Bacula version: Bacula 5.0.3 (04 August 2010) [...] SQL binaries Directory /usr/bin Large file support: yes Bacula conio support: no -lreadline -lhistory readline support: yes TCP Wrappers support: yes -lwrap TLS support: yes Encryption support: yes ZLIB support: yes [...] ===== [10] <http://anonscm.debian.org/gitweb/?p=pkg-bacula/bacula.git;a=commitdiff;h=488241e3d2c90af4011597c652670d3c74e95cbe> dpkg-shlibdeps still complains about unused symbols, but this time only for libhistory.so.6: ===== dpkg-shlibdeps: warning: package could avoid a useless dependency if debian/bacula-console/usr/sbin/bacula-console was not linked against libdl.so.2 (it uses none of the library's symbols). dpkg-shlibdeps: warning: package could avoid a useless dependency if debian/bacula-console/usr/sbin/bacula-console was not linked against libz.so.1 (it uses none of the library's symbols). dpkg-shlibdeps: warning: package could avoid a useless dependency if debian/bacula-console/usr/sbin/bacula-console was not linked against libhistory.so.6 (it uses none of the library's symbols). dpkg-shlibdeps: warning: package could avoid a useless dependency if debian/bacula-console/usr/sbin/bacula-console was not linked against libcrypto.so.1.0.0 (it uses none of the library's symbols). dpkg-shlibdeps: warning: package could avoid a useless dependency if debian/bacula-console/usr/sbin/bacula-console was not linked against libwrap.so.0 (it uses none of the library's symbols). dpkg-shlibdeps: warning: package could avoid a useless dependency if debian/bacula-console/usr/sbin/bacula-console was not linked against libssl.so.1.0.0 (it uses none of the library's symbols). ===== The resulting bacula-console binary works as expected (CTRL-UP/DOWN and TAB-COMPLETION) against a 5.0.2 Director. Is there something I am missing? ====================================================================== ---------------------------------------------------------------------- (0006326) kern (administrator) - 2012-05-24 07:34 http://bugs.bacula.org/view.php?id=1871#c6326 ---------------------------------------------------------------------- Thanks for the patch. This patch looks quite reasonable to me. We will test it and unless some problems come up, highly unlikely given your testing, we will include it in the next release. I will leave the bug report open until the patch is committed. ---------------------------------------------------------------------- (0006383) kern (administrator) - 2012-06-10 08:59 http://bugs.bacula.org/view.php?id=1871#c6383 ---------------------------------------------------------------------- Thanks again for the patch. It is applied and will be available in version 5.2.8 to be released today or tomorrow. ---------------------------------------------------------------------- (0006385) marcovw (developer) - 2012-06-11 09:46 http://bugs.bacula.org/view.php?id=1871#c6385 ---------------------------------------------------------------------- Maybe this works on Linux but on Solaris its a disaster as readline uses a quite some symbols from the termcap library Undefined first referenced symbol in file tgoto /usr/lib/libreadline.so tputs /usr/lib/libreadline.so tgetent /usr/lib/libreadline.so tgetnum /usr/lib/libreadline.so tgetstr /usr/lib/libreadline.so tgetflag /usr/lib/libreadline.so I also see NO reason in changing this behaviour as on Linux this also never been a problem (at least not on OpenSuse/Fedora and SLES/RHEL/CentOS) ---------------------------------------------------------------------- (0006387) slaanesh (reporter) - 2012-06-11 12:56 http://bugs.bacula.org/view.php?id=1871#c6387 ---------------------------------------------------------------------- Hello, this also breaks compilation with RHEL 5 libraries: make[1]: Entering directory `/builddir/build/BUILD/bacula-5.2.8/src/console' Compiling console.c Compiling console_conf.c Compiling authenticate.c /builddir/build/BUILD/bacula-5.2.8/libtool --silent --tag=CXX --mode=link /usr/bin/g++ -L../lib -L../cats -o bconsole console.o console_conf.o authenticate.o \ -lreadline -lbaccfg -lbac -lm -lpthread -ldl -ldl \ -lssl -lcrypto /usr/lib/gcc/x86_64-redhat-linux/4.1.2/../../../../lib64/libreadline.so: undefined reference to `tgetnum' /usr/lib/gcc/x86_64-redhat-linux/4.1.2/../../../../lib64/libreadline.so: undefined reference to `tgetent' /usr/lib/gcc/x86_64-redhat-linux/4.1.2/../../../../lib64/libreadline.so: undefined reference to `tgetstr' /usr/lib/gcc/x86_64-redhat-linux/4.1.2/../../../../lib64/libreadline.so: undefined reference to `tgoto' /usr/lib/gcc/x86_64-redhat-linux/4.1.2/../../../../lib64/libreadline.so: undefined reference to `UP' /usr/lib/gcc/x86_64-redhat-linux/4.1.2/../../../../lib64/libreadline.so: undefined reference to `BC' /usr/lib/gcc/x86_64-redhat-linux/4.1.2/../../../../lib64/libreadline.so: undefined reference to `tputs' /usr/lib/gcc/x86_64-redhat-linux/4.1.2/../../../../lib64/libreadline.so: undefined reference to `PC' /usr/lib/gcc/x86_64-redhat-linux/4.1.2/../../../../lib64/libreadline.so: undefined reference to `tgetflag' collect2: ld returned 1 exit status make[1]: Leaving directory `/builddir/build/BUILD/bacula-5.2.8/src/console' make[1]: *** [bconsole] Error 1 Works as expected in RHEL 6 and Fedora. ---------------------------------------------------------------------- (0006390) kern (administrator) - 2012-06-11 14:52 http://bugs.bacula.org/view.php?id=1871#c6390 ---------------------------------------------------------------------- I guess I should have tried building it on a few more systems. I will back out the patch and release a new version without it sometime tonight ... ---------------------------------------------------------------------- (0006398) kern (administrator) - 2012-06-17 12:44 http://bugs.bacula.org/view.php?id=1871#c6398 ---------------------------------------------------------------------- Sorry but this patch is rejected because it is not portable (i.e. it breaks the builds on a number of other systems). Issue History Date Modified Username Field Change ====================================================================== 2012-05-19 13:34 lu...@pc... New Issue 2012-05-19 13:34 lu...@pc... File Added: fix-readline-ncurses-depends.patch 2012-05-24 07:34 kern Note Added: 0006326 2012-05-24 07:34 kern Assigned To => kern 2012-05-24 07:34 kern Status new => acknowledged 2012-06-10 08:59 kern Note Added: 0006383 2012-06-10 08:59 kern Status acknowledged => closed 2012-06-10 08:59 kern Resolution open => fixed 2012-06-10 08:59 kern Fixed in Version => 5.2.8 2012-06-11 09:46 marcovw Note Added: 0006385 2012-06-11 09:46 marcovw Status closed => feedback 2012-06-11 09:46 marcovw Resolution fixed => reopened 2012-06-11 12:56 slaanesh Note Added: 0006387 2012-06-11 14:52 kern Note Added: 0006390 2012-06-17 12:44 kern Note Added: 0006398 2012-06-17 12:44 kern Status feedback => closed 2012-06-17 12:44 kern Resolution reopened => won't fix 2012-06-17 12:44 kern Fixed in Version 5.2.8 => ====================================================================== |
From: Mantis B. T. <no...@bu...> - 2013-05-23 14:50:37
|
A NOTE has been added to this issue. ====================================================================== http://bugs.bacula.org/view.php?id=1871 ====================================================================== Reported By: lu...@pc... Assigned To: kern ====================================================================== Project: bacula Issue ID: 1871 Category: configure/build process Reproducibility: always Severity: minor Priority: normal Status: closed Resolution: won't fix Fixed in Version: ====================================================================== Date Submitted: 2012-05-19 13:34 BST Last Modified: 2013-05-23 15:50 BST ====================================================================== Summary: readline support not available without TERM_LIB Description: This bug has been reported in Debian by Sven Joachim [1]. [1] <http://bugs.debian.org/646730> It is impossible to build the bconsole readline support in Bacula_5.0.2+ without the TERM_LIB. I know what the Bacula manual says about readline: - System Requirements [2] * If you want to enable command line editing and history, you will need to have /usr/include/termcap.h and either the termcap or the ncurses library loaded (libtermcap-devel or ncurses-devel). [2] <http://www.bacula.org/5.2.x-manuals/en/main/main/System_Requirements.html> - Building Bacula from Source [3] The --enable-conio or --enable-readline options are useful because they provide a command line history and editing capability for the Console program. If you have included either option in the build, either the termcap or the ncurses package will be needed to link. [...] readline is no longer supported after version 1.34. The code within Bacula remains, so it should be usable, and if users submit patches for it, we will be happy to apply them. However, due to the fact that each version of readline seems to be incompatible with previous versions, and that there are significant differences between systems, we can no longer afford to support it. [3] <http://www.bacula.org/manuals/en/install/install/Installing_Bacula.html> However, the readline support is never available if you do not have TERM_LIB because of [4], which basically put conio and readline at the same level in terms of dependencies. I was not able to find any reason for this change in the ChangeLog neither in the BTS and similar bugs were either too old (http://bugs.bacula.org/view.php?id=743 [5]) or not related (http://bugs.bacula.org/view.php?id=1226 [6]). [4] <http://www.bacula.org/git/cgit.cgi/bacula/commit/?h=Branch-5.0&id=967d4334e69d2ab739a2f270f2df1bcfb0ffc7d4> [5] <http://bugs.bacula.org/view.php?id=743> [6] <http://bugs.bacula.org/view.php?id=1226> However, it seems that the readline support does not need any TERM_LIB to be built, given that according to build logs [7][8] /usr/sbin/bacula-console (this is upstream bconsole, see [9] for the future fix in Debian) does not use any symbols from libhistory.so.6, nor libtinfo.so.5 neither libncurses.so.5. [7] <http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=646730#31> [8] <https://buildd.debian.org/status/fetch.php?pkg=bacula&arch=amd64&ver=5.0.3%2Bdfsg-0.1&stamp=1335779247> [9] <http://anonscm.debian.org/gitweb/?p=pkg-bacula/bacula.git;a=commitdiff;h=2cdfcfec4764ae6cde77c44f474bd521f7562e1d> With the attached patch provided by Alexander Golovko [10] the Debian packages are built with: ===== $ cd /tmp/buildd/bacula-5.0.3+dfsgreal/debian/tmp-build-sqlite3 && \ QMAKE=/usr/bin/qmake-qt4 ./configure --config-cache \ --host=x86_64-linux-gnu --build=x86_64-linux-gnu \ --prefix=/usr \ --with-archivedir=/nonexistant/path/to/file/archive/dir \ --sysconfdir=/etc/bacula --with-scriptdir=/etc/bacula/scripts \ --sharedstatedir=/var/lib/bacula \ --localstatedir=/var/lib/bacula \ --with-pid-dir=/var/run/bacula --with-smtp-host=localhost \ --with-working-dir=/var/lib/bacula \ --with-subsys-dir=/var/lock \ --mandir=\${prefix}/share/man \ --infodir=\${prefix}/share/info \ --enable-smartalloc --with-python --with-tcp-wrappers --with-openssl --with-libiconv-prefix=/usr/include --with-readline=/usr/include/readline --disable-conio --with-libintl-prefix=/usr/include --with-x --docdir=\${prefix}/share/doc/bacula-common --htmldir=\${prefix}/share/doc/bacula-common/html --libdir=\${prefix}/lib/bacula --enable-batch-insert --disable-bwx-console --without-qwt --enable-ipv6 --with-dir-password=XXX_DIRPASSWORD_XXX --with-fd-password=XXX_FDPASSWORD_XXX --with-sd-password=XXX_SDPASSWORD_XXX --with-mon-dir-password=XXX_MONDIRPASSWORD_XXX --with-mon-fd-password=XXX_MONFDPASSWORD_XXX --with-mon-sd-password=XXX_MONSDPASSWORD_XXX --with-db-name=XXX_DBNAME_XXX --with-db-user=XXX_DBUSER_XXX --with-db-password=XXX_DBPASSWORD_XXX --with-sqlite3 --without-mysql --without-postgresql --without-sqlite --enable-tray-monitor --enable-bat [...] Configuration on Fri May 18 19:54:40 UTC 2012: Host: x86_64-pc-linux-gnu -- debian wheezy/sid Bacula version: Bacula 5.0.3 (04 August 2010) [...] SQL binaries Directory /usr/bin Large file support: yes Bacula conio support: no -lreadline -lhistory readline support: yes TCP Wrappers support: yes -lwrap TLS support: yes Encryption support: yes ZLIB support: yes [...] ===== [10] <http://anonscm.debian.org/gitweb/?p=pkg-bacula/bacula.git;a=commitdiff;h=488241e3d2c90af4011597c652670d3c74e95cbe> dpkg-shlibdeps still complains about unused symbols, but this time only for libhistory.so.6: ===== dpkg-shlibdeps: warning: package could avoid a useless dependency if debian/bacula-console/usr/sbin/bacula-console was not linked against libdl.so.2 (it uses none of the library's symbols). dpkg-shlibdeps: warning: package could avoid a useless dependency if debian/bacula-console/usr/sbin/bacula-console was not linked against libz.so.1 (it uses none of the library's symbols). dpkg-shlibdeps: warning: package could avoid a useless dependency if debian/bacula-console/usr/sbin/bacula-console was not linked against libhistory.so.6 (it uses none of the library's symbols). dpkg-shlibdeps: warning: package could avoid a useless dependency if debian/bacula-console/usr/sbin/bacula-console was not linked against libcrypto.so.1.0.0 (it uses none of the library's symbols). dpkg-shlibdeps: warning: package could avoid a useless dependency if debian/bacula-console/usr/sbin/bacula-console was not linked against libwrap.so.0 (it uses none of the library's symbols). dpkg-shlibdeps: warning: package could avoid a useless dependency if debian/bacula-console/usr/sbin/bacula-console was not linked against libssl.so.1.0.0 (it uses none of the library's symbols). ===== The resulting bacula-console binary works as expected (CTRL-UP/DOWN and TAB-COMPLETION) against a 5.0.2 Director. Is there something I am missing? ====================================================================== ---------------------------------------------------------------------- (0006326) kern (administrator) - 2012-05-24 07:34 http://bugs.bacula.org/view.php?id=1871#c6326 ---------------------------------------------------------------------- Thanks for the patch. This patch looks quite reasonable to me. We will test it and unless some problems come up, highly unlikely given your testing, we will include it in the next release. I will leave the bug report open until the patch is committed. ---------------------------------------------------------------------- (0006383) kern (administrator) - 2012-06-10 08:59 http://bugs.bacula.org/view.php?id=1871#c6383 ---------------------------------------------------------------------- Thanks again for the patch. It is applied and will be available in version 5.2.8 to be released today or tomorrow. ---------------------------------------------------------------------- (0006385) marcovw (viewer) - 2012-06-11 09:46 http://bugs.bacula.org/view.php?id=1871#c6385 ---------------------------------------------------------------------- Maybe this works on Linux but on Solaris its a disaster as readline uses a quite some symbols from the termcap library Undefined first referenced symbol in file tgoto /usr/lib/libreadline.so tputs /usr/lib/libreadline.so tgetent /usr/lib/libreadline.so tgetnum /usr/lib/libreadline.so tgetstr /usr/lib/libreadline.so tgetflag /usr/lib/libreadline.so I also see NO reason in changing this behaviour as on Linux this also never been a problem (at least not on OpenSuse/Fedora and SLES/RHEL/CentOS) ---------------------------------------------------------------------- (0006387) slaanesh (reporter) - 2012-06-11 12:56 http://bugs.bacula.org/view.php?id=1871#c6387 ---------------------------------------------------------------------- Hello, this also breaks compilation with RHEL 5 libraries: make[1]: Entering directory `/builddir/build/BUILD/bacula-5.2.8/src/console' Compiling console.c Compiling console_conf.c Compiling authenticate.c /builddir/build/BUILD/bacula-5.2.8/libtool --silent --tag=CXX --mode=link /usr/bin/g++ -L../lib -L../cats -o bconsole console.o console_conf.o authenticate.o \ -lreadline -lbaccfg -lbac -lm -lpthread -ldl -ldl \ -lssl -lcrypto /usr/lib/gcc/x86_64-redhat-linux/4.1.2/../../../../lib64/libreadline.so: undefined reference to `tgetnum' /usr/lib/gcc/x86_64-redhat-linux/4.1.2/../../../../lib64/libreadline.so: undefined reference to `tgetent' /usr/lib/gcc/x86_64-redhat-linux/4.1.2/../../../../lib64/libreadline.so: undefined reference to `tgetstr' /usr/lib/gcc/x86_64-redhat-linux/4.1.2/../../../../lib64/libreadline.so: undefined reference to `tgoto' /usr/lib/gcc/x86_64-redhat-linux/4.1.2/../../../../lib64/libreadline.so: undefined reference to `UP' /usr/lib/gcc/x86_64-redhat-linux/4.1.2/../../../../lib64/libreadline.so: undefined reference to `BC' /usr/lib/gcc/x86_64-redhat-linux/4.1.2/../../../../lib64/libreadline.so: undefined reference to `tputs' /usr/lib/gcc/x86_64-redhat-linux/4.1.2/../../../../lib64/libreadline.so: undefined reference to `PC' /usr/lib/gcc/x86_64-redhat-linux/4.1.2/../../../../lib64/libreadline.so: undefined reference to `tgetflag' collect2: ld returned 1 exit status make[1]: Leaving directory `/builddir/build/BUILD/bacula-5.2.8/src/console' make[1]: *** [bconsole] Error 1 Works as expected in RHEL 6 and Fedora. ---------------------------------------------------------------------- (0006390) kern (administrator) - 2012-06-11 14:52 http://bugs.bacula.org/view.php?id=1871#c6390 ---------------------------------------------------------------------- I guess I should have tried building it on a few more systems. I will back out the patch and release a new version without it sometime tonight ... ---------------------------------------------------------------------- (0006398) kern (administrator) - 2012-06-17 12:44 http://bugs.bacula.org/view.php?id=1871#c6398 ---------------------------------------------------------------------- Sorry but this patch is rejected because it is not portable (i.e. it breaks the builds on a number of other systems). ---------------------------------------------------------------------- (0006716) kern (administrator) - 2013-05-23 15:50 http://bugs.bacula.org/view.php?id=1871#c6716 ---------------------------------------------------------------------- Unfortunately, this patch got into a later version, and it also broke that version causing some problems :-( Issue History Date Modified Username Field Change ====================================================================== 2012-05-19 13:34 lu...@pc... New Issue 2012-05-19 13:34 lu...@pc... File Added: fix-readline-ncurses-depends.patch 2012-05-24 07:34 kern Note Added: 0006326 2012-05-24 07:34 kern Assigned To => kern 2012-05-24 07:34 kern Status new => acknowledged 2012-06-10 08:59 kern Note Added: 0006383 2012-06-10 08:59 kern Status acknowledged => closed 2012-06-10 08:59 kern Resolution open => fixed 2012-06-10 08:59 kern Fixed in Version => 5.2.8 2012-06-11 09:46 marcovw Note Added: 0006385 2012-06-11 09:46 marcovw Status closed => feedback 2012-06-11 09:46 marcovw Resolution fixed => reopened 2012-06-11 12:56 slaanesh Note Added: 0006387 2012-06-11 14:52 kern Note Added: 0006390 2012-06-17 12:44 kern Note Added: 0006398 2012-06-17 12:44 kern Status feedback => closed 2012-06-17 12:44 kern Resolution reopened => won't fix 2012-06-17 12:44 kern Fixed in Version 5.2.8 => 2013-05-23 15:50 kern Note Added: 0006716 ====================================================================== |