Re: [PATCH 3/4] Use autoreconf in autogen.sh

2013-04-08 Thread Jeff Squyres (jsquyres)
Roland --

If there are no objections, can this patch (and patch 4 of this set: 
https://patchwork.kernel.org/patch/2387321/) be committed?  Neither should not 
have any real impact other than the modernization of the libibverbs build 
system.


On Apr 3, 2013, at 9:06 AM, Jeff Squyres jsquy...@cisco.com wrote:

 The old sequence of Autotools commands listed in autogen.sh is no
 longer correct.  Instead, just use the single autoreconf command,
 which will invoke all the Right Autotools commands in the correct
 order.
 
 ---
 autogen.sh | 6 +-
 1 file changed, 1 insertion(+), 5 deletions(-)
 
 diff --git a/autogen.sh b/autogen.sh
 index fd47839..6c9233e 100755
 --- a/autogen.sh
 +++ b/autogen.sh
 @@ -1,8 +1,4 @@
 #! /bin/sh
 
 set -x
 -aclocal -I config
 -libtoolize --force --copy
 -autoheader
 -automake --foreign --add-missing --copy
 -autoconf
 +autoreconf -ifv -I config
 -- 
 1.8.1.1
 


-- 
Jeff Squyres
jsquy...@cisco.com
For corporate legal information go to: 
http://www.cisco.com/web/about/doing_business/legal/cri/

--
To unsubscribe from this list: send the line unsubscribe linux-rdma in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH 3/4] Use autoreconf in autogen.sh

2013-04-08 Thread Or Gerlitz

On 09/04/2013 00:59, Jeff Squyres (jsquyres) wrote:

Roland -- If there are no objections, can this patch (and patch 4 of this set: 
https://patchwork.kernel.org/patch/2387321/) be committed?  Neither should not 
have any real impact other than the modernization of the libibverbs build 
system.


Jeff, the patches don't carry signed-of-by signature line... I would 
suggest you resubmit patches 3 and 4 with a signature


Or.

--
To unsubscribe from this list: send the line unsubscribe linux-rdma in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html