Re: [PATCH 04/15] prlimit: add warning diagnostic when attempting to change prlimit's limits

[Bernhard Voelker <mail@bernhard-voelker.de>]

On 11/16/2011 01:45 PM, Karel Zak wrote:
> On Mon, Nov 14, 2011 at 02:48:11AM +0100, Bernhard Voelker wrote:
>> >  +		if (lims[i].modify) {
>> >  +			if (!pid)
>> >  +				warnx(_("setting limits of the prlimit process is pointless"));
>
>   Well, I don't think that we want such warnings in our utils:-)  Let's
>   inform users about errors, the rest should be described in man pages.

hmm, then I tend to make it an error: prlimit acts too similar to
ulimit, but the *big* difference - and therefore the *big* trap for
the user - is that ulimit is a shell builtin which affects the current
shell process whereas prlimit runs in it's own process and, as a result,
will just do not have an effect without a PID.

Have a nice day,
Berny