Re: [gentoo-portage-dev] [PATCH 4/4] sync: Add backwards compat with SYNC_{UMASK,USER} variables

2014-12-07 Thread Michał Górny
Dnia 2014-12-06, o godz. 23:15:09 Zac Medico zmed...@gentoo.org napisał(a): On 12/05/2014 04:03 PM, Michał Górny wrote: Support SYNC_UMASK and SYNC_USER variables that were used in Funtoo Portage, as fallbacks to sync-umask and sync-user repo keys. ---

[gentoo-portage-dev] [PATCH] sync: allow overriding sync-umask for the repository

2014-12-07 Thread Michał Górny
--- Changes: * added a entry in the manpage man/portage.5| 5 + pym/portage/repository/config.py | 16 pym/portage/sync/controller.py | 3 +++ 3 files changed, 20 insertions(+), 4 deletions(-) diff --git a/man/portage.5 b/man/portage.5 index

[gentoo-portage-dev] [PATCH] sync: allow overriding sync-user for the repository

2014-12-07 Thread Michał Górny
--- Changes: * manpage entry * perform username lookup before UID lookup (POSIX agrees) * support specifying ':group' only * fix ValueError on errors man/portage.5| 12 ++ pym/portage/repository/config.py | 13 +++--- pym/portage/sync/controller.py | 51

Re: [gentoo-portage-dev] [PATCH 4/4] sync: Add backwards compat with SYNC_{UMASK,USER} variables

2014-12-07 Thread Zac Medico
On 12/07/2014 12:22 AM, Michał Górny wrote: Dnia 2014-12-06, o godz. 23:15:09 Zac Medico zmed...@gentoo.org napisał(a): On 12/05/2014 04:03 PM, Michał Górny wrote: Support SYNC_UMASK and SYNC_USER variables that were used in Funtoo Portage, as fallbacks to sync-umask and sync-user repo

Re: [gentoo-portage-dev] [PATCH 4/4] sync: Add backwards compat with SYNC_{UMASK,USER} variables

2014-12-07 Thread Michał Górny
Dnia 2014-12-07, o godz. 01:04:19 Zac Medico zmed...@gentoo.org napisał(a): On 12/07/2014 12:22 AM, Michał Górny wrote: Dnia 2014-12-06, o godz. 23:15:09 Zac Medico zmed...@gentoo.org napisał(a): On 12/05/2014 04:03 PM, Michał Górny wrote: Support SYNC_UMASK and SYNC_USER variables

Re: [gentoo-portage-dev] [PATCH] sync: allow overriding sync-user for the repository

2014-12-07 Thread Michał Górny
Dnia 2014-12-07, o godz. 10:01:23 Michał Górny mgo...@gentoo.org napisał(a): + if uid is not None Missing ':' here, I won't resend the patch for it :P. + spawn_kwargs[uid] = uid + self.usersync_uid = uid +

Re: [gentoo-portage-dev] [PATCH 1/2] bin/phase-helpers.shL Fix unpack() for deb2targz .xz types

2014-12-07 Thread Ulrich Mueller
On Sat, 06 Dec 2014, Zac Medico wrote: The PMS people should be *very* interested in any changes to unpack behavior like this. It supports behavior that will lead to failures for older versions of portage and other package managers. Some remarks: - The upstream deb2targz program supports

Re: [gentoo-portage-dev] [PATCH 1/2] bin/phase-helpers.shL Fix unpack() for deb2targz .xz types

2014-12-07 Thread Zac Medico
On 12/07/2014 03:04 AM, Ulrich Mueller wrote: On Sat, 06 Dec 2014, Zac Medico wrote: The PMS people should be *very* interested in any changes to unpack behavior like this. It supports behavior that will lead to failures for older versions of portage and other package managers. Some

Re: [gentoo-portage-dev] [PATCH 1/2] bin/phase-helpers.shL Fix unpack() for deb2targz .xz types

2014-12-07 Thread Zac Medico
On 12/06/2014 04:52 PM, Brian Dolbec wrote: From 4cb661d994cf8503c4459b8f7da7ee0f739a9826 Mon Sep 17 00:00:00 2001 From: Brian Dolbec dol...@gentoo.org Date: Sat, 6 Dec 2014 14:51:13 -0800 Subject: [PATCH 1/2] bin/phase-helpers.shL Fix unpack() for deb2targz .xz types MIME-Version: 1.0

