[Sugar-devel] [PATCH sugar] Control Panel: making about my computer section hardware independent, OLPC #11232
quozl at laptop.org
Mon Oct 17 03:36:37 EDT 2011
On Mon, Oct 17, 2011 at 09:23:18AM +0200, Simon Schampijer wrote:
> def _parse_firmware_number(firmware_no):
> if firmware_no is None:
> firmware_no = _not_available
> # try to extract Open Firmware version from OLPC style
> # version string, e.g. "CL2 Q4B11 Q4B"
> fields = re.split(' +', firmware_no)
> if len(fields) == 3:
> firmware_no = fields
> return firmware_no
Again I disagree with this method, since it expects whitespace between
fields, and OpenFirmware does not guarantee whitespace.
Instead, you should extract byte positions. Skip six characters, then
extract seven characters. Trim any trailing spaces from the result.
Mitch and I discussed this during this last week.
More information about the Sugar-devel