Re: [PATCH] xf86-video-intel: Update autotools configuration

[Chris Wilson <chris@chris-wilson.co.uk>]

[-- Attachment #1: Type: text/plain, Size: 1331 bytes --]

On Wed, 30 Mar 2011 03:43:29 +0200, Cyril Brulebois <kibi@debian.org> wrote:
> Hi again,
> 
> Cyril Brulebois <kibi@debian.org> (10/02/2011):
> > The commit message says:
> > | Update autotools configuration
> > | 
> > | Use new libtool syntax and silent-rules to silent
> > | the build output a bit (linux-like)
> > 
> > What it doesn't say is:
> > -AC_CONFIG_AUX_DIR(.)
> > +AC_CONFIG_AUX_DIR([build-aux])
> 
> sorry to get back to this, but it's actually a bit more annoying than
> just breaking tinderbox at the time. Out of the 56 X drivers I'm
> currently maintaining, that's the only one no longer searching in '.',
> and I don't know why that should be.
> 
> Also, “autoreconf -vfi” puts files there, but those aren't cleaned,
> even through targets like distclean or maintainer-clean. Having to
> clean things through “git clean -xdf” sounds like a no-no to me. (Of
> course, our build system is flexible enough that I can do anything,
> like rm -rf'ing that directory, but I'd rather not have to come to
> this kind of things…)

I am slightly perplexed by this, since the darn thing passed make
distcheck which I thought included a test to make sure that all files
were deleted after a make clean. Was I just naive in believing that was
sufficient?
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre

[-- Attachment #2: Type: text/plain, Size: 159 bytes --]

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx