On Wed, 2010-03-17 at 18:39 +0100, Michael 'Mickey' Lauer wrote: > > > +def _package_unlink (f): > > > + import os > > > + try: > > > + os.unlink(f) > > > + return True > > > + except: > > > + return False > > > + > > While it won't hurt in this particular case, a catch-all 'except' > usually is frowned upon. Better catch specific errors and let other > errors be handled by the upper layers. I'd recommend catching IOError > only here.
We need to catch OSError too as this continues to leave behind directories. -- Tom Rini <[email protected]> Mentor Graphics Corporation _______________________________________________ Openembedded-devel mailing list [email protected] http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel
