Re: [lxc-devel] [PATCH] Use LXCPATH and LOCALSTATEDIR instead of hardcoded /var

2012-12-05 Thread Dwight Engen
On Fri, 30 Nov 2012 17:09:28 -0500 Stéphane Graber wrote: > On 11/30/2012 09:50 AM, Dwight Engen wrote: > > Signed-off-by: Dwight Engen > > It looks like you're mixing LOCALSTATEDIR "/lib/lxc" and LXCPATH > quite a bit. Shouldn't we use LXCPATH whenever possible? Sorry, just now noticed your c

Re: [lxc-devel] [PATCH] Use LXCPATH and LOCALSTATEDIR instead of hardcoded /var

2012-11-30 Thread Stéphane Graber
On 11/30/2012 09:50 AM, Dwight Engen wrote: > Signed-off-by: Dwight Engen It looks like you're mixing LOCALSTATEDIR "/lib/lxc" and LXCPATH quite a bit. Shouldn't we use LXCPATH whenever possible? > --- > src/lxc/Makefile.am | 3 ++- > src/lxc/conf.c| 2 +- > src/lxc/

Re: [lxc-devel] [PATCH] Use LXCPATH and LOCALSTATEDIR instead of hardcoded /var

2012-11-30 Thread Serge Hallyn
Quoting Dwight Engen (dwight.en...@oracle.com): > Signed-off-by: Dwight Engen Acked-by: Serge E. Hallyn > --- > src/lxc/Makefile.am | 3 ++- > src/lxc/conf.c| 2 +- > src/lxc/lxccontainer.c| 2 +- > src/tests/Makefile.am | 3 ++- > src/tests/contai

[lxc-devel] [PATCH] Use LXCPATH and LOCALSTATEDIR instead of hardcoded /var

2012-11-30 Thread Dwight Engen
Signed-off-by: Dwight Engen --- src/lxc/Makefile.am | 3 ++- src/lxc/conf.c| 2 +- src/lxc/lxccontainer.c| 2 +- src/tests/Makefile.am | 3 ++- src/tests/containertests.c| 2 +- src/tests/saveconfig.c| 2 +- src/tests/startone.c