Date: Fri, 8 Dec 2017 18:07:06 -0800 From: Eitan Adler <eadler@freebsd.org> To: rgrimes@freebsd.org Cc: cem@freebsd.org, src-committers <src-committers@freebsd.org>, "svn-src-all@freebsd.org" <svn-src-all@freebsd.org>, "svn-src-head@freebsd.org" <svn-src-head@freebsd.org> Subject: Re: svn commit: r326614 - in head: etc/mtree usr.bin usr.bin/sponge usr.bin/tee Message-ID: <CAF6rxgmexdxt7q1=9%2BNZoRAZmr%2BMHDdhtPZAcdw-DcjjhLPa=A@mail.gmail.com> In-Reply-To: <201712082011.vB8KBUsA068552@pdx.rh.CN85.dnsmgr.net> References: <CAG6CVpUm5KZYox_yNgg59ZiAM45rKp-69PnpdJTCJt6PdMO8_Q@mail.gmail.com> <201712082011.vB8KBUsA068552@pdx.rh.CN85.dnsmgr.net>
next in thread | previous in thread | raw e-mail | index | archive | help
On 8 December 2017 at 12:11, Rodney W. Grimes
<freebsd@pdx.rh.cn85.dnsmgr.net> wrote:
> Ping, I have not seen this reverted and recommitted with a reasonable
> explination as to why this change is actually desired.
Sorry, work has been busy and as the crux was reverted it didn't feel
as urgent[0]. The answer is that shell scripts in scripts don't need
to be chmod +x to work. In fact most are not.
Of the tests I found from a simple search:
65 are chmod +x
84 are chmod -x
The entirety of the changed config is:
"chmod": {
"type": "chmod",
+ "exclude": "(/tests/)"
},
which simply disables this test during 'arc diff'.
[0] I still owe everyone a reply the original thread
--
Eitan Adler
Source, Ports, Doc committer
Bugmeister, Ports Security teams
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAF6rxgmexdxt7q1=9%2BNZoRAZmr%2BMHDdhtPZAcdw-DcjjhLPa=A>