Re: [gentoo-dev] [PATCH v2] vcs-snapshot.eclass: set -o (--no-same-owner) when unpacking, bug #645182

[David Seifert <soap@gentoo.org>]

On Sun, 2018-01-21 at 00:26 +0000, Sergei Trofimovich wrote:
> Fixes build failures in unprivileged containers like bug #645182:
>   Package:    dev-python/pycparser-2.14
>   >>> Unpacking source...
>   tar: CHANGES: Cannot change ownership to uid 1000, gid 1000:
> Invalid argument
> 
> In such containers uid=0 can't really change file ownership.
> 
> Closes: https://bugs.gentoo.org/645182
> Signed-off-by: Sergei Trofimovich <slyfox@gentoo.org>
> ---
>  eclass/vcs-snapshot.eclass | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/eclass/vcs-snapshot.eclass b/eclass/vcs-snapshot.eclass
> index 3eff6995fae..2b3f73897ce 100644
> --- a/eclass/vcs-snapshot.eclass
> +++ b/eclass/vcs-snapshot.eclass
> @@ -67,7 +67,8 @@ vcs-snapshot_src_unpack() {
>  				# XXX: check whether the directory
> structure inside is
>  				# fine? i.e. if the tarball has
> actually a parent dir.
>  				mkdir "${destdir}" || die
> -				tar -C "${destdir}" -x --strip-
> components 1 \
> +				# -o (--no-same-owner) to avoid
> restoring original owner
> +				tar -C "${destdir}" -x -o --strip-
> components 1 \
>  					-f "${DISTDIR}/${f}" || die
>  				;;
>  			*)

Also, ultra-bikeshed: I think using "Signed-off-by" with yourself in it
is generally frowned upon and adds no real value.