[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: autotools clean up

From: Rob Savoye
Subject: Re: autotools clean up
Date: Tue, 26 Jun 2012 07:18:53 -0600
User-agent: Mozilla/5.0 (X11; Linux i686; rv:12.0) Gecko/20120430 Thunderbird/12.0.1

On 06/25/12 23:33, Sami Kerola wrote:
> Hello dejagnu maintainers,
> See attachment for rather large patch. I took liberty to remove all
> autofiles from git repository.
> $ git diff HEAD~1 --shortstat
>  52 files changed, 189 insertions(+), 45448 deletions(-)
> I described commit having the following changes.
>     Do not include autotools generated files in git
>         * remove files autotools will generate
>         * add '' script to allow easy regeneration of autofiles
>         * modify sub-projects to be sub-directory builds
>         * remove files required by sub-projects
>         * add m4 directory to top level
>         * add libtool
>         * add autoscan recommended checks to
>         * fix include paths in couple .c files
>         * add .gitignore file
>         * enable maintainer to 'make dist' package
>         * modify README file to tell how to build git version

  So some of these changes I'd agree with, and some not. I'll be honest
that for my Gnash project, which has a very complex config/build
infrastructure, I do some of the same things you mention here. DejaGnu's
is very simple, as it has no real compiled code, it's just a big Tcl
script. The parts of your patch I like are adding (as long as
it doesn't then run configure). Adding .gitignore is ok, but we don't
need libtool, nor autoscan. Also make dist works for me already.

  It is traditional to not commit the configure script, but I see no
problem with using aclocal.m4 (should be acinclude.m4) instead of an m4
sub directory.

        - rob -

reply via email to

[Prev in Thread] Current Thread [Next in Thread]