[ewg] Re: [PATCH] ofed_1_4_scripts: Fix ppc64 building of rnfs-utils

[Jon Mason]

On Mon, Mar 30, 2009 at 11:25:53AM +0200, Alexander Schmidt wrote:
> On Fri, 27 Mar 2009 14:52:27 -0500
> Jon Mason <jon at opengridcomputing.com> wrote:
> 
> > rnfs-utils will not build on ppc64 without a 32bit version of
> > e2fsprogs-devel.  The 32bit version is needed due to logic previously
> > added to install.pl to force the compile to be 32bit.  Another way to
> > fix this issue would be to allow 64bit version of the rnfs-utils by
> > removing the logic which forces it to be 32bit.
> > 
> > With the added dependency, rnfs-utils compiles on RHEL5.2 on ppc64
> > without issue.
> > 
> > Thanks,
> > Jon
> 
> Hi Jon,
> 
> thanks for the patch, it also works fine on RHEL5.3 now.
> 
> I've also tried to install the rnfs-utils package on SLES11, and there I run
> into another problem. The SRPM has the following dependencies:

Thanks for testing this on non-RHEL installs.

> 
> [alexschm at BL3D1974 SRPMS]$ rpm -qpR rnfs-utils-1.1.5-rc1.OFED.src.rpm 
> krb5-devel >= 1.3.1
> autoconf >= 2.57
> automake  
> krb5-devel  
> libtool  
> krb5-libs  
> rpmlib(CompressedFileNames) <= 3.0.4-1
> 
> I've created the patch below for teaching the installer that rnfs-utils needs
> the krb5 packages. But on SLES10 & 11, there is no krb5-libs package, so it's
> not possible to install rnfs-utils. Could you try to fix this?

I think the name of the package on SuSE is krb5 or krb5-server.  I've
attached a SRPM to this e-mail without the krb5-libs dependency.  See if
the package fixes any configure issues you could/will hit when trying to
compile rnfs-utils.  You should be able to copy the SRPM over the one in
the OFED tree and it will build as it would normally.

 
> --- OFED-1.4.1-20090329-0600.orig/install.pl
> +++ OFED-1.4.1-20090329-0600/install.pl
> @@ -947,7 +947,8 @@ my %packages_info = (
>              { name => "rnfs-utils", parent => "rnfs-utils",
>              selected => 0, installed => 0, rpm_exist => 0, rpm_exist32 => 0,
>              available => 0, mode => "user", dist_req_build => [],
> -            dist_req_inst => [], ofa_req_build => [],
> +            dist_req_inst => ( $build32 == 1 )?["krb5-devel$suffix_32bit", "krb5-libs"]:
> +	    ["krb5-devel$suffix_64bit", "krb5-libs"], ofa_req_build => [],
>              ofa_req_inst => [],
>              install32 => 0, exception => 0, configure_options => '' },
>          'rnfs-utils-debuginfo' =>

I was forcing 32bit builds, so the extra logic should not be necessary.
Also, you can add the check for 32bit krb5-libs to the logic I added in
the previous patch to check for 32bit e2fsprogs.  There, I can add extra
logic to determine if the distro is suse or redhat and switch on the dep
for krb5-libs vs krb5/krb5-server.

Thanks,
Jon
-------------- next part --------------
A non-text attachment was scrubbed...
Name: rnfs-utils-1.1.5-rc2.OFED.src.rpm
Type: application/x-redhat-package-manager
Size: 610780 bytes
Desc: not available
URL: <http://lists.openfabrics.org/pipermail/ewg/attachments/20090330/ed94c954/attachment.rpm>