On Sun, Apr 17, 2022 at 11:42:35AM +0100, Alexey Sokolov wrote:
> 09.04.2022 17:37, Michał Górny пишет:
> > +           if [[ ${GPEP517_TESTING} ]]; then
> > +                   rm -rf build || die
> > +           else
> > +                   esetup.py clean -a
> > +           fi
> >     fi
> >   }
> >   
> 
> rm -r build || die
> 
> -f makes it not fail, which makes ||die useless

-f mostly just means it wont error out if the directory is missing
which I believe is what is wanted here (this covers a lot of packages
so there's less certainty than in an ebuild). Then rm could fail for
other reasons such as failure to call the command, or directory not
being removable (like no permission to), so should still || die

-- 
ionen

Attachment: signature.asc
Description: PGP signature

Reply via email to