[Freeciv-Dev] Re: diff_ignore patch

[Raimar Falke <hawk@xxxxxxxxxxxxxxxxxxxxxxx>]

On Mon, Apr 29, 2002 at 01:21:04PM +0200, Per I. Mathisen wrote:
> Anyone opposed to this patch? It adds some files that are new with
> autoconf 2.52 and generalises ignored savegame names a bit (I often use
> "civdiff" and "civgame" for savegames, and want both ignored).

> -civgame*.sav
> +civ*.sav

Ok.

> -config.h
> -config.h.in
> -config.log
> -config.status
> -config.sub
> +config.*

Not ok since we have config.* files which are in the CVS.

> +depcomp

Ok.

> -stamp-h
> -stamp-h.in
> +stamp-h*

I would rather stamp-h.in removed from the list.

> -undep.sh
> +undep.sh*

Not ok since undep.sh.in is in the CVS.

The general rule should IMHO be that files which are in the CVS should
be included in a diff (and so be excluded from diff_ignore).

        Raimar

-- 
 email: rf13@xxxxxxxxxxxxxxxxx
 "- Amiga Y2K fixes (a bit late, wouldn't you say?)"
    -- Linus Torvalds about linux 2.4.0 at 4 Jan 2001