Bug#696625: EXDEV not catched properly

2013-01-30 Thread Julien Cristau
On Mon, Jan 21, 2013 at 20:27:59 +0100, Christoph Berg wrote:

> tags 696625 patch
> severity 696625 grave
> thanks
> 
> I think this should be fixed in wheezy. It makes the package unusable
> on all systems where I tried it, including a cowbuilder chroot.
> 
I'm sure you're aware that bumping the severity means the package might
well get removed, not fixed, in wheezy.

Cheers,
Julien


signature.asc
Description: Digital signature


Bug#696625: EXDEV not catched properly

2013-01-21 Thread Christoph Berg
tags 696625 patch
severity 696625 grave
thanks

I think this should be fixed in wheezy. It makes the package unusable
on all systems where I tried it, including a cowbuilder chroot.

Christoph
-- 
c...@df7cb.de | http://www.df7cb.de/


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org



Bug#696625: EXDEV not catched properly

2012-12-24 Thread Christoph Berg
Package: autopkgtest
Version: 2.2.3
Severity: normal

adt-run: trace: ** init
adt-run: trace: ** start
adt-run: trace: $ vserver: adt-virt-null
adt-run: trace:  starting
adt-run: trace: ** open, scratch=None
adt-run: trace: ** cmdl = ['sh', '-c', 'exec "$@"', 'x']
adt-run: trace: * initialising
adt-run: trace: * no key generation needed
adt-run: trace: * reset
adt-run: trace:  builds ...
adt-run: trace:  deb deb0
adt-run: trace: * register what=deb0 deb_forbuilds=auto pkg=pgbouncer 
af=deb0-deb~pgbouncer_1.5.4-1_amd64.deb:pgbouncer_1.5.4-1_amd64.deb!|-
adt-run: unexpected, exceptional, error:
Traceback (most recent call last):
  File "/usr/bin/adt-run", line 1883, in main
process_actions()
  File "/usr/bin/adt-run", line 1823, in process_actions
'forbuilds',testbed.blamed)
  File "/usr/bin/adt-run", line 1486, in register
shutil.copy(af.read(), dest)
  File "/usr/lib/python2.7/shutil.py", line 117, in copy
if os.path.isdir(dst):
  File "/usr/lib/python2.7/genericpath.py", line 41, in isdir
st = os.stat(s)
TypeError: coercing to Unicode: need string or buffer, instance found
adt-run: trace: ** stop
adt-run: trace: ** close, 
scratch=tb-scratch~/tmp/tmp.WCCrAUmben:-/|/tmp/tmp.WCCrAUmben/!

This patch fixes the issue:

--- adt-run.orig2012-12-24 09:07:54.0 +0100
+++ adt-run 2012-12-24 09:07:57.0 +0100
@@ -1482,7 +1482,7 @@
try: os.link(af.read(), dest.write())
except (IOError,OSError), oe:
if oe.errno != errno.EXDEV: raise e
-   shutil.copy(af.read(), dest)
+   shutil.copy(af.read(), dest.write())
 
if act.ah['deb_'+forwhat] == 'install':
b.install.append(pkg)

PS: git://git.debian.org/git/autopkgtest/autopkgtest.git (from the Vcs
header) seems pretty out of date

Christoph
-- 
c...@df7cb.de | http://www.df7cb.de/


signature.asc
Description: Digital signature