From: Jim Meyering <jim@meyering.net>
To: Jeff Garzik <jeff@garzik.org>
Cc: Pete Zaitcev <zaitcev@redhat.com>, hail-devel@vger.kernel.org
Subject: Re: AC_CONFIG_MACRO_DIR([m4])
Date: Mon, 06 Dec 2010 20:26:46 +0100 [thread overview]
Message-ID: <8739qawvm1.fsf@meyering.net> (raw)
In-Reply-To: <4CFD31E5.2090508@garzik.org> (Jeff Garzik's message of "Mon, 06 Dec 2010 13:56:37 -0500")
Jeff Garzik wrote:
> On 12/06/2010 12:44 PM, Pete Zaitcev wrote:
>> On Mon, 06 Dec 2010 12:32:22 -0500
>> Jeff Garzik<jeff@garzik.org> wrote:
>>
>>> Keeping the "correct libtool macros in-tree" implies adding a pointless
>>> maintenance burden. The distro always gives us correct, up-to-date
>>> files. Why would hail want to potentially lag upstream's version of
>>> these macros, forcing us to manually track macros that are currently
>>> updated automatically for each ./autogen.sh invocation?
>>
>> I presumed that the important part is a compatibility between the
>> syntax used in various .am files and the libtool scriptography that
>> underpins them. "Lagging" upstream has no downside in this case
>> (unlike zlib, where security fixes may exist).
>
> It does not seem optimal to run a current libtool with outdated macro
> files. In all cases except current one, you're checking in third
> party, maintained, versioned files to hail.git where they will be
> less-well maintained, and generally out-of-date vis a vis current
> [upstream | Fedora].
Hi Jeff,
The patch that adds those two lines does not (and IMHO should not)
imply that a project would version-control those files.
Typically, those symlinks exist only in your working directory,
and not in any project's VC repository.
If you have no other files in m4/, you can simply .gitignore
the entire m4/ directory.
> Where is the value in performing this additional work, besides
> silencing a warning seen only by git repo users?
Yeah, either way it's not a big deal.
prev parent reply other threads:[~2010-12-06 19:26 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-12-05 21:56 AC_CONFIG_MACRO_DIR([m4]) Pete Zaitcev
2010-12-06 11:12 ` AC_CONFIG_MACRO_DIR([m4]) Jim Meyering
2010-12-06 17:32 ` AC_CONFIG_MACRO_DIR([m4]) Jeff Garzik
2010-12-06 17:44 ` AC_CONFIG_MACRO_DIR([m4]) Pete Zaitcev
2010-12-06 18:56 ` AC_CONFIG_MACRO_DIR([m4]) Jeff Garzik
2010-12-06 19:26 ` Jim Meyering [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=8739qawvm1.fsf@meyering.net \
--to=jim@meyering.net \
--cc=hail-devel@vger.kernel.org \
--cc=jeff@garzik.org \
--cc=zaitcev@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.