Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 10 Oct 2005 14:10:55 +0000
From:      Alexey Dokuchaev <danfe@FreeBSD.org>
To:        Renato Botelho <garga@FreeBSD.org>
Cc:        cvs-ports@FreeBSD.org, cvs-all@FreeBSD.org, ports-committers@FreeBSD.org
Subject:   Re: cvs commit: ports/games/galaxyhack Makefile ports/games/galaxyhack/files patch-Makefile
Message-ID:  <20051010141055.GA17353@FreeBSD.org>
In-Reply-To: <200510101327.j9ADRXJL014167@repoman.freebsd.org>
References:  <200510101327.j9ADRXJL014167@repoman.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Mon, Oct 10, 2005 at 01:27:31PM +0000, Renato Botelho wrote:
> garga       2005-10-10 13:27:31 UTC
> 
>   FreeBSD ports repository
> 
>   Modified files:
>     games/galaxyhack     Makefile 
>     games/galaxyhack/files patch-Makefile 
>   Log:
>   o Fix wrong patch in "files" directory and convert DOS to UNIX text files in
>     post-extract.

CR/LF -> LF conversion is somewhat suboptimal: by doing -exec, find(1)
spawns ${REINPLACE_CMD} (and probably ${ECHO_CMD}, but this one is cheap
since it's a shell builtin) on every file it finds.  Correct way would
be to use xargs(1), as mentioned in section ``4.4 Patching'' of Porter's
Handbook.

./danfe



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20051010141055.GA17353>