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 |