Review: Approve code This looks good.
A small redundancy: there's no need for the .strip() because the split will remove any leading, trailing, or duplicate whitespace around the package names. -- https://code.launchpad.net/~gary/charms/oneiric/buildbot-slave/extra-packages/+merge/93515 Your team Launchpad Yellow Squad is subscribed to branch lp:~yellow/charms/oneiric/buildbot-slave/trunk. -- Mailing list: https://launchpad.net/~yellow Post to : [email protected] Unsubscribe : https://launchpad.net/~yellow More help : https://help.launchpad.net/ListHelp
