On Fri, 2010-08-06 at 10:04 +0100, Matthew Booth wrote: > On 05/08/10 21:58, [email protected] wrote: > > From: David Lutterkort<[email protected]> > > > > The problem was that when we replaced the old tree for a file with the new > > one we just read from the file, the replace operation did not update > > nodesets in variables, since we did not pass the global symtab into > > tree_replace. > > > > * src/internal.h (tree_replace): pass in struct augeas, so that we have > > access to the global symtab > > * src/augeas.c (tree_replace): make sure we create the pathx with the > > global symtab > > * src/transform.c (load_file): adapt to new prototype for tree_replace > > * tests/test-load.c (testReloadExternalMod): test that we do not segfault > > when a file was modified externally > > --- > > src/augeas.c | 6 +++--- > > src/internal.h | 2 +- > > src/transform.c | 2 +- > > tests/test-load.c | 52 > > ++++++++++++++++++++++++++++++++++++++++++++++++++-- > > 4 files changed, 55 insertions(+), 7 deletions(-) > > > > I've tested this patch and it fixes the V2V crasher. Thanks, David.
Cool. Pushed the patch series. David _______________________________________________ augeas-devel mailing list [email protected] https://www.redhat.com/mailman/listinfo/augeas-devel
