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