Re: [gentoo-portage-dev] [PATCH 1/2] bin/phase-helpers.shL Fix unpack() for deb2targz .xz types

2014-12-07 Thread Ulrich Mueller
On Sun, 07 Dec 2014, Zac Medico wrote: Anyway, Brian's patch for xz support with debt2targz appears to be compatible with other package managers as well as AIX, so that seems like a good way to go. I still think that ar should always be used on platforms where GNU binutils is installed.

Re: [gentoo-portage-dev] [PATCH 1/2] bin/phase-helpers.shL Fix unpack() for deb2targz .xz types

2014-12-07 Thread Zac Medico
On 12/07/2014 10:05 AM, Ulrich Mueller wrote: On Sun, 07 Dec 2014, Zac Medico wrote: Anyway, Brian's patch for xz support with debt2targz appears to be compatible with other package managers as well as AIX, so that seems like a good way to go. I still think that ar should always be used

Re: [gentoo-portage-dev] [PATCH 1/2] bin/phase-helpers.shL Fix unpack() for deb2targz .xz types

2014-12-07 Thread Zac Medico
On 12/07/2014 09:54 AM, Zac Medico wrote: On 12/06/2014 04:52 PM, Brian Dolbec wrote: From 4cb661d994cf8503c4459b8f7da7ee0f739a9826 Mon Sep 17 00:00:00 2001 From: Brian Dolbec dol...@gentoo.org Date: Sat, 6 Dec 2014 14:51:13 -0800 Subject: [PATCH 1/2] bin/phase-helpers.shL Fix unpack() for

Re: [gentoo-portage-dev] [PATCH] sync: allow overriding sync-user for the repository

2014-12-07 Thread Zac Medico
On 12/07/2014 01:07 AM, Michał Górny wrote: Dnia 2014-12-07, o godz. 10:01:23 Michał Górny mgo...@gentoo.org napisał(a): +if uid is not None Missing ':' here, I won't resend the patch for it :P. +spawn_kwargs[uid] = uid +

Re: [gentoo-alt] Re: [gentoo-portage-dev] [PATCH 1/2] bin/phase-helpers.shL Fix unpack() for deb2targz .xz types

2014-12-07 Thread Zac Medico
On 12/07/2014 11:23 AM, Fabian Groffen wrote: On 07-12-2014 10:49:30 -0800, Zac Medico wrote: On 12/07/2014 10:47 AM, Ulrich Mueller wrote: On Sun, 07 Dec 2014, Zac Medico wrote: On 12/07/2014 10:37 AM, Ulrich Mueller wrote: It's sort of trivial, but here is a patch: From

Re: [gentoo-alt] Re: [gentoo-portage-dev] [PATCH 1/2] bin/phase-helpers.shL Fix unpack() for deb2targz .xz types

2014-12-07 Thread Ulrich Mueller
On Sun, 07 Dec 2014, Zac Medico wrote: On 12/07/2014 11:23 AM, Fabian Groffen wrote: FYI: % portageq envvar CHOST x86_64-apple-darwin13 % ar --version ar: illegal option -- - usage: ar -d [-TLsv] archive file ... [...] It's hard to whitelist it if doesn't support --version, so it

Re: [gentoo-alt] Re: [gentoo-portage-dev] [PATCH 1/2] bin/phase-helpers.shL Fix unpack() for deb2targz .xz types

2014-12-07 Thread Zac Medico
On 12/07/2014 11:41 AM, Ulrich Mueller wrote: On Sun, 07 Dec 2014, Zac Medico wrote: On 12/07/2014 11:23 AM, Fabian Groffen wrote: FYI: % portageq envvar CHOST x86_64-apple-darwin13 % ar --version ar: illegal option -- - usage: ar -d [-TLsv] archive file ... [...] It's hard to

Re: [gentoo-portage-dev] [PATCH] sync: allow overriding sync-user for the repository

2014-12-07 Thread Arfrever Frehtes Taifersar Arahesis
I would suggest to have a separate sync-group attribute and to support only user in sync-user attribute. signature.asc Description: This is a digitally signed message part.