[lxc-devel] [PATCH 3/3] Use correct parameter name (default_path instead of lxcpath)

[S.Çağlar Onur]

On Tue, Apr 30, 2013 at 3:06 PM, Serge Hallyn <serge.hallyn at ubuntu.com>wrote:

> Quoting S.Çağlar Onur (caglar at 10ur.org):
> > Signed-off-by: S.Çağlar Onur <caglar at 10ur.org>
>
> Oh sorry, i didn't update it in enough places did I.
>
> Per Dwight's comment we want this to be lxcpath, but that means
> that lxc.functions.in needs to be updated to call 'lxc-config
> default_path'.
> Is there any other place wehre that needs to be changed?


Wait, you confused me :) Right now lxc.functions.in is already calling

lxc_path=`lxc-config default_path`

I guess you meant calling "'lxc-config lxcpath" in lxc.functions.in right?

 > ---
> >  src/lxc/lxc_config.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/src/lxc/lxc_config.c b/src/lxc/lxc_config.c
> > index a454e07..cf19c7b 100644
> > --- a/src/lxc/lxc_config.c
> > +++ b/src/lxc/lxc_config.c
> > @@ -9,7 +9,7 @@ struct lxc_config_items {
> >
> >  struct lxc_config_items items[] =
> >  {
> > -     { .name = "lxcpath", .fn = &lxc_get_default_config_path, },
> > +     { .name = "default_path", .fn = &lxc_get_default_config_path, },
> >       { .name = "lvm_vg", .fn = &lxc_get_default_lvm_vg, },
> >       { .name = "zfsroot", .fn = &lxc_get_default_zfs_root, },
> >       { .name = NULL, },
> > --
> > 1.8.1.2
> >
> >
> >
> ------------------------------------------------------------------------------
> > Introducing AppDynamics Lite, a free troubleshooting tool for Java/.NET
> > Get 100% visibility into your production application - at no cost.
> > Code-level diagnostics for performance bottlenecks with <2% overhead
> > Download for free and get started troubleshooting in minutes.
> > http://p.sf.net/sfu/appdyn_d2d_ap1
> > _______________________________________________
> > Lxc-devel mailing list
> > Lxc-devel at lists.sourceforge.net
> > https://lists.sourceforge.net/lists/listinfo/lxc-devel
>



-- 
S.Çağlar Onur <caglar at 10ur.org>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.linuxcontainers.org/pipermail/lxc-devel/attachments/20130430/8cd6a6e4/attachment.html>