[macports-ports] branch master updated: Multiple ports: add missing required variables

Zero King l2dy at macports.org
Fri Oct 13 11:07:48 UTC 2017

On Thu, Oct 05, 2017 at 10:29:52PM -0500, Ryan Schmidt wrote:
>On Oct 5, 2017, at 17:49, Kurt Hindenburg wrote:
>> Kurt Hindenburg (kurthindenburg) pushed a commit to branch master
>> in repository macports-ports.
>> https://github.com/macports/macports-ports/commit/e8d12170f2e412c25e53a770acd58cb2f40e27ea
>> The following commit(s) were added to refs/heads/master by this push:
>>      new e8d1217  Multiple ports: add missing required variables
>> --- a/sysutils/detach/Portfile
>> +++ b/sysutils/detach/Portfile
>> @@ -11,6 +11,7 @@ long_description        The  command is a grungy little program for \
>>  			executing programs in the background, \
>>  			without use of a control  terminal. \
>>  			(In the style of most common daemon processes...)
>> +homepage        https://www.macports.org
>>  master_sites    http://ftp.ntnu.no/old/pub/unix/utils/
>>  checksums       md5 843c6ff1590a56c1733c958a86cd8a93
>>  pre-configure   { reinplace "s|/usr/local|${destroot}${prefix}|g" ${worksrcpath}/Makefile
>This is the only change here I'd disagree with. We should only use that homepage on software we developed. For software that doesn't have a web page, using the download location as the homepage is probably acceptable. It's also acceptable not to set a homepage in those cases, in my opinion. "port info" simply omits the homepage information in that case, and "port gohome" knows to display an error message, so nothing breaks if there is no homepage set.

Then you should "fix" the obsolete 1.0 PortGroup

>homepage        https://www.macports.org/

Best regards,
Zero King
-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/x-pkcs7-signature
Size: 3612 bytes
Desc: not available
URL: <http://lists.macports.org/pipermail/macports-dev/attachments/20171013/f9af1626/attachment.bin>

More information about the macports-dev mailing list