[Sugar-devel] [PATCH] Adding another path for lookup of mfg-data directory

Manuel Quiñones manuq at laptop.org
Tue Nov 22 05:47:53 EST 2011


The mfg-data directory is located in another path for some builds, so
the activity has to check in both places for existence.  This fixes
Log for olpc #11116.

Signed-off-by: Manuel Quiñones <manuq at laptop.org>
---
 logcollect.py |   18 +++++++++++++-----
 1 files changed, 13 insertions(+), 5 deletions(-)

diff --git a/logcollect.py b/logcollect.py
index a1251a4..82c1bba 100644
--- a/logcollect.py
+++ b/logcollect.py
@@ -51,6 +51,9 @@ import httplib
 import mimetypes
 import urlparse
 
+MFG_DATA_PATHS = ['/ofw/mfg-data/', '/proc/device-tree/mfg-data/']
+
+
 class MachineProperties:
     """Various machine properties in easy to access chunks.
     """
@@ -111,12 +114,17 @@ class MachineProperties:
                 return line[8:].strip()
 
     def _mfg_data(self, item):
-        """Return mfg data item from /ofw/mfg-data/"""
-        
-        if not os.path.exists('/ofw/mfg-data/'+item):
+        """Return mfg data item from mfg-data directory"""
+
+        mfg_path = None
+        for test_path in MFG_DATA_PATHS:
+            if os.path.exists(test_path + item):
+                mfg_path = test_path + item
+                break
+        if mfg_path == None:
             return ''
-        
-        v = self.__read_file('/ofw/mfg-data/'+item)
+
+        v = self.__read_file(mfg_path)
         # Remove trailing 0 character, if any:
         if v != '' and ord(v[len(v)-1]) == 0:
             v = v[:len(v)-1]
-- 
1.7.7.3



More information about the Sugar-devel mailing list