| 1 |
On Fri, 1 Jul 2016 22:40:47 -0700 |
| 2 |
Zac Medico <zmedico@g.o> wrote: |
| 3 |
|
| 4 |
> On 07/01/2016 03:46 PM, Zac Medico wrote: |
| 5 |
> > On 07/01/2016 12:37 AM, Zac Medico wrote: |
| 6 |
> >> @@ -327,6 +341,11 @@ def action_build(settings, trees, mtimedb, |
| 7 |
> >> display_missing_pkg_set(root_config, |
| 8 |
> >> e.value) return 1 |
| 9 |
> >> |
| 10 |
> >> + if success and mydepgraph.need_config_reload(): |
| 11 |
> >> + |
| 12 |
> >> load_emerge_config(emerge_config=emerge_config) |
| 13 |
> >> + adjust_configs(emerge_config.opts, |
| 14 |
> >> emerge_config.trees) |
| 15 |
> >> + settings, trees, mtimedb = emerge_config |
| 16 |
> >> + |
| 17 |
> > |
| 18 |
> > The Package instances inside the depgraph are still going to |
| 19 |
> > reference the old RootConfig instances, so I'll update the patch to |
| 20 |
> > solve that somehow. |
| 21 |
> |
| 22 |
> Fixed in v2 by making load_emerge_config update existing RootConfig |
| 23 |
> instances in-place. |
| 24 |
|
| 25 |
looks fine to me |
| 26 |
-- |
| 27 |
Brian Dolbec <dolsen> |
Archives