Bug#431248: piuparts: AssertionError

2007-07-04 Thread Ana Guerrero
On Sun, Jul 01, 2007 at 04:45:44PM +1000, Aníbal Monsalve Salazar wrote:
 I'm getting the following messages after upgrading piuparts today:
 
 Traceback (most recent call last):
   File /usr/sbin/piuparts, line 1232, in ?
 main()
   File /usr/sbin/piuparts, line 1217, in main
 if install_and_upgrade_between_distros(args, packages):
   File /usr/sbin/piuparts, line 936, in install_and_upgrade_between_distros
 root_info = chroot.save_meta_data()
   File /usr/sbin/piuparts, line 621, in save_meta_data
 assert dirpath[:4] == root
 AssertionError


This bug is caused by m4. From piuparts-0.21/Makefile :

piuparts.py: piuparts.py.in
m4 -D __PIUPARTS_VERSION__=$$(dpkg-parsechangelog | grep ^Version: \
| cut -d' ' -f2)  $  $@

m4 not only replaces __PIUPARTS_VERSION__ for the current version, it makes
another weird substitutions in the piuparts.py that triggers this problem.
[1]. If somebody speaks m4 and know what this happens please share it :D

So John, if you do not mind, i'm going to drop m4 and just use sed to make 
this substitution :)


Ana

[1]
Diff from installed piuparts agains the source code:

diff piuparts-0.21/piuparts.py.in /usr/sbin/piuparts
33c33
 VERSION = __PIUPARTS_VERSION__
---
 VERSION = 0.21
288c288
 (fd, path) = tempfile.mkstemp(dir=settings.tmpdir)
---
 (fd, path) = tempfile.dir=settings.tmpdirUoPUp5
621,622c621,622
 assert dirpath[:len(root)] == root
 if dirpath[:len(proc) + 1] in [proc, proc + /]:
---
 assert dirpath[:4] == root
 if dirpath[:4 + 1] in [proc, proc + /]:
631c631
 dict[name[len(root):]] = (st, target)
---
 dict[name[4:]] = (st, target)
643c643
 pkg = basename[:-len(.list)]
---
 pkg = basename[:-7]
661c661
 count = len(output.split(\n)) - 1
---
 count = 18 - 1
696c696
 name = name[len(self.name):]
---
 name = name[9:]
804c804
 if line[:len(Package:)] == Package::
---
 if line[:10] == Package::
999c999
 if len(parts)  2 and parts[0] == deb:
---
 if 5  2 and parts[0] == deb:
1152c1152
(not settings.basetgz or len(settings.debian_distros)  1):
---
(not settings.basetgz or 23  1):
1189c1189
 if len(settings.debian_distros) == 1:
---
 if 23 == 1:
 







Bug#431248: [Piuparts-devel] Bug#431248: piuparts: AssertionError

2007-07-04 Thread Ana Guerrero
On Wed, Jul 04, 2007 at 11:15:12AM -0600, John Wright wrote:
 tags 431248 +pending
 thanks
 
 On Wed, Jul 04, 2007 at 06:09:43PM +0200, Ana Guerrero wrote:
  On Sun, Jul 01, 2007 at 04:45:44PM +1000, Aníbal Monsalve Salazar wrote:
   I'm getting the following messages after upgrading piuparts today:
   
   Traceback (most recent call last):
 File /usr/sbin/piuparts, line 1232, in ?
   main()
 File /usr/sbin/piuparts, line 1217, in main
   if install_and_upgrade_between_distros(args, packages):
 File /usr/sbin/piuparts, line 936, in 
   install_and_upgrade_between_distros
   root_info = chroot.save_meta_data()
 File /usr/sbin/piuparts, line 621, in save_meta_data
   assert dirpath[:4] == root
   AssertionError
  
  
  This bug is caused by m4. From piuparts-0.21/Makefile :
  
  piuparts.py: piuparts.py.in
  m4 -D __PIUPARTS_VERSION__=$$(dpkg-parsechangelog | grep ^Version: \
  | cut -d' ' -f2)  $  $@
  
  m4 not only replaces __PIUPARTS_VERSION__ for the current version, it makes
  another weird substitutions in the piuparts.py that triggers this problem.
  [1]. If somebody speaks m4 and know what this happens please share it :D
 
 Ok, fixed by using m4's -P option.  Now, we get something more like
 what was indended:
 
 
 [EMAIL PROTECTED]:~/debian/piuparts/trunk$ make piuparts.py
 m4 -P -D __PIUPARTS_VERSION__=$(dpkg-parsechangelog | grep ^Version: | cut 
 -d' ' -f2)  piuparts.py.in  piuparts.py
 [EMAIL PROTECTED]:~/debian/piuparts/trunk$ diff -u piuparts.py.in piuparts.py
 --- piuparts.py.in  2007-07-03 23:34:30.0 -0600
 +++ piuparts.py 2007-07-04 11:10:07.0 -0600
 @@ -30,7 +30,7 @@
  
 
 
 -VERSION = __PIUPARTS_VERSION__
 +VERSION = 0.22
 
 
  import time
 
 
 I've committed the fix...


