Quoting Stéphane Graber (stgra...@ubuntu.com): > > @@ -134,9 +133,12 @@ int lxc_execute(const char *name, char *const argv[], > > int quiet, > > .argv = argv, > > .quiet = quiet > > }; > > + int ret; > > > > if (lxc_check_inherited(conf, -1)) > > return -1; > > > > - return __lxc_start(name, conf, &execute_start_ops, &args); > > + ret = __lxc_start(name, conf, &execute_start_ops, &args); > > + > > + return ret; > > } > > What's the reason for that bit? Looks to me as functionally identical.
D'oh - good catch. The reason for it was that at first I had the rename_phys_on_shutdown in the lxc_start callers, not in __lxc_start itself. I moved it right at the end, but forgot about this now- unneeded bit. thanks, -serge ------------------------------------------------------------------------------ LogMeIn Rescue: Anywhere, Anytime Remote support for IT. Free Trial Remotely access PCs and mobile devices and provide instant support Improve your efficiency, and focus on delivering more value-add services Discover what IT Professionals Know. Rescue delivers http://p.sf.net/sfu/logmein_12329d2d _______________________________________________ Lxc-devel mailing list Lxc-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/lxc-devel