Failing buildword due to execution permission (with fix)

[Bryan Drewery]

On 11/9/2015 6:01 PM, Garrett Cooper wrote:
> 
>> On Nov 9, 2015, at 17:27, Bryan Drewery <bdrewery at FreeBSD.org> wrote:
>>
>>> On 11/9/2015 5:17 PM, Garrett Cooper wrote:
>>>
>>>> On Nov 9, 2015, at 16:13, Bryan Drewery <bdrewery at FreeBSD.org> wrote:
>>>
>>>
>>> ...
>>>
>>>> If this is a shell file then it is best to invoke it with 'sh' rather
>>>> than a chmod/#!. The src checkout should be noexec-safe.
>>>
>>> Right. I think it'd be a good idea for me to hunt down other issues though in the build by setting -o noexec.
>>>
>>>
>>> The only thing that concerns me with doing that is that it could result in weirdness, e.g. The osreldate.h generation script in include/ .
>>
>> It prepends 'sh'.
>>
>> include/Makefile:MK_OSRELDATE_SH=       ${.CURDIR}/mk-osreldate.sh
>> include/Makefile:           sh ${MK_OSRELDATE_SH}
> 
> Yeah... I forgot.
> 
> I wrote up that patch at iX, and it was iterated over a bit. I was just remembering what happens when you use ${SHELL} (hint: no bueno if your build is kicked off with a csh/non-POSIX sh..).
> 

I actually wrote up a patch recently to use ${SH} in all places of 'sh'
and '/bin/sh', and noted on SHELL?= that was not useful to use, but did
not commit it (yet).


-- 
Regards,
Bryan Drewery

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 473 bytes
Desc: OpenPGP digital signature
URL: <http://lists.freebsd.org/pipermail/freebsd-current/attachments/20151109/1a61c71e/attachment.bin>