[oe-commits] Paul Eggleton : buildhistory_analysis: correctly handle whitespace when splitting lists

[git at git.openembedded.org]

Module: openembedded-core.git
Branch: master
Commit: 15ad5d2c0e92fefdbb7c0cf064134b1cabfd84ac
URL:    http://git.openembedded.org/?p=openembedded-core.git&a=commit;h=15ad5d2c0e92fefdbb7c0cf064134b1cabfd84ac

Author: Paul Eggleton <paul.eggleton at linux.intel.com>
Date:   Thu Jan 19 10:32:10 2012 +0000

buildhistory_analysis: correctly handle whitespace when splitting lists

Don't specify any argument to the split() function when handling changes
to list type variables (e.g. PACKAGES) so that the values are split by
any whitespace and only split once for a block of multiple whitespace
characters.

Signed-off-by: Paul Eggleton <paul.eggleton at linux.intel.com>
Signed-off-by: Richard Purdie <richard.purdie at linuxfoundation.org>

---

 meta/lib/oe/buildhistory_analysis.py |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/meta/lib/oe/buildhistory_analysis.py b/meta/lib/oe/buildhistory_analysis.py
index d3c0448..a2fa643 100644
--- a/meta/lib/oe/buildhistory_analysis.py
+++ b/meta/lib/oe/buildhistory_analysis.py
@@ -34,8 +34,8 @@ class ChangeRecord:
 
     def __str__(self):
         if self.fieldname in list_fields:
-            aitems = self.oldvalue.split(' ')
-            bitems = self.newvalue.split(' ')
+            aitems = self.oldvalue.split()
+            bitems = self.newvalue.split()
             removed = list(set(aitems) - set(bitems))
             added = list(set(bitems) - set(aitems))
             return '%s: %s:%s%s' % (self.path, self.fieldname, ' removed "%s"' % ' '.join(removed) if removed else '', ' added "%s"' % ' '.join(added) if added else '')