[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