Rudi Chiarito wrote:
> I found a few more problems with anaconda and kickstart files. It took me
> a whole day to figure them out, also because I had never programmed in
> Python before. ;)
> 
> First of all, as soon as I managed to find how to run anaconda in test
> mode (which is not THAT much helpful, anyway), I noticed that the
> kickstart was being removed every time I ran the installer! Line 308 of
> anaconda says "os.unlink(kickstart)"; I wonder why?!?
> 
> Then, if I passed the exact name of a card to "xconfig" in the kickstart
> file, I'd get an error at line 174 of file xf86config.py, which says
> 'self.devID = self.vidCards[self.primary]["NAME"]'. As far as I can tell,
> that's trying to access key NAME in the array of associative arrays
> vidCards at position self.primary (which equals to 0). The problem is
> that vidCards is set, a few lines before, to a string (the parameter
> "card"). I solved this by replacing "self.vidCards = [card]" with
> "self.vidCards = self.cards (card)" (method "cards" returns an
> associative array matching the string it is passed). I had still
> problems, though, because vidCards is still NOT an array of arrays. It
> looks to me as if the code in method "cards" is quite ambiguous, as it
> can return a single array or the whole card database (look at the two
> statements at the very end of it, "return card" and "return cards"). I'm
> not a Python expert, so I can't really tell how this should be fixed.
> I've found, though, that removing "[self.primary]" is enough here.
> 
 I tried Rudi's patch (and the mouse patch in todo.py) but it
 still blew up trying to write the X config file later on.
 Not knowing Python, I found a wizard who suggested a different
 change which worked for me (I only wanted to install the
 "Generic VGA compatible" card and "Generic Monitor" as part
 of a hands-free network install for a Beowulf cluster)

 Anyway, my wizard said that the problem was not with line 174,
 but with the line 170 which Rudi had also noted. It reads
   self.vidCards = [card]
 and line 174 should be replaced so as to read:
   self.vidCards = [self.cards(card)]
 which creates the array needed in line 174 (and also later on).

Sorry the above is not in proper context diff patch format, but
I wanted to get this minor correction out there to follow up
on Rudi's original stellar effort.

/John
-----
John Gerth - [EMAIL PROTECTED]



_______________________________________________
Redhat-devel-list mailing list
[EMAIL PROTECTED]
https://listman.redhat.com/mailman/listinfo/redhat-devel-list

Reply via email to