On (16/01/15 10:23), Simo Sorce wrote:
On Fri, 16 Jan 2015 15:49:51 +0100
Lukas Slebodnik <lslebodn(a)redhat.com> wrote:
> On (16/01/15 09:21), Simo Sorce wrote:
> >On Fri, 16 Jan 2015 11:10:42 +0100
> >Lukas Slebodnik <lslebodn(a)redhat.com> wrote:
> >
> >> ehlo,
> >>
> >> I was testing mock build of srpm just on fedora-21.
> >> But there is a problem on rawhide due to newer version of automake.
> >> and Stephan's static analysers cron job uses rawhide for testing.
> >>
> >> I tested mock build on fedora-{20,21,rawhide} epel7 with attached
> >> patch.
> >>
> >> LS
> >
> >I actually resolved the same problem in gssntlmssp by simply removing
> >the aux dir completely, so unless we need that directive for some
> >reason I'd do the same here.
> >
> >Mind sending a new patch with those directives just removed ?
> >
> Small benefit of AC_CONFIG_AUX_DIR([build]) it that autoreconf will
> copy all files to sub-directory build. So .gitignore contains just
> line with this directory instead of individual files.
>
> sh$ ls build/
> ar-lib config.guess config.sub install-sh missing
> compile config.rpath depcomp ltmain.sh mkinstalldirs
>
> If you really want I can remove AC_CONFIG_AUX_DIR. There is no
> functional difference.
Yes I prefer to remove it, .gitignore is easy to adjust
Updated patch is attached.
LS