svn commit: r331046 - head/Mk/Scripts

[Eitan Adler]

On Mon, Oct 21, 2013 at 2:47 AM, Baptiste Daroussin <bapt at freebsd.org> wrote:
> On Sun, Oct 20, 2013 at 03:01:38PM -0400, Eitan Adler wrote:
>> On Sun, Oct 20, 2013 at 2:12 PM, Alex Kozlov <ak at freebsd.org> wrote:
>> > Author: ak
>> > Date: Sun Oct 20 18:12:49 2013
>> > New Revision: 331046
>> > URL: http://svnweb.freebsd.org/changeset/ports/331046
>> >
>> > Log:
>> >   - Add check for suid files and directories
>>
>> Can you please add an in-port variable to disable this (NEEDS_SUID /
>> NEEDS_ROOT) or remove this check?
>>
>> Warnings which may be ignored are not very useful.
>>
>
> I do not think this is really a good way. First NEED_ROOT is a temporary knob
> and should die as soon a possible as everything should be doable as a user.

As I said to Alex privately,

it should be possible to run check-sanity across the tree and get
*zero* results.  If some warnings could be ignored people will start
ignoring all of them or people will start to "fix" them incorrectly.
Look at what happens with portlint warnings.

There needs to be an in-port method of quieting bogus warnings.


-- 
Eitan Adler
Source, Ports, Doc committer
Bugmeister, Ports Security teams