See the mail I just send you about this... we were both commiting different
fixes for this almost at the same time :(
I would rather skip m4.

Ana



Bug#431248: [Piuparts-devel] Bug#431248: piuparts: AssertionError

2007-07-04 Thread John Wright
tags 431248 +pending
thanks

On Wed, Jul 04, 2007 at 06:09:43PM +0200, Ana Guerrero wrote:
 On Sun, Jul 01, 2007 at 04:45:44PM +1000, Aníbal Monsalve Salazar wrote:
  I'm getting the following messages after upgrading piuparts today:
  
  Traceback (most recent call last):
File /usr/sbin/piuparts, line 1232, in ?
  main()
File /usr/sbin/piuparts, line 1217, in main
  if install_and_upgrade_between_distros(args, packages):
File /usr/sbin/piuparts, line 936, in 
  install_and_upgrade_between_distros
  root_info = chroot.save_meta_data()
File /usr/sbin/piuparts, line 621, in save_meta_data
  assert dirpath[:4] == root
  AssertionError
 
 
 This bug is caused by m4. From piuparts-0.21/Makefile :
 
 piuparts.py: piuparts.py.in
 m4 -D __PIUPARTS_VERSION__=$$(dpkg-parsechangelog | grep ^Version: \
   | cut -d' ' -f2)  $  $@
 
 m4 not only replaces __PIUPARTS_VERSION__ for the current version, it makes
 another weird substitutions in the piuparts.py that triggers this problem.
 [1]. If somebody speaks m4 and know what this happens please share it :D

Ok, fixed by using m4's -P option.  Now, we get something more like
what was indended:


[EMAIL PROTECTED]:~/debian/piuparts/trunk$ make piuparts.py
m4 -P -D __PIUPARTS_VERSION__=$(dpkg-parsechangelog | grep ^Version: | cut -d' 
' -f2)  piuparts.py.in  piuparts.py
[EMAIL PROTECTED]:~/debian/piuparts/trunk$ diff -u piuparts.py.in piuparts.py
--- piuparts.py.in  2007-07-03 23:34:30.0 -0600
+++ piuparts.py 2007-07-04 11:10:07.0 -0600
@@ -30,7 +30,7 @@
 


-VERSION = __PIUPARTS_VERSION__
+VERSION = 0.22


 import time


I've committed the fix...

John



Bug#431248: [Piuparts-devel] Bug#431248: piuparts: AssertionError

2007-07-04 Thread John Wright
On Wed, Jul 04, 2007 at 07:18:03PM +0200, Ana Guerrero wrote:
 On Wed, Jul 04, 2007 at 11:15:12AM -0600, John Wright wrote:
  tags 431248 +pending
  thanks
  
  On Wed, Jul 04, 2007 at 06:09:43PM +0200, Ana Guerrero wrote:
   On Sun, Jul 01, 2007 at 04:45:44PM +1000, Aníbal Monsalve Salazar wrote:
I'm getting the following messages after upgrading piuparts today:

Traceback (most recent call last):
  File /usr/sbin/piuparts, line 1232, in ?
main()
  File /usr/sbin/piuparts, line 1217, in main
if install_and_upgrade_between_distros(args, packages):
  File /usr/sbin/piuparts, line 936, in 
install_and_upgrade_between_distros
root_info = chroot.save_meta_data()
  File /usr/sbin/piuparts, line 621, in save_meta_data
assert dirpath[:4] == root
AssertionError
   
   
   This bug is caused by m4. From piuparts-0.21/Makefile :
   
   piuparts.py: piuparts.py.in
   m4 -D __PIUPARTS_VERSION__=$$(dpkg-parsechangelog | grep ^Version: \
 | cut -d' ' -f2)  $  $@
   
   m4 not only replaces __PIUPARTS_VERSION__ for the current version, it 
   makes
   another weird substitutions in the piuparts.py that triggers this problem.
   [1]. If somebody speaks m4 and know what this happens please share it :D
  
  Ok, fixed by using m4's -P option.  Now, we get something more like
  what was indended:
  
  
  [EMAIL PROTECTED]:~/debian/piuparts/trunk$ make piuparts.py
  m4 -P -D __PIUPARTS_VERSION__=$(dpkg-parsechangelog | grep ^Version: | cut 
  -d' ' -f2)  piuparts.py.in  piuparts.py
  [EMAIL PROTECTED]:~/debian/piuparts/trunk$ diff -u piuparts.py.in 
  piuparts.py
  --- piuparts.py.in  2007-07-03 23:34:30.0 -0600
  +++ piuparts.py 2007-07-04 11:10:07.0 -0600
  @@ -30,7 +30,7 @@
   
  
  
  -VERSION = __PIUPARTS_VERSION__
  +VERSION = 0.22
  
  
   import time
  
  
  I've committed the fix...
 
 
 See the mail I just send you about this... we were both commiting different
 fixes for this almost at the same time :(
 I would rather skip m4.

Ana,

Please feel free to revert and use sed.

John



Processed: Re: Bug#431248: piuparts: AssertionError

2007-07-03 Thread Debian Bug Tracking System
Processing commands for [EMAIL PROTECTED]:

 severity 431248 grave
Bug#431248: piuparts: AssertionError
Severity set to `grave' from `important'

 thanks
Stopping processing here.

Please contact me if you need assistance.

Debian bug tracking system administrator
(administrator, Debian Bugs database)


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]