Date: Tue, 19 Dec 2017 04:41:30 +0000 From: Conrad Meyer <cem@freebsd.org> To: Eitan Adler <eadler@freebsd.org> Cc: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: Re: svn commit: r326959 - head Message-ID: <CAG6CVpVkMUKBo9JQ_wsXjQUxcHJPsJD5g%2BEk%2BVx77rF--MNEKg@mail.gmail.com> In-Reply-To: <201712190338.vBJ3c6E1074327@repo.freebsd.org> References: <201712190338.vBJ3c6E1074327@repo.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
Thanks for fixing it. Conrad On Mon, Dec 18, 2017 at 7:38 PM Eitan Adler <eadler@freebsd.org> wrote: > Author: eadler > Date: Tue Dec 19 03:38:06 2017 > New Revision: 326959 > URL: https://svnweb.freebsd.org/changeset/base/326959 > > Log: > arc lint: ignore /tests/ in chmod > > 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 > > simply disable the check for test shell scripts. > > Recommit requested by: cem, rgrimes > > Modified: > head/.arclint > > Modified: head/.arclint > > ============================================================================== > --- head/.arclint Tue Dec 19 03:35:39 2017 (r326958) > +++ head/.arclint Tue Dec 19 03:38:06 2017 (r326959) > @@ -9,7 +9,8 @@ > "type": "spelling" > }, > "chmod": { > - "type": "chmod" > + "type": "chmod", > + "exclude": "(/tests/)" > }, > "merge-conflict": { > "type": "merge-conflict" > >
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAG6CVpVkMUKBo9JQ_wsXjQUxcHJPsJD5g%2BEk%2BVx77rF--MNEKg>