squash and merge, license documentation, "size" documentation

[Zero King]

On Fri, Aug 17, 2018 at 09:44:47AM -0400, Perry E. Metzger wrote:
>As many of you are aware, I merge a significant fraction of the pull
>requests these days. A few requests to make it easier, these three
>things on their own represent a surprising fraction of the
>communication I need to have with contributors:
>
>1. We have squash and merge disabled. This means that I often have to
>coach people through doing the squash on their own. If we enabled
>that, often it would not be necessary because I could do it for them.
>Is there a good reason we have that disabled?

None, AFAIK. See also this thread https://marc.info/?l=macports-dev&m=151901677718048&w=2.

>2. The guide has poor documentation on what license strings are
>acceptable, and port lint doesn't check that the license string is
>an acceptable string. If these were fixed, it would be much easier to
>direct people to the correct license string etc.

I would check https://trac.macports.org/wiki/PortfileRecipes#licensekeyword.

>3. There's no real documentation of the "size" parameter to
>checksums, and I'm constantly asking people to add the size. Note
>that I don't think "size" is a reasonable thing to require given that
>finding two files of the same size with the same SHA-2 hash is
>probably worth a doctoral dissertation at this point, but if we are
>going to require it (why do we require it?), it should be documented,
>and port lint should complain that it isn't there, and doing
>port -v checksums should spit it out if it isn't there.

`port -v checksums` does so if any one of the checksums is incorrect.

>Perry
>-- 
>Perry E. Metzger		perry at piermont.com

-- 
Zero
-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/pkcs7-signature
Size: 3533 bytes
Desc: not available
URL: <http://lists.macports.org/pipermail/macports-dev/attachments/20180817/8b373507/attachment.bin>