On Fri, 16 Jan 2015 17:09:41 +0100
Lukas Slebodnik <lslebodn(a)redhat.com> wrote:
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
ACK and pushed to master.
Thank you,
Simo.
--
Simo Sorce * Red Hat, Inc * New York