public inbox for gentoo-commits@lists.gentoo.org
 help / color / mirror / Atom feed
From: "Zac Medico" <zmedico@gentoo.org>
To: gentoo-commits@lists.gentoo.org
Subject: [gentoo-commits] proj/portage:master commit in: pym/portage/tests/resolver/, pym/portage/tests/ebuild/, pym/_emerge/
Date: Mon, 16 May 2011 20:12:41 +0000 (UTC)	[thread overview]
Message-ID: <bd486e676cf4fb1893f8d06220c1f60ed04760f2.zmedico@gentoo> (raw)

commit:     bd486e676cf4fb1893f8d06220c1f60ed04760f2
Author:     Sebastian Luther <SebastianLuther <AT> gmx <DOT> de>
AuthorDate: Mon May 16 20:06:40 2011 +0000
Commit:     Zac Medico <zmedico <AT> gentoo <DOT> org>
CommitDate: Mon May 16 20:09:25 2011 +0000
URL:        http://git.overlays.gentoo.org/gitweb/?p=proj/portage.git;a=commit;h=bd486e67

Enable --autounmask by default

---
 pym/_emerge/depgraph.py                            |    4 ++--
 pym/portage/tests/ebuild/test_config.py            |    2 ++
 pym/portage/tests/resolver/test_autounmask.py      |    2 +-
 .../tests/resolver/test_circular_dependencies.py   |    1 -
 pym/portage/tests/resolver/test_multirepo.py       |    3 +++
 .../tests/resolver/test_old_dep_chain_display.py   |    2 ++
 pym/portage/tests/resolver/test_simple.py          |    2 +-
 pym/portage/tests/resolver/test_slot_collisions.py |    3 ++-
 8 files changed, 13 insertions(+), 6 deletions(-)

diff --git a/pym/_emerge/depgraph.py b/pym/_emerge/depgraph.py
index 2a6d1b1..1de147a 100644
--- a/pym/_emerge/depgraph.py
+++ b/pym/_emerge/depgraph.py
@@ -424,7 +424,7 @@ class _dynamic_depgraph_config(object):
 		self._skip_restart = False
 		self._backtrack_infos = {}
 
-		self._autounmask = depgraph._frozen_config.myopts.get('--autounmask', 'n') == True
+		self._autounmask = depgraph._frozen_config.myopts.get('--autounmask') != 'n'
 		self._success_without_autounmask = False
 		self._traverse_ignored_deps = False
 
@@ -3317,7 +3317,7 @@ class depgraph(object):
 		if pkg.visible:
 			return True
 
-		if self._frozen_config.myopts.get('--autounmask', 'n') is not True:
+		if not self._dynamic_config._autounmask:
 			return False
 
 		pkgsettings = self._frozen_config.pkgsettings[pkg.root]

diff --git a/pym/portage/tests/ebuild/test_config.py b/pym/portage/tests/ebuild/test_config.py
index 7640317..bc10441 100644
--- a/pym/portage/tests/ebuild/test_config.py
+++ b/pym/portage/tests/ebuild/test_config.py
@@ -168,6 +168,7 @@ class ConfigTestCase(TestCase):
 		test_cases = (
 				ResolverPlaygroundTestCase(
 					["dev-libs/A"],
+					options = { "--autounmask": 'n' },
 					success = False),
 				ResolverPlaygroundTestCase(
 					["dev-libs/B"],
@@ -183,6 +184,7 @@ class ConfigTestCase(TestCase):
 					mergelist = ["dev-libs/D-1"]),
 				ResolverPlaygroundTestCase(
 					["dev-libs/E"],
+					options = { "--autounmask": 'n' },
 					success = False),
 		)
 

