]> code.ossystems Code Review - openembedded-core.git/commitdiff
lib/package_manager: Use shutil.copy instead of bb.utils.copyfile for intercepts
authorRichard Purdie <richard.purdie@linuxfoundation.org>
Wed, 5 May 2021 21:13:10 +0000 (22:13 +0100)
committerRichard Purdie <richard.purdie@linuxfoundation.org>
Thu, 6 May 2021 10:15:53 +0000 (11:15 +0100)
If the scripts/postinst-intercepts is owned by root/root then the copyfile() calls
will fail due to chown issues. We don't care about ownership of these files so
use shutil.copy() instead which won't perform any chown.

Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
meta/lib/oe/package_manager/__init__.py

index 8e7128b19588d9b7d5ddfab2277a0a0d79bc63ce..4d22bc0296c1b7200105482c56cdadfcf3360245 100644 (file)
@@ -189,7 +189,7 @@ class PackageManager(object, metaclass=ABCMeta):
         bb.utils.remove(self.intercepts_dir, True)
         bb.utils.mkdirhier(self.intercepts_dir)
         for intercept in postinst_intercepts:
-            bb.utils.copyfile(intercept, os.path.join(self.intercepts_dir, os.path.basename(intercept)))
+            shutil.copy(intercept, os.path.join(self.intercepts_dir, os.path.basename(intercept)))
 
     @abstractmethod
     def _handle_intercept_failure(self, failed_script):