[oe] [PATCH v2] packaged-staging.bbclass: use 'os.unlink()' instead of spawning 'rm'
Enrico Scholz
enrico.scholz at sigma-chemnitz.de
Wed Apr 14 13:21:51 UTC 2010
"Sebastian Spaeth" <Sebastian at SSpaeth.de> writes:
>> @@ -227,7 +238,9 @@ python packagestage_scenefunc () {
>> - os.system('rm -f ' + stamp + '.*')
>> + for fname in glob.glob(stamp + '.*'):
>> + _package_unlink(fname)
>> +
>
>> @@ -260,8 +273,8 @@ python packagedstage_stampfixing_eventhandler() {
>> - os.system('rm -f ' + dir)
>> - os.system('rm -f ' + stamp)
>> + _package_unlink(dir)
>> + _package_unlink(stamp)
>
> Given you have made sure that those changed indentations are actually
> intended (I have not looked at the context of that patch):
File had already a mixed tab/whitespace indentation. I used the style
which I found at the top of the file (tabulators) and did not noticed
that it changed below.
Enrico
More information about the Openembedded-devel
mailing list