Otavio Salvador wrote:
> On Wed, Apr 8, 2009 at 4:29 PM, Jim Meyering <[email protected]> wrote:
>> Otavio Salvador wrote:
>> IMHO, upstream should not be beholden to distribution versions
>> of automake when they're stuck at 1.10.1 which is over a year old.
>> By that argument, depending on which distro you choose, we'd never
>> be able to use newer tools.
>
> We should not be beholden, agreed but please read bellow.
>
>> Besides, there are several desirable features in automake-1.10b
>> for which I intend to switch parted:
>>
>>  silent-rules
>>  parallel-tests
>>  dist-xz support
>>
>> and maybe one or two others, not counting all of the bug fixes.
>
> Until we add those, there's no need to bump the _requirement_. As
> the name suggest it is the version that is _required_ to build, not a
> suggested one.
>
>> If you require to be able to build using some older version of automake,
>> you can build from a patched branch or even from a release tarball,
>> at which point the version of automake doesn't even matter.
>>
>> Developers can be expected to be able to install a recent automake,
>> even from git, although that would be exceptional.
>
> Sure but I disagree with requiring it for a gratious reason. Once
> we start to use the new features I fully agree but until it is done
> I think we ought to stay with the last compatible one.

When I raised the requirement to 1.10a,
it was most certainly not a gratuitous change.
Even back when I did that, there had been enough
improvements in automake to make it worthwhile.

I've just pushed this change:

From 40d53c44069e85cec28ad886749d3335e42cb47f Mon Sep 17 00:00:00 2001
From: Jim Meyering <[email protected]>
Date: Wed, 8 Apr 2009 22:40:23 +0200
Subject: [PATCH] build-from-git: depend on automake-1.10b

* bootstrap.conf (buildreq): Note the automake-1.10b requirement.
* configure.ac (AM_INIT_AUTOMAKE): Enable new automake options:
dist-xz, color-tests, parallel-tests, silent-rules.
---
 bootstrap.conf |    2 +-
 configure.ac   |    4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/bootstrap.conf b/bootstrap.conf
index 760da9f..e2b7225 100644
--- a/bootstrap.conf
+++ b/bootstrap.conf
@@ -114,7 +114,7 @@ mkdir -p gnulib-tests
 # Build prerequisites
 buildreq="\
 autoconf   2.61
-automake   1.10
+automake   1.10b
 autopoint  -
 gettext    -
 git        1.4.4
diff --git a/configure.ac b/configure.ac
index 65ddedc..5f3f7d9 100644
--- a/configure.ac
+++ b/configure.ac
@@ -1,5 +1,5 @@
 dnl GNU Parted - a library and front end for manipulation hard disk partitions
-dnl Copyright (C) 1998-2002, 2005-2008 Free Software Foundation, Inc.
+dnl Copyright (C) 1998-2002, 2005-2009 Free Software Foundation, Inc.
 dnl
 dnl This file may be modified and/or distributed without restriction.

@@ -52,7 +52,7 @@ AC_SUBST([LT_CURRENT])
 AC_SUBST([LT_REVISION])
 AC_SUBST([LT_AGE])

-AM_INIT_AUTOMAKE([1.10 dist-bzip2])
+AM_INIT_AUTOMAKE([1.10b dist-xz color-tests parallel-tests silent-rules])

 AC_CANONICAL_HOST
 case "$host_os" in
--
1.6.2.rc1.285.gc5f54

_______________________________________________
parted-devel mailing list
[email protected]
http://lists.alioth.debian.org/mailman/listinfo/parted-devel

Reply via email to