On Tue, 02 Oct 2007 12:38:21 +0200 Oliver Falk <oliver linux-kernel at> wrote: > It might be likely possible, that nobody using Fedora and using pungi > ever tried creating isos, is that right? I don't think so. But it > might likely be possible, that nobody ever tried to create non-DVD > isos That's far more likely. In fact, my last couple releases the validation I did didn't include split iso tests, so this code path is a bit unused. > and therefor nobody encountered the following problem: > > [root gosa pungi]# sh ~/bin/run_pungi.sh > Traceback (most recent call last): > File "/usr/bin/pungi", line 178, in <module> > main() > File "/usr/bin/pungi", line 43, in main > config.set('default', 'cdsize', part.size) > File "/usr/lib/python2.5/ConfigParser.py", line 639, in set > raise TypeError("option values must be strings") > TypeError: option values must be strings > > > This came up after I enabled the following in my pungi (ks/kickstart) > config: > > part iso --size=650 Ok, yeah, this is just clearly a bug in pungi. Pykickstart hands this back as an int, but pungi is trying to stuff that into a ConfigParser set, which requires strings. The correct thing to do here is to wrap that stuffing in str(). I've made that change and done some local testing and this appears to solve the problem. I'll commit to tip and get a new build out today. -- Jesse Keating Fedora -- All my bits are free, are yours?
Description: PGP signature