Re: [gentoo-portage-dev] [PATCH] depgraph: clarify "update has been skipped" message (bug 602854)

2016-12-29 Thread Zac Medico
On 12/28/2016 05:25 PM, Brian Dolbec wrote:
> On Wed, 28 Dec 2016 16:01:04 -0800
> Zac Medico  wrote:
> 
>> When an update has been skipped, clearly display both the selected and
>> skipped packages.
>>
>> X-Gentoo-bug: 602854
>> X-Gentoo-bug-url: https://bugs.gentoo.org/602854
>> ---
>>  pym/_emerge/depgraph.py | 9 -
>>  1 file changed, 8 insertions(+), 1 deletion(-)
>>
>> diff --git a/pym/_emerge/depgraph.py b/pym/_emerge/depgraph.py
>> index cb12b05..e298337 100644
>> --- a/pym/_emerge/depgraph.py
>> +++ b/pym/_emerge/depgraph.py
>> @@ -1044,7 +1044,14 @@ class depgraph(object):
>>  writemsg(str(pkg.slot_atom), noiselevel=-1)
>>  if pkg.root_config.settings["ROOT"] != "/":
>>  writemsg(" for %s" % (pkg.root,),
>> noiselevel=-1)
>> -writemsg("\n", noiselevel=-1)
>> +writemsg("\n\n", noiselevel=-1)
>> +
>> +selected_pkg =
>> next(self._dynamic_config._package_tracker.match(
>> +pkg.root, pkg.slot_atom), None)
>> +
>> +writemsg("  selected: %s\n" %
>> (selected_pkg,), noiselevel=-1)
>> +writemsg("  skipped: %s (see unsatisfied
>> dependency below)\n"
>> +% (pkg,), noiselevel=-1)
>>  
>>  for parent, root, atom in parent_atoms:
>>  self._show_unsatisfied_dep(root,
>> atom, myparent=parent)
> 
> 
> looks good :)
> 

Thanks, pushed:

https://gitweb.gentoo.org/proj/portage.git/commit/?id=7dc23bc66b410b5a2f9640dc93ca65f5b2678670
-- 
Thanks,
Zac



Re: [gentoo-portage-dev] [PATCH] depgraph: clarify "update has been skipped" message (bug 602854)

2016-12-28 Thread Brian Dolbec
On Wed, 28 Dec 2016 16:01:04 -0800
Zac Medico  wrote:

> When an update has been skipped, clearly display both the selected and
> skipped packages.
> 
> X-Gentoo-bug: 602854
> X-Gentoo-bug-url: https://bugs.gentoo.org/602854
> ---
>  pym/_emerge/depgraph.py | 9 -
>  1 file changed, 8 insertions(+), 1 deletion(-)
> 
> diff --git a/pym/_emerge/depgraph.py b/pym/_emerge/depgraph.py
> index cb12b05..e298337 100644
> --- a/pym/_emerge/depgraph.py
> +++ b/pym/_emerge/depgraph.py
> @@ -1044,7 +1044,14 @@ class depgraph(object):
>   writemsg(str(pkg.slot_atom), noiselevel=-1)
>   if pkg.root_config.settings["ROOT"] != "/":
>   writemsg(" for %s" % (pkg.root,),
> noiselevel=-1)
> - writemsg("\n", noiselevel=-1)
> + writemsg("\n\n", noiselevel=-1)
> +
> + selected_pkg =
> next(self._dynamic_config._package_tracker.match(
> + pkg.root, pkg.slot_atom), None)
> +
> + writemsg("  selected: %s\n" %
> (selected_pkg,), noiselevel=-1)
> + writemsg("  skipped: %s (see unsatisfied
> dependency below)\n"
> + % (pkg,), noiselevel=-1)
>  
>   for parent, root, atom in parent_atoms:
>   self._show_unsatisfied_dep(root,
> atom, myparent=parent)


looks good :)
-- 
Brian Dolbec 




[gentoo-portage-dev] [PATCH] depgraph: clarify "update has been skipped" message (bug 602854)

2016-12-28 Thread Zac Medico
When an update has been skipped, clearly display both the selected and
skipped packages.

X-Gentoo-bug: 602854
X-Gentoo-bug-url: https://bugs.gentoo.org/602854
---
 pym/_emerge/depgraph.py | 9 -
 1 file changed, 8 insertions(+), 1 deletion(-)

diff --git a/pym/_emerge/depgraph.py b/pym/_emerge/depgraph.py
index cb12b05..e298337 100644
--- a/pym/_emerge/depgraph.py
+++ b/pym/_emerge/depgraph.py
@@ -1044,7 +1044,14 @@ class depgraph(object):
writemsg(str(pkg.slot_atom), noiselevel=-1)
if pkg.root_config.settings["ROOT"] != "/":
writemsg(" for %s" % (pkg.root,), noiselevel=-1)
-   writemsg("\n", noiselevel=-1)
+   writemsg("\n\n", noiselevel=-1)
+
+   selected_pkg = 
next(self._dynamic_config._package_tracker.match(
+   pkg.root, pkg.slot_atom), None)
+
+   writemsg("  selected: %s\n" % (selected_pkg,), 
noiselevel=-1)
+   writemsg("  skipped: %s (see unsatisfied dependency 
below)\n"
+   % (pkg,), noiselevel=-1)
 
for parent, root, atom in parent_atoms:
self._show_unsatisfied_dep(root, atom, 
myparent=parent)
-- 
2.7.4