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

John Marino freebsd.contact at marino.st
Fri Aug 1 06:14:35 UTC 2014


On 8/1/2014 08:05, Alexey Dokuchaev wrote:
> On Fri, Aug 01, 2014 at 08:02:21AM +0200, Antoine Brodin wrote:
>> Refusing _ in seperator seems total bullshit,  this convention has
>> been in place for 9 years, wtf!?
> 
> ... PATCH_PATH_SEPARATOR must be one of the [-+_] characters.  Where am I
> refusing _?
> 
> ./danfe
> 

I suspect he means "__" not "_"

logic flaw: You can't just replace "/" with "-" or "_" without altering
"-" or "_" in the filename.

example, given these files:
  ${WRKSRC}/alpha-bravo
  ${WRKSRC}/alpha/bravo

if you replace "/" with "-", both have the same suggested patch name.
You need to alter the hyphen in alpha-bravo first.  Maybe this is an
existing logic flaw in makepatch.

And no, this is not contrived.  I've actually run into this.

So what would this version of makepatch do with my example?

John



More information about the svn-ports-all mailing list