Gentoo Archives: gentoo-portage-dev

From: Zac Medico <zmedico@g.o>
To: gentoo-portage-dev@l.g.o
Subject: Re: [gentoo-portage-dev] Re: [PATCH] Add emerge --autounmask-continue option (bug 582624)
Date: Mon, 04 Jul 2016 18:02:59
Message-Id: 577AA4CF.4000206@gentoo.org
In Reply to: Re: [gentoo-portage-dev] Re: [PATCH] Add emerge --autounmask-continue option (bug 582624) by Brian Dolbec
1 On 07/02/2016 12:31 PM, Brian Dolbec wrote:
2 > On Fri, 1 Jul 2016 22:40:47 -0700
3 > Zac Medico <zmedico@g.o> wrote:
4 >
5 >> On 07/01/2016 03:46 PM, Zac Medico wrote:
6 >>> On 07/01/2016 12:37 AM, Zac Medico wrote:
7 >>>> @@ -327,6 +341,11 @@ def action_build(settings, trees, mtimedb,
8 >>>> display_missing_pkg_set(root_config,
9 >>>> e.value) return 1
10 >>>>
11 >>>> + if success and mydepgraph.need_config_reload():
12 >>>> +
13 >>>> load_emerge_config(emerge_config=emerge_config)
14 >>>> + adjust_configs(emerge_config.opts,
15 >>>> emerge_config.trees)
16 >>>> + settings, trees, mtimedb = emerge_config
17 >>>> +
18 >>>
19 >>> The Package instances inside the depgraph are still going to
20 >>> reference the old RootConfig instances, so I'll update the patch to
21 >>> solve that somehow.
22 >>
23 >> Fixed in v2 by making load_emerge_config update existing RootConfig
24 >> instances in-place.
25 >
26 > looks fine to me
27
28 Thanks, pushed:
29
30 https://gitweb.gentoo.org/proj/portage.git/commit/?id=e2d88ef3ff5958571ef2da5f69e8756522390f0f
31 --
32 Thanks,
33 Zac