From 9953048c6f4abda4eb7ded7dd3e992e574155a1d Mon Sep 17 00:00:00 2001 From: Fabio Erculiani Date: Sun, 1 Dec 2013 19:07:56 +0100 Subject: [PATCH] [entropy.client.package.action] install: apply tarball files ownership to extra download files as well --- .../interfaces/package/actions/install.py | 36 +++++++++++-------- 1 file changed, 21 insertions(+), 15 deletions(-) diff --git a/lib/entropy/client/interfaces/package/actions/install.py b/lib/entropy/client/interfaces/package/actions/install.py index 54baf250c..c2c4f795a 100644 --- a/lib/entropy/client/interfaces/package/actions/install.py +++ b/lib/entropy/client/interfaces/package/actions/install.py @@ -735,23 +735,29 @@ class _PackageInstallAction(_PackageInstallRemoveAction): # NOTE: fixup permissions in the image directory # the setup phase could have created additional users and groups - package_path = self._meta['pkgpath'] - prefix_dir = self._meta['imagedir'] - try: - entropy.tools.apply_tarball_ownership(package_path, prefix_dir) - except IOError as err: - msg = "%s: %s" % ( - brown(_("Error during package files permissions setup")), - err,) - self._entropy.output( - msg, - importance = 1, - level = "error", - header = darkred(" !!! ") + package_paths = [self._meta['pkgpath']] + for extra_download in self._meta['extra_download']: + package_paths.append( + self.get_standard_fetch_disk_path(extra_download['download']) ) - exit_st = 1 - return exit_st + for package_path in package_paths: + try: + entropy.tools.apply_tarball_ownership( + package_path, self._meta['imagedir']) + except IOError as err: + msg = "%s: %s" % ( + brown(_("Error during package files permissions setup")), + err,) + self._entropy.output( + msg, + importance = 1, + level = "error", + header = darkred(" !!! ") + ) + return 1 + + return 0 def _pre_install_phase(self): """