commit f6a74c53fc0e802d3a7b9882a2d04de132ebacca
author Ben Chan <benchan@chromium.org> Tue Feb 19 14:06:17 2013 -0800
committer ChromeBot <chrome-bot@google.com> Tue Feb 19 17:13:05 2013 -0800
tree 287df51f9b9f76bf53bf115ff02133940272f89b
parent c9f2999c3df1c1d267bbf916f862a8a2b66de11d

cros_disks: Handle the case when chmod() is not implemented by filesystems.

Some filesystem implementations do not support chmod(). This CL modifies
FilesystemTestFile in cros_disks.py to ignore chmod() errors raised as
OSError.

BUG=chromium-os:39041
TEST=Run platform_CrosDisksFilesystem with exFAT tests.

Change-Id: I5ca4c7f22ad4227f4ce2ba7c7153a6ea1757580d
Reviewed-on: https://gerrit.chromium.org/gerrit/43563
Reviewed-by: Chris Sosa <sosa@chromium.org>
Commit-Queue: Ben Chan <benchan@chromium.org>
Tested-by: Ben Chan <benchan@chromium.org>

client/cros/cros_disks.py

diff --git a/client/cros/cros_disks.py b/client/cros/cros_disks.py
index 025d4ce..9ee80f3 100644
--- a/client/cros/cros_disks.py
+++ b/client/cros/cros_disks.py
@@ -507,7 +507,8 @@
         with ExceptionSuppressor(IOError):
             with open(path, 'wb+') as f:
                 f.write(self._content)
-            os.chmod(path, self._mode)
+            with ExceptionSuppressor(OSError):
+                os.chmod(path, self._mode)
             return True
         return False