Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 29 Nov 2010 07:51:25 +0000
From:      Alexey Dokuchaev <danfe@FreeBSD.org>
To:        wen heping <wenheping@gmail.com>
Cc:        Wen Heping <wen@freebsd.org>, cvs-ports@freebsd.org, cvs-all@freebsd.org, ports-committers@freebsd.org
Subject:   Re: cvs commit: ports/Mk bsd.python.mk ports/lang/python27 Makefile distinfo pkg-plist ports/lang/python27/files patch-Lib-test-test_socket.py patch-Lib-test-test_threading.py patch-Lib_test_test_threading.py
Message-ID:  <20101129075125.GA58770@FreeBSD.org>
In-Reply-To: <AANLkTik-EKAcHxOzEM=p0CxwvVDyaCbcA5n%2BLfA3i%2BR_@mail.gmail.com>
References:  <201011290629.oAT6TD4u084226@repoman.freebsd.org> <20101129071448.GA56903@FreeBSD.org> <AANLkTik-EKAcHxOzEM=p0CxwvVDyaCbcA5n%2BLfA3i%2BR_@mail.gmail.com>

next in thread | previous in thread | raw e-mail | index | archive | help
On Mon, Nov 29, 2010 at 03:21:37PM +0800, Wen Heping wrote:
> 2010/11/29 Alexey Dokuchaev <danfe@freebsd.org>:
> > On Mon, Nov 29, 2010 at 06:29:13AM +0000, Wen Heping wrote:
> >> wen     2010-11-29 06:29:13 UTC
> >>
> >>  Added files:
> >>   patch-Lib-test-test_threading.py
> >>  Removed files:
> >>   patch-Lib_test_test_threading.py
> >
> > Can you explain why this patch was renamed? "Beautifying" the name in
> > this case does not warrant the repo churn.
> 
> Previous patch could not patch cleanly and I created a new one.
> When I created it I named it as other patches in this port.
> 
> Should it be forced use the previous name ?

My point is that once file is in CVS, we should try to use it even if
the name is not perfectly in line with standard scheme.  The rationale
behind this rule is roughly the same as why we are not renaming existing
`patch-xy' patches or patches containing ::, for example.

If existing patch name looks good enough (and it obviously does, since
the file name is essentially the same), I think you should have not
created new file (regardless of the contents).

./danfe



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