On Thursday, March 9, 2017 1:29:55 PM EST Michael Orlitzky wrote: > On 03/09/2017 01:21 PM, William L. Thomson Jr. wrote: > > Pretty sure no need for the ||die, rm -fr should never fail. > > > > rm -fr "${VENDOR_PATH}/${VENDORPN}" || die > > $ rm -rf /bin/cp || echo "it failed" > rm: cannot remove '/bin/cp': Permission denied > it failed Under what circumstances? It is possible, but not to likely, as the dirs are part of a package unpacked. User would have permissions. If portage is not being run as root. If run as root, only extended file attributions ( immutable ) can prevent root from removing. Not likely set in a packaged unpacked. This part of the eclass is handling sources from VC. I am not sure how they can be pulled in with permissions that would prevent what ever user portage is running under from removing a directory. Seems like it is not possible to generate the above permission issue. -- William L. Thomson Jr.