On Thu, Mar 09, 2006 at 09:56:32AM -0700, Alan Robertson wrote: > Xun Sun wrote: > >Hi, > > > >Now that we don't depend on openssl functionality, it seems that the > >BuildPrereq of openssl-devel in heartbeat.spec.in should be removed. > >Any objections to doing so? > > > >--- heartbeat.spec.in 27 Feb 2006 17:01:24 -0000 1.164 > >+++ heartbeat.spec.in 9 Mar 2006 13:42:03 -0000 > >@@ -13,7 +13,7 @@ > > Packager: Alan Robertson <[EMAIL PROTECTED]> > > Source: http://linux-ha.org/download/@[EMAIL PROTECTED]@[EMAIL PROTECTED] > > BuildRoot: %{_tmppath}/%{name}-%{version}-build > >-BuildPrereq: glib-devel, @UCDSNMPDEVEL@, perl, iputils, /usr/bin/ssh, > >openssl-devel, libnet > >+BuildPrereq: glib-devel, @UCDSNMPDEVEL@, perl, iputils, /usr/bin/ssh, > >libnet > > # libnet available from www.ultramonkey.org and others > > Requires: sysklogd, heartbeat-stonith = %{version}-%{release}, > >heartbeat-pils = %{version}-%{release} > > > >On 3/9/06, [email protected] > ><[email protected]> wrote: > >>linux-ha CVS committal > >> > >>Author : xunsun > >>Host : > >>Module : linux-ha > >> > >>Dir : linux-ha > >> > >> > >>Modified Files: > >> configure.in > >> > >> > >>Log Message: > >>removed checking for openssl headers > >>=================================================================== > >>RCS file: /home/cvs/linux-ha/linux-ha/configure.in,v > >>retrieving revision 1.492 > >>retrieving revision 1.493 > >>diff -u -3 -r1.492 -r1.493 > >>--- configure.in 8 Mar 2006 22:24:29 -0000 1.492 > >>+++ configure.in 9 Mar 2006 13:42:04 -0000 1.493 > >>@@ -10,7 +10,7 @@ > >> AC_INIT(heartbeat.spec.in) > >> > >> AC_CONFIG_AUX_DIR(.) > >>-AC_REVISION($Revision: 1.492 $) dnl cvs revision > >>+AC_REVISION($Revision: 1.493 $) dnl cvs revision > >> AC_CANONICAL_HOST > >> > >> > >>@@ -930,15 +930,13 @@ > >> AC_CHECK_HEADERS(linux/icmpv6.h,[],[],[#include <asm/types.h>]) > >> AM_CONDITIONAL(USE_IPV6ADDR, test "$ac_cv_header_linux_icmpv6_h" = yes > >> -a $new_libnet = yes ) > >> AC_CHECK_HEADERS(libxml/xpath.h) > >>-AC_CHECK_HEADERS(openssl/md5.h) > >>-AC_CHECK_HEADERS(openssl/evp.h) > >> > >> dnl > >> dnl Could check for the libraries, but if you have the headers, > >> dnl you're a dolt for not having installed the libs ;-) > >> dnl > >> AM_CONDITIONAL(USE_VACM, test "$ac_cv_header_vacmclient_api_h" = yes) > >>-AM_CONDITIONAL(USE_DRAC3, test "$ac_cv_header_curl_curl_h" = yes -a > >>"$ac_cv_header_libxml_xpath_h" = yes -a "$ac_cv_header_openssl_md5_h" = > >>yes) > >>+AM_CONDITIONAL(USE_DRAC3, test "$ac_cv_header_curl_curl_h" = yes -a > >>"$ac_cv_header_libxml_xpath_h" = yes) > > > That sounds fine to me. > > Horms: Is this dependency really gone in practice?
I suspect there is still an indirect dependancy there. That is, a library (like say libsnmp) that heartbeat needs which in turn needs libopenssl. That isn't really a problem from a Debian point of view (it'll go away as soon as the library gets changed), and thus not something that we need to worry about. Do you want me to look into it if you want some data. -- Horms _______________________________________________________ Linux-HA-Dev: [email protected] http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev Home Page: http://linux-ha.org/
