If there are any troubles reading the treeinfo file, return no addons.
Why is this read method called like that, instead of c.read(treeinfo)? It looks funny to me.
We can use ConfigParser.Error in the except clause to catch all ConfigParser errors.
----- "Ales Kozumplik" <firstname.lastname@example.org> wrote:
> Hi Chris,
> thanks for fixing this. One comment for this patch:
> On 10/21/2010 06:53 PM, Chris Lumens wrote:
> > + try:
> > + ConfigParser.read(c, treeinfo)
> > + variant = c.get("general", "variant")
> > + except:
> > + return retval
> We should get rid of all except: clauses without any exception,
> otherwise this code will sustain even SystemExit or KeyboardInterrupt.
> except Exception as e:
> log.info("failed to parse treeinfo file: %s" % e)
> return retval
> or something.
> Anaconda-devel-list mailing list
Anaconda-devel-list mailing list
|All times are GMT. The time now is 01:01 PM.|
Content Relevant URLs by vBSEO ©2007, Crawlability, Inc.