Re: [PATCH] Clean up autogen.sh script

2008-09-30 Thread Kenton Varda
After some offline discussion I've gone ahead and submitted this.

On Tue, Sep 30, 2008 at 11:40 AM, Jeff Bailey <[EMAIL PROTECTED]> wrote:

>
> The autogen.sh script has all sorts of hardcoded magic in it,
> including the version of the autotools, and the fact that we're
> requiring "foriegn" mode.  With this patch, this will attempt to use
> whatever the system autotools is, and also makes the foreign mode
> explicit.
>
> diff --git a/Makefile.am b/Makefile.am
> index cd90b6c..52f2caf 100644
> --- a/Makefile.am
> +++ b/Makefile.am
> @@ -2,6 +2,8 @@
>
>  ACLOCAL_AMFLAGS = -I m4
>
> +AUTOMAKE_OPTIONS = foreign
> +
>  SUBDIRS = src
>
>  EXTRA_DIST
> = \
> diff --git a/autogen.sh b/autogen.sh
> index 34a4d2e..959279b 100755
> --- a/autogen.sh
> +++ b/autogen.sh
> @@ -15,13 +15,7 @@ fi
>
>  set -ex
>
> -rm -rf autom4te.cache
> -
> -aclocal-1.9 --force -I m4
> -libtoolize -c -f
> -autoheader -f -W all
> -automake-1.9 --foreign -a -c -f -W all
> -autoconf -f -W all,no-obsolete
> +autoreconf -Wall -f -i
>
>  rm -rf autom4te.cache config.h.in~
>  exit 0
>
> >
>

--~--~-~--~~~---~--~~
You received this message because you are subscribed to the Google Groups 
"Protocol Buffers" group.
To post to this group, send email to protobuf@googlegroups.com
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at 
http://groups.google.com/group/protobuf?hl=en
-~--~~~~--~~--~--~---



Re: [PATCH] Clean up autogen.sh script

2008-09-30 Thread Kenton Varda
The reason I specified automake-1.9 explicitly is because our corp
workstations actually have the regular "automake" command linked to
automake-1.4, which is not good enough.  automake-1.9 is installed on our
workstations, but is not the default.  Can we somehow auto-detect this
situation and do something about it?

On Tue, Sep 30, 2008 at 11:40 AM, Jeff Bailey <[EMAIL PROTECTED]> wrote:

>
> The autogen.sh script has all sorts of hardcoded magic in it,
> including the version of the autotools, and the fact that we're
> requiring "foriegn" mode.  With this patch, this will attempt to use
> whatever the system autotools is, and also makes the foreign mode
> explicit.
>
> diff --git a/Makefile.am b/Makefile.am
> index cd90b6c..52f2caf 100644
> --- a/Makefile.am
> +++ b/Makefile.am
> @@ -2,6 +2,8 @@
>
>  ACLOCAL_AMFLAGS = -I m4
>
> +AUTOMAKE_OPTIONS = foreign
> +
>  SUBDIRS = src
>
>  EXTRA_DIST
> = \
> diff --git a/autogen.sh b/autogen.sh
> index 34a4d2e..959279b 100755
> --- a/autogen.sh
> +++ b/autogen.sh
> @@ -15,13 +15,7 @@ fi
>
>  set -ex
>
> -rm -rf autom4te.cache
> -
> -aclocal-1.9 --force -I m4
> -libtoolize -c -f
> -autoheader -f -W all
> -automake-1.9 --foreign -a -c -f -W all
> -autoconf -f -W all,no-obsolete
> +autoreconf -Wall -f -i
>
>  rm -rf autom4te.cache config.h.in~
>  exit 0
>
> >
>

--~--~-~--~~~---~--~~
You received this message because you are subscribed to the Google Groups 
"Protocol Buffers" group.
To post to this group, send email to protobuf@googlegroups.com
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at 
http://groups.google.com/group/protobuf?hl=en
-~--~~~~--~~--~--~---