diff --git a/pym/portage/tests/resolver/test_autounmask.py b/pym/portage/tests/resolver/test_autounmask.py
index 0689a37..6ffdaaf 100644
--- a/pym/portage/tests/resolver/test_autounmask.py
+++ b/pym/portage/tests/resolver/test_autounmask.py
@@ -281,7 +281,7 @@ class AutounmaskTestCase(TestCase):
 		test_cases = (
 				ResolverPlaygroundTestCase(
 					["=dev-libs/A-1"],
-					options = {"--autounmask": False},
+					options = {"--autounmask": 'n'},
 					success = False),
 				ResolverPlaygroundTestCase(
 					["=dev-libs/A-1"],

diff --git a/pym/portage/tests/resolver/test_circular_dependencies.py b/pym/portage/tests/resolver/test_circular_dependencies.py
index 5132659..204f3c9 100644
--- a/pym/portage/tests/resolver/test_circular_dependencies.py
+++ b/pym/portage/tests/resolver/test_circular_dependencies.py
@@ -60,7 +60,6 @@ class CircularDependencyTestCase(TestCase):
 			#Conflict with autounmask
 			ResolverPlaygroundTestCase(
 				["=dev-libs/W-3"],
-				options = { "--autounmask": True },
 				circular_dependency_solutions = { "dev-libs/Y-1": frozenset([frozenset([("foo", False)])])},
 				use_changes = { "dev-libs/Z-3": {"bar": True}},
 				success = False),

diff --git a/pym/portage/tests/resolver/test_multirepo.py b/pym/portage/tests/resolver/test_multirepo.py
index a4b905e..34c6d45 100644
--- a/pym/portage/tests/resolver/test_multirepo.py
+++ b/pym/portage/tests/resolver/test_multirepo.py
@@ -248,6 +248,7 @@ class MultirepoTestCase(TestCase):
 				mergelist = ["dev-libs/A-2::repo1", "dev-libs/B-2"]),
 			ResolverPlaygroundTestCase(
 				["=dev-libs/B-3"],
+				options = { "--autounmask": 'n' },
 				success = False,
 				check_repo_names = True),
 
@@ -297,11 +298,13 @@ class MultirepoTestCase(TestCase):
 				mergelist = ["dev-libs/G-1"]),
 			ResolverPlaygroundTestCase(
 				["dev-libs/H"],
+				options = { "--autounmask": 'n' },
 				success = False),
 
 			#package.mask with wildcards
 			ResolverPlaygroundTestCase(
 				["dev-libs/Z"],
+				options = { "--autounmask": 'n' },
 				success = False),
 			)
 

diff --git a/pym/portage/tests/resolver/test_old_dep_chain_display.py b/pym/portage/tests/resolver/test_old_dep_chain_display.py
index 57f1ff1..8aedf59 100644
--- a/pym/portage/tests/resolver/test_old_dep_chain_display.py
+++ b/pym/portage/tests/resolver/test_old_dep_chain_display.py
@@ -18,9 +18,11 @@ class OldDepChainDisplayTestCase(TestCase):
 		test_cases = (
 			ResolverPlaygroundTestCase(
 				["=dev-libs/A-1"],
+				options = { "--autounmask": 'n' },
 				success = False),
 			ResolverPlaygroundTestCase(
 				["=dev-libs/A-2"],
+				options = { "--autounmask": 'n' },
 				success = False),
 			)
 

diff --git a/pym/portage/tests/resolver/test_simple.py b/pym/portage/tests/resolver/test_simple.py
index b8d403d..0bcfc4b 100644
--- a/pym/portage/tests/resolver/test_simple.py
+++ b/pym/portage/tests/resolver/test_simple.py
@@ -24,7 +24,7 @@ class SimpleResolverTestCase(TestCase):
 
 		test_cases = (
 			ResolverPlaygroundTestCase(["dev-libs/A"], success = True, mergelist = ["dev-libs/A-1"]),
-			ResolverPlaygroundTestCase(["=dev-libs/A-2"], success = False),
+			ResolverPlaygroundTestCase(["=dev-libs/A-2"], options = { "--autounmask": 'n' }, success = False),
 
 			ResolverPlaygroundTestCase(
 				["dev-libs/A"],

diff --git a/pym/portage/tests/resolver/test_slot_collisions.py b/pym/portage/tests/resolver/test_slot_collisions.py
index 1f428ef..5839a39 100644
--- a/pym/portage/tests/resolver/test_slot_collisions.py
+++ b/pym/portage/tests/resolver/test_slot_collisions.py
@@ -56,6 +56,7 @@ class SlotCollisionTestCase(TestCase):
 			#A qt-*[qt3support] like mess.
 			ResolverPlaygroundTestCase(
 				["dev-libs/A", "dev-libs/B", "dev-libs/C", "dev-libs/D"],
+				options = { "--autounmask": 'n' },
 				success = False,
 				mergelist = ["dev-libs/A-1", "dev-libs/B-1", "dev-libs/C-1", "dev-libs/D-1"],
 				ignore_mergelist_order = True,
@@ -98,7 +99,7 @@ class SlotCollisionTestCase(TestCase):
 			ResolverPlaygroundTestCase(
 				["sci-libs/P", "sci-libs/Q:2"],
 				success = False,
-				options = { "--update": True, "--complete-graph": True },
+				options = { "--update": True, "--complete-graph": True, "--autounmask": 'n' },
 				mergelist = ["sci-libs/P-1", "sci-libs/Q-1"],
 				ignore_mergelist_order = True,
 				all_permutations=True,



                 reply	other threads:[~2011-05-16 20:12 UTC|newest]

Thread overview: [no followups] expand[flat|nested]  mbox.gz  Atom feed

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=bd486e676cf4fb1893f8d06220c1f60ed04760f2.zmedico@gentoo \
    --to=zmedico@gentoo.org \
    --cc=gentoo-commits@lists.gentoo.org \
    --cc=gentoo-dev@lists.gentoo.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox