Failing buildword due to execution permission (with fix)
Garrett Cooper
yaneurabeya at gmail.com
Tue Nov 10 02:01:52 UTC 2015
> 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'll do that soon-ish.
Thanks!
-NGie
More information about the freebsd-current
mailing list