svn commit: r363361 - in head/editors/fte: . files

Alexey Dokuchaev danfe at FreeBSD.org
Fri Aug 1 05:16:53 UTC 2014


On Wed, Jul 30, 2014 at 06:13:48PM +0200, John Marino wrote:
> On 7/30/2014 18:08, Alexey Dokuchaev wrote:
> > Given some thought over it, I propose the following changes to makepatch:
> > 
> >   - default to dash (minus, '-')
> >   - provide a command line option (-s?) or make(1) variable (PATH_SEPARATOR)
> >     to override it when needed
> >   - assert that forced value is not something insane ('__', '::', '../../',
> >     etc.) -- basically, match it over a prerefined list of acceptable values
> > 
> > I can cook up a patch unless John had beaten me on that already.
> 
> Nope, go for it.

Have a look (attached).  I've also fixed annoying bug (appending of leading
"./" in the ---/+++ lines of generated patches), and improved the comment.

./danfe
-------------- next part --------------
A non-text attachment was scrubbed...
Name: makepatch.diff
Type: text/x-diff
Size: 1575 bytes
Desc: not available
URL: <http://lists.freebsd.org/pipermail/svn-ports-all/attachments/20140801/fea578d2/attachment.diff>


More information about the svn-ports-all mailing list