On Fri, 1 Jul 2016 22:40:47 -0700
Zac Medico <zmed...@gentoo.org> wrote:

> On 07/01/2016 03:46 PM, Zac Medico wrote:
> > On 07/01/2016 12:37 AM, Zac Medico wrote:  
> >> @@ -327,6 +341,11 @@ def action_build(settings, trees, mtimedb,
> >>                    display_missing_pkg_set(root_config,
> >> e.value) return 1
> >>  
> >> +          if success and mydepgraph.need_config_reload():
> >> +
> >> load_emerge_config(emerge_config=emerge_config)
> >> +                  adjust_configs(emerge_config.opts,
> >> emerge_config.trees)
> >> +                  settings, trees, mtimedb = emerge_config
> >> +  
> > 
> > The Package instances inside the depgraph are still going to
> > reference the old RootConfig instances, so I'll update the patch to
> > solve that somehow.  
> 
> Fixed in v2 by making load_emerge_config update existing RootConfig
> instances in-place.

looks fine to me
-- 
Brian Dolbec <dolsen>


Reply via email to