[bitbake-devel] [RFC PATCH 1/2] bitbake: track skipped packages

[Paul Eggleton]

On Thursday 02 June 2011 21:01:52 Joshua Lock wrote:
> This doesn't seem very Pythonic. I think you should just be able to do:
> bb.data.setVar("__SKIPPED", e, d)
> or possibly:
> bb.data.setVar("__SKIPPED", str(e), d)

Why is accessing args[0] un-Pythonic? As I understand it, args is provided 
(and documented) as the way to get access to the arguments to the exception, 
and the first argument is being used by us for the reason in the case of 
SkipPackage. At the moment e or str(e) will accomplish the same thing, but 
were we to add another argument then it seems to me that the results would be 
different.

Now I'm not a Python expert by any stretch of the imagination but I'd like to 
understand the reason why this usage might be considered undesirable.

Cheers,
Paul

-- 

Paul Eggleton
Intel Open Source Technology Centre