On Mon, Nov 05, 2007 at 03:55:20PM -0600, Eric Sandeen wrote: > From: Thomas Woerner <twoerner@xxxxxxxxxx <mailto:twoerner@xxxxxxxxxx>> > > (Thomas was the original author of this patch) > > I do not claim to be an autotools guru, hence the RFC. > RH bugzilla #220715 claims that newer autoconf/automake no longer > support MKINSTALLDIRS, and the changelog seems to confirm that, as > does testing. :) MKINSTALLDIRS is defined in aclocal.m4, so we're good. This patch isn't necessary, as far as I can tell. Have you actually shown that with some newer version of autoconf, it produces a configure script which doesn't define MKINSTALLDIRS? As far as I know e2fsprogs builds Just Fine, and using autoconf 2.61 isn't a problem. What version of autoconf are you using that triggers this problem? (Note that e2fsprogs does not use automake, since automake is a horrible kludge of backwards non-compatibility and ugliness....) At some point when we go to a newer version of gettext we may need to update things, but I'd much rather save that for when we really uplift gettext and aclocal.m4 all in one go. For now, I suspect you could drop this patch without any problems, since we don't use automake, and aclocal.m4 has a version of AM_MKINSTALLDIRS that will do the right thing. Regards, - Ted - To unsubscribe from this list: send the line "unsubscribe linux-ext4" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
