[Piuparts-devel] Bug#574936: Bug#574936: piuparts: Option 'do-not-verify-signatures' makes piuparts fail

Cleto Martin Angelina cleto.martin at gmail.com
Tue Apr 20 18:47:08 UTC 2010


Hi, Holger!

> a.) your patch to delete empty arguments seems to modify the arguments, did
> you test if everything still works: (?)
>
>>>> command = 'huhu huhu'
>>>> command = [x for x in command if x]
>>>> print command
> ['h', 'u', 'h', 'u', ' ', 'h', 'u', 'h', 'u']
>>>> command = 'huhu huhu'
>>>> print command
> huhu huhu

In your example, 'command' is a string, but in piuparts code a command
is a list of strings. Thus:

command = ['huhu', '', 'huhu']
command = [x for x in command if x]
print command
['huhu', 'huhu']

Unless I missed something, function run requires a list of string. In
fact, there is an assertion cheking it. Probably, the code is too
general because it eliminates empty strings and None args. But I don't
see any problem on that way.


> b.) what is the debian_bundle issue?

Sorry about my poor explanation. In sid, if you import debian_budle
you get this message:

import debian_bundle
/usr/bin/ipython:1: DeprecationWarning: please use 'debian' instead of
'debian_bundle'
  #!/usr/bin/python

The issue is only the warning message. That's all :-)

Thanks for your attention, Holger (and for your patience).

Regards,
Cleto.





More information about the Piuparts-devel mailing list