[78818] trunk/dports/textproc/multimarkdown/Portfile

[Ryan Schmidt]

On May 23, 2011, at 22:53, Ryan Schmidt wrote:

> 
> On May 23, 2011, at 13:56, singingwolfboy at macports.org wrote:
> 
>> Revision: 78818
>>         http://trac.macports.org/changeset/78818
>> Author:   singingwolfboy at macports.org
>> Date:     2011-05-23 11:56:28 -0700 (Mon, 23 May 2011)
>> Log Message:
>> -----------
>> multimarkdown: fixed incorrect permissions setting on multimarkdown binary
>> 
>> Modified Paths:
>> --------------
>>   trunk/dports/textproc/multimarkdown/Portfile
>> 
>> Modified: trunk/dports/textproc/multimarkdown/Portfile
>> ===================================================================
>> --- trunk/dports/textproc/multimarkdown/Portfile	2011-05-23 18:44:03 UTC (rev 78817)
>> +++ trunk/dports/textproc/multimarkdown/Portfile	2011-05-23 18:56:28 UTC (rev 78818)
>> @@ -6,6 +6,7 @@
>> set git_name        peg-${name}
>> set git_author      fletcher
>> version             3.0b11
>> +revision            1
>> categories          textproc
>> license             GPL or MIT
>> maintainers         singingwolfboy openmaintainer
>> @@ -43,7 +44,7 @@
>> build.target        docs
>> 
>> destroot {
>> -    xinstall -m 644 ${worksrcpath}/${name} ${destroot}${prefix}/bin/
>> +    xinstall -m 744 ${worksrcpath}/${name} ${destroot}${prefix}/bin/
> 
> Why wouldn't you use 755? (or not specify it at all, which would use 755?)

Using 744 means only the owner (i.e. root) can execute the program. multimarkdown is a program that users are expected to use, so the correct permissions to use would be 755. Not doing this leads to:

$ multimarkdown
-bash: /opt/local/bin/multimarkdown: Permission denied

Fixed in https://trac.macports.org/changeset/78852