diff options
| author | Zac Medico <zmedico@gentoo.org> | 2011-03-04 08:39:53 -0800 |
|---|---|---|
| committer | Zac Medico <zmedico@gentoo.org> | 2011-03-14 09:21:28 -0700 |
| commit | 47500360b3063b114904c0a7dc4817a2ac6390d4 (patch) | |
| tree | 2cdf402a5d5de7bb3632d090ae35ac2187da027f /pym/portage | |
| parent | cf7984f4bf2a050be26551c4e696a905711f3c8c (diff) | |
| download | portage-47500360b3063b114904c0a7dc4817a2ac6390d4.tar.gz portage-47500360b3063b114904c0a7dc4817a2ac6390d4.tar.bz2 portage-47500360b3063b114904c0a7dc4817a2ac6390d4.zip | |
config: quote overlays containing spaces
This will fix bug #357297.
Diffstat (limited to 'pym/portage')
| -rw-r--r-- | pym/portage/package/ebuild/config.py | 23 |
1 files changed, 13 insertions, 10 deletions
diff --git a/pym/portage/package/ebuild/config.py b/pym/portage/package/ebuild/config.py index b63cff145..6981e24ca 100644 --- a/pym/portage/package/ebuild/config.py +++ b/pym/portage/package/ebuild/config.py @@ -501,21 +501,24 @@ class config(object): self._ppropertiesdict = portage.dep.ExtendedAtomDict(dict) self._penvdict = portage.dep.ExtendedAtomDict(dict) - """ repoman controls PORTDIR_OVERLAY via the environment, so no - special cases are needed here.""" - - overlays = shlex_split(self.get('PORTDIR_OVERLAY', '')) - if overlays: - new_ov = [] - for ov in overlays: + # repoman controls PORTDIR_OVERLAY via the environment, so no + # special cases are needed here. + portdir_overlay = shlex_split(self.get("PORTDIR_OVERLAY", "")) + new_ov = [] + if portdir_overlay: + whitespace_re = re.compile(r"\s") + for ov in portdir_overlay: ov = normalize_path(ov) if os.path.isdir(ov): + if whitespace_re.search(ov) is not None: + ov = portage._shell_quote(ov) new_ov.append(ov) else: writemsg(_("!!! Invalid PORTDIR_OVERLAY" " (not a dir): '%s'\n") % ov, noiselevel=-1) - self["PORTDIR_OVERLAY"] = " ".join(new_ov) - self.backup_changes("PORTDIR_OVERLAY") + + self["PORTDIR_OVERLAY"] = " ".join(new_ov) + self.backup_changes("PORTDIR_OVERLAY") locations_manager.set_port_dirs(self["PORTDIR"], self["PORTDIR_OVERLAY"]) @@ -2129,7 +2132,7 @@ class config(object): def thirdpartymirrors(self): if getattr(self, "_thirdpartymirrors", None) is None: profileroots = [os.path.join(self["PORTDIR"], "profiles")] - for x in self["PORTDIR_OVERLAY"].split(): + for x in shlex_split(self.get("PORTDIR_OVERLAY", "")): profileroots.insert(0, os.path.join(x, "profiles")) thirdparty_lists = [grabdict(os.path.join(x, "thirdpartymirrors")) for x in profileroots] self._thirdpartymirrors = stack_dictlist(thirdparty_lists, incremental=True) |