Activity log for bug #882161

Date Who What changed Old value New value Message
2011-10-26 17:25:51 Brendan Donegan bug added bug
2011-10-26 17:26:29 Brendan Donegan bug task added checkbox-certification
2011-10-26 17:26:37 Brendan Donegan checkbox: status New Confirmed
2011-10-26 17:26:40 Brendan Donegan checkbox-certification: status New Confirmed
2011-10-26 17:26:50 Brendan Donegan checkbox: status Confirmed Triaged
2011-10-26 17:26:53 Brendan Donegan checkbox-certification: status Confirmed Triaged
2011-10-26 17:26:56 Brendan Donegan checkbox: importance Undecided Low
2011-10-26 17:26:59 Brendan Donegan checkbox-certification: importance Undecided Low
2011-10-26 17:27:46 Brendan Donegan description The cpu_topology script has a section of code to extract values of fields in /proc/cpuinfo that it is interested in. Unfortunately the logic gets tripped up where the word 'processor' is contained in the model name of a CPU, for example: model name: Intel(R) Celeron(R) M processor 900MHz Instead of using a simple 'string in string' statement it might be better to use string.startswith. I wouldn't go as far as saying regexes are necessary, but the possibility that the string might occur somewhere other than the start of the line needs to be guarded against. The cpu_topology script has a section of code to extract values of fields in /proc/cpuinfo that it is interested in. Unfortunately the logic gets tripped up where the word 'processor' is contained in the model name of a CPU, for example: model name: Intel(R) Celeron(R) M processor 900MHz Instead of using a simple 'string in string' statement it might be better to use string.startswith. I wouldn't go as far as saying regexes are necessary, but the possibility that the string might occur somewhere other than the start of the line needs to be guarded against. This problem only appears to affect one old system at the moment so the importance of a fix is not that great.
2011-10-27 16:24:33 Launchpad Janitor branch linked lp:~brendan-donegan/checkbox/bug882161_cpu_topology
2012-02-01 20:39:20 Launchpad Janitor branch linked lp:~roadmr/ubuntu/precise/checkbox/0.13.1
2012-02-01 21:20:26 Daniel Manrique branch unlinked lp:~roadmr/ubuntu/precise/checkbox/0.13.1
2012-02-01 21:21:20 Launchpad Janitor branch linked lp:~roadmr/ubuntu/precise/checkbox/0.13.1
2012-02-01 21:28:00 Daniel Manrique checkbox: status Triaged Fix Committed
2012-02-07 22:33:18 Launchpad Janitor branch linked lp:~roadmr/ubuntu/precise/checkbox/0.13.1-retry
2012-02-11 13:24:30 Launchpad Janitor branch linked lp:ubuntu/checkbox
2012-08-30 15:49:34 Brendan Donegan checkbox: status Fix Committed Fix Released
2013-05-30 13:26:53 Daniel Manrique checkbox-certification: status Triaged Incomplete
2017-11-27 09:54:10 Jeff Lane  checkbox-certification: status Incomplete Invalid