Re: [gentoo-dev] [patch] golang-vcs-snapshot.eclass: add vendoring of external dependencies

["William L. Thomson Jr." <wlt-ml@o-sinc.com>]

[-- Attachment #1: Type: text/plain, Size: 949 bytes --]

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.

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 195 bytes --]