Re: [gentoo-dev] [PATCH 1/2] Introduce multibuild_merge_root() to merge interim installs.

2013-03-24 Thread Ulrich Mueller
On Sat, 23 Mar 2013, Michał Górny wrote: +multibuild_merge_root() { + [...] + if type -P lockf /dev/null; then + # On BSD, we have 'lockf' wrapper. + tar -C ${src} -f - -c . \ + | lockf ${lockfile} tar -x -f - -C ${dest} + else + [...] +

Re: [gentoo-dev] [PATCH 1/2] Introduce multibuild_merge_root() to merge interim installs.

2013-03-24 Thread Michał Górny
On Sun, 24 Mar 2013 08:47:41 +0100 Ulrich Mueller u...@gentoo.org wrote: On Sat, 23 Mar 2013, Michał Górny wrote: +multibuild_merge_root() { + [...] +if type -P lockf /dev/null; then +# On BSD, we have 'lockf' wrapper. +tar -C ${src} -f -

Re: [gentoo-dev] [PATCH 1/2] Introduce multibuild_merge_root() to merge interim installs.

2013-03-24 Thread Ulrich Mueller
On Sun, 24 Mar 2013, Michał Górny wrote: +multibuild_merge_root() { + [...] + if type -P lockf /dev/null; then + # On BSD, we have 'lockf' wrapper. + tar -C ${src} -f - -c . \ + | lockf ${lockfile} tar -x -f - -C ${dest}

Re: [gentoo-dev] [PATCH 1/2] Introduce multibuild_merge_root() to merge interim installs.

2013-03-24 Thread Michał Górny
On Sun, 24 Mar 2013 14:40:58 +0100 Ulrich Mueller u...@gentoo.org wrote: On Sun, 24 Mar 2013, Michał Górny wrote: +multibuild_merge_root() { + [...] + if type -P lockf /dev/null; then + # On BSD, we have 'lockf' wrapper. + tar -C ${src} -f - -c

Re: [gentoo-dev] [PATCH 1/2] Introduce multibuild_merge_root() to merge interim installs.

2013-03-24 Thread Zac Medico
On 03/24/2013 07:13 AM, Michał Górny wrote: Oh, I didn't reply to use of userland. I thought about using that but that would mean that I'd have to check all the userlands we support. BSD and GNU should be the only relevant USERLAND values (no others are listed in profiles/desc/userland.desc).

Re: [gentoo-dev] [PATCH 1/2] Introduce multibuild_merge_root() to merge interim installs.

2013-03-23 Thread Ulrich Mueller
On Sat, 23 Mar 2013, Michał Górny wrote: +multibuild_merge_root() { + [...] + if type -P lockf /dev/null; then + # On BSD, we have 'lockf' wrapper. + tar -C ${src} -f - -c . \ + | lockf ${lockfile} tar -x -f - -C ${dest} + else + [...]

Re: [gentoo-dev] [PATCH 1/2] Introduce multibuild_merge_root() to merge interim installs.

2013-03-23 Thread Michał Górny
On Sat, 23 Mar 2013 18:00:41 +0100 Ulrich Mueller u...@gentoo.org wrote: On Sat, 23 Mar 2013, Michał Górny wrote: +multibuild_merge_root() { + [...] + if type -P lockf /dev/null; then + # On BSD, we have 'lockf' wrapper. + tar -C ${src} -f - -c . \ +

Re: [gentoo-dev] [PATCH 1/2] Introduce multibuild_merge_root() to merge interim installs.

2013-03-23 Thread Alec Warner
On Sat, Mar 23, 2013 at 9:26 AM, Michał Górny mgo...@gentoo.org wrote: This is mostly a copy from distutils-r1. The function does copy all the files from one location onto another, preserving whatever possible. It can be run in parallel too without the risk of race conditions. ---

Re: [gentoo-dev] [PATCH 1/2] Introduce multibuild_merge_root() to merge interim installs.

2013-03-23 Thread Michał Górny
On Sat, 23 Mar 2013 10:44:59 -0700 Alec Warner anta...@gentoo.org wrote: On Sat, Mar 23, 2013 at 9:26 AM, Michał Górny mgo...@gentoo.org wrote: + if [[ ${lock_fd} ]]; then + # Close the lock file when we are done with it. + #