Patch naming policy (was: Re: [153574] trunk/dports/sysutils/skey)

Chris Jones jonesc at hep.phy.cam.ac.uk
Wed Oct 5 07:13:52 PDT 2016



> On 5 Oct 2016, at 2:56 pm, Rainer Müller <raimue at macports.org> wrote:
> 
> On 2016-10-05 01:45, Ryan Schmidt wrote:
>>> Patches should follow the patch-*.diff naming scheme, so this
>>> should be named patch-skeyprune-man8.diff or similar.
>>> 
>>> https://guide.macports.org/#development.patches.source
>> 
>> Popular opinion seems to be that we should relax that restriction.
>> 
>> I think it's reasonable if we change it to "patch-*.diff or
>> *.patch".
>> 
>> My primary gripe was with the way patches used to be named
>> "patch-foo" or "patch-foo.c" which caused editors to use incorrect
>> syntax highlighting. As long as we use a .diff or .patch extension,
>> to indicate to a syntax highlighter that this is a diff or patch
>> file, that should be fine.
> 
> The initial patch-* policy was adopted from FreeBSD ports. The filename
> extension .diff was added later for this reason.
> 
> If we want to change the patch naming policy, should we allow both
> *.diff and *.patch or would one file extension be better to avoid
> configuring editors twice?

My bet is any decent editor, at least those which support software development, would already support both.

> 
> Rainer
> _______________________________________________
> macports-dev mailing list
> macports-dev at lists.macosforge.org
> https://lists.macosforge.org/mailman/listinfo/macports-dev



More information about the macports-dev mailing list