Tweak profile validation and warning logic a bit.

svn path=/main/trunk/; revision=9015

diff --git a/pym/_emerge/__init__.py b/pym/_emerge/__init__.py
index 48d2c0326051b594bf3a3032fc2ad9586235257b..869894d7859c4e65d7ede3c16c5139e3adb4a80b 100644 (file)
--- a/pym/_emerge/__init__.py
+++ b/pym/_emerge/__init__.py
@@ -6741,7 +6741,9 @@ def parse_opts(tmpcmdline, silent=False):
        return myaction, myopts, myfiles
 
 def validate_ebuild_environment(trees):
-       pass
+       for myroot in trees:
+               settings = trees[myroot]["vartree"].settings
+               settings.validate()
 
 def load_emerge_config(trees=None):
        kwargs = {}

diff --git a/pym/portage/__init__.py b/pym/portage/__init__.py
index c84ea2a28bc9b6dbe79d93a82352064655fa36b8..a6077c1e4e949a54a2d0ce220a970561f1f26488 100644 (file)
--- a/pym/portage/__init__.py
+++ b/pym/portage/__init__.py
@@ -1674,9 +1674,9 @@ class config(object):
 
                abs_profile_path = os.path.join(self["PORTAGE_CONFIGROOT"],
                        PROFILE_PATH.lstrip(os.path.sep))
-               if not os.path.islink(abs_profile_path) and \
+               if not self.profile_path or (not os.path.islink(abs_profile_path) and \
                        not os.path.exists(os.path.join(abs_profile_path, "parent")) and \
-                       os.path.exists(os.path.join(self["PORTDIR"], "profiles")):
+                       os.path.exists(os.path.join(self["PORTDIR"], "profiles"))):
                        writemsg("\a\n\n!!! %s is not a symlink and will probably prevent most merges.\n" % abs_profile_path,
                                noiselevel=-1)
                        writemsg("!!! It should point into a profile within %s/profiles/\n" % self["PORTDIR"])
@@ -6166,7 +6166,6 @@ def create_trees(config_root=None, target_root=None, trees=None):
        settings = config(config_root=config_root, target_root=target_root,
                config_incrementals=portage.const.INCREMENTALS)
        settings.lock()
-       settings.validate()
 
        myroots = [(settings["ROOT"], settings)]
        if settings["ROOT"] != "/":
@@ -6192,7 +6191,6 @@ def create_trees(config_root=None, target_root=None, trees=None):
                                backupenv.pop(k, None)
                settings.regenerate()
                settings.lock()
-               settings.validate()
                myroots.append((settings["ROOT"], settings))
 
        for myroot, mysettings in myroots: