On Mon, 11 Jan 2021, Elan Ruusamäe wrote:

> 
> On 09.01.2021 11:51, baggins wrote:
> > +   # Unlock database for rebuild, safe since this is posttrans
> > +   %{__rm} -rf/var/lib/rpm/.rpm.lock >/dev/null 2>/dev/null || :
> 
> it's likely a file, so -r is superfluous

Yeah, removed.
> 
> and why hide unlink errors? I prefer to see "permission denied" or 
> similar errors if rm failed. easier to diagnose issues.

All you could see is "no such file or directory", which is just noise.
This script was like this for years.

-- 
Jan Rękorajski                    | PLD/Linux
SysAdm | baggins<at>pld-linux.org | http://www.pld-linux.org/
_______________________________________________
pld-devel-en mailing list
pld-devel-en@lists.pld-linux.org
http://lists.pld-linux.org/mailman/listinfo/pld-devel-en

Reply via email to