On Tue, 2012-01-03 at 15:54:38 +0000, Alexey Dokuchaev wrote:

> ...
> >   Revision  Changes    Path
> >   1.1       +11 -0     ports/chinese/tin/files/patch-tin.defaults (new)
> >   1.2       +0 -11     ports/chinese/tin/files/patch-tin_defaults (dead)
>
> This renaming of the patch file just because someone thought that dot
> looks better than underscore is gratuitous and should not have been
> done.  (Not to mention that things that this patch tries to accomplish
> are usually done with one simple REINPLACE_CMD line.)

I suspect the motivation for the change was to canonicalize the filename
as described in the PH.  But to do that correctly, I believe the
filename should be be: patch-doc-tin.defaults. :)  Anyway, as you noted,
such simple replacements belong in the Makefile.

-- 
Sahil Tandon
_______________________________________________
cvs-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/cvs-all
To unsubscribe, send any mail to "cvs-all-unsubscr...@freebsd.org"

Reply via email to