xfs
[Top] [All Lists]

Re: [PATCH 0/5] Use libtool to install apps and remove relative lib path

To: Rich Johnston <rjohnston@xxxxxxx>
Subject: Re: [PATCH 0/5] Use libtool to install apps and remove relative lib paths
From: Ben Myers <bpm@xxxxxxx>
Date: Tue, 4 Sep 2012 11:07:06 -0500
Cc: xfs@xxxxxxxxxxx
In-reply-to: <20120828172605.607853317@xxxxxxx>
References: <20120828172605.607853317@xxxxxxx>
User-agent: Mutt/1.5.20 (2009-06-14)
Hey Rich,

On Tue, Aug 28, 2012 at 12:26:05PM -0500, Rich Johnston wrote:
> This 5 part patchset (for xfstests, dmapi and xfsdump):
> 
> 1. Uses libtool to install the binary from the proper location and display a
>    warning if any shared library dependancies are not properly installed.
> 2. Ensures that a libtool wrapper shell script is not installed in place of
>    the application when libtool wrappers are being used.
> 3. Removes the undocumented feature which allowed xfs developers to build and
>    run xfstests without having to install the libraries.  This can lead to
>    touble if you expect that xfstests is using the versions of the libraries
>    installed on the system.

This patchset looks great to me.

Although Dave has mentioned that the first patch probably isn't strictly
necessary... I think that it's worth pulling in.  Since we build with libtool
we should also install with libtool.

Dave also asked about dmapi.  I took a brief look at the dmapi Makefiles.  It
appears the library is already installed using INSTALL_LTLIB.

Looks good!

Reviewed-by: Ben Myers <bpm@xxxxxxx>

<Prev in Thread] Current Thread [Next in Thread>