[150874] trunk/dports/lang/python32
Jeremy Huddleston Sequoia
jeremyhu at apple.com
Sat Aug 6 13:08:15 PDT 2016
FWIW, I prefer 'git format-patch' output for patches. If I actually had to rename patches per file, maintaining llvm would be a nightmare. =/
> On Aug 4, 2016, at 05:35, Clemens Lang <cal at macports.org> wrote:
>
> FWIW, I agree with Lawrence here. I think we should improve our patch
> naming, especially since it doesn't really matter *where* a patch comes
> from, either. Why do we really need to know whether a patch-*.diff file
> comes from MacPorts when compared to a different patch? We should be
> judging patches by their content, not their origin.
>
> What I've seen in other systems and would also recommend as a guideline
> for patches in MacPorts:
> - Write a commit message into the patch file. Patch headers can include
> arbitrary text, so type a message explaining what the patch does and
> why it is needed. Add references to any upstream tickets (I've
> started doing this for my ports according to OpenEmbedded's
> Upstream-Status tag [1]).
> - Use the summary line of the commit message as patch name, like git
> format-patch does it.
>
> I have in the past also ignored our patch naming guidelines when I
> thought it made sense; for example, I generally backport patches from
> upstream git repositories using <commithash>.patch, because that's what
> GitHub generates.
>
> [1] http://www.openembedded.org/wiki/Commit_Patch_Message_Guidelines#Patch_Header_Recommendations
>
> --
> Clemens
> _______________________________________________
> macports-dev mailing list
> macports-dev at lists.macosforge.org
> https://lists.macosforge.org/mailman/listinfo/macports-dev
-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/pkcs7-signature
Size: 4465 bytes
Desc: not available
URL: <https://lists.macosforge.org/pipermail/macports-dev/attachments/20160806/ebeaa5d2/attachment.p7s>
More information about the macports-dev
mailing list