diff mbox

[1/2] libibverbs: Use autoreconf in autogen.sh

Message ID 1366652516-9547-1-git-send-email-jsquyres@cisco.com (mailing list archive)
State Accepted, archived
Headers show

Commit Message

Jeff Squyres April 22, 2013, 5:41 p.m. UTC
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.

Signed-off-by: Jeff Squyres <jsquyres@cisco.com>
---
 autogen.sh | 6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

Comments

Jeff Squyres April 25, 2013, 3:38 p.m. UTC | #1
Bump.

On Apr 22, 2013, at 1:41 PM, Jeff Squyres <jsquyres@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.
> 
> Signed-off-by: Jeff Squyres <jsquyres@cisco.com>
> ---
> 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 April 30, 2013, 2:29 p.m. UTC | #2
Bump bump.  :-)

On Apr 25, 2013, at 11:38 AM, Jeff Squyres (jsquyres) <jsquyres@cisco.com> wrote:

> Bump.
> 
> On Apr 22, 2013, at 1:41 PM, Jeff Squyres <jsquyres@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.
>> 
>> Signed-off-by: Jeff Squyres <jsquyres@cisco.com>
>> ---
>> 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
> jsquyres@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 majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
Doug Ledford May 1, 2013, 3:30 p.m. UTC | #3
On 04/30/2013 10:29 AM, Jeff Squyres (jsquyres) wrote:
> Bump bump.  :-)

This is fine with me, however, I think you also need to bump the
autotools version to the latest upstream.  The automated checkers in our
build environment is spitting out errors about a number of upstream
packages where the autotools used to configure the package does not
include proper arm support.  The latest autotools bring in all of the
forthcoming arm variants.  So I would like to see both of these things done.

> On Apr 25, 2013, at 11:38 AM, Jeff Squyres (jsquyres) <jsquyres@cisco.com> wrote:
> 
>> Bump.
>>
>> On Apr 22, 2013, at 1:41 PM, Jeff Squyres <jsquyres@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.
>>>
>>> Signed-off-by: Jeff Squyres <jsquyres@cisco.com>
>>> ---
>>> 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
>> jsquyres@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 majordomo@vger.kernel.org
>> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 
> 

--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Jeff Squyres May 2, 2013, 10:38 a.m. UTC | #4
On May 1, 2013, at 11:30 AM, Doug Ledford <dledford@redhat.com> wrote:

> This is fine with me, however, I think you also need to bump the
> autotools version to the latest upstream.  The automated checkers in our
> build environment is spitting out errors about a number of upstream
> packages where the autotools used to configure the package does not
> include proper arm support.  The latest autotools bring in all of the
> forthcoming arm variants.  So I would like to see both of these things done.

Are you referring to the version of Autotools that Roland uses to create his tarballs?

Because I have no control over that.  :-)


>> On Apr 25, 2013, at 11:38 AM, Jeff Squyres (jsquyres) <jsquyres@cisco.com> wrote:
>> 
>>> Bump.
>>> 
>>> On Apr 22, 2013, at 1:41 PM, Jeff Squyres <jsquyres@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.
>>>> 
>>>> Signed-off-by: Jeff Squyres <jsquyres@cisco.com>
>>>> ---
>>>> 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
>>> jsquyres@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 majordomo@vger.kernel.org
>>> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>> 
>> 
>
diff mbox

Patch

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