On 05/31/2013 11:35 AM, Serge Hallyn wrote:
> And use it in place of the various ways we were deducing 
> /etc/lxc/default.conf.
> 
> Signed-off-by: Serge Hallyn <serge.hal...@ubuntu.com>

Acked-by: Stéphane Graber <stgra...@ubuntu.com>

> ---
>  configure.ac               | 1 +
>  src/lxc/Makefile.am        | 3 ++-
>  src/lxc/lxc_create.c       | 4 +---
>  src/lxc/lxccontainer.c     | 3 ---
>  src/tests/Makefile.am      | 3 ++-
>  src/tests/containertests.c | 2 +-
>  src/tests/destroytest.c    | 2 +-
>  src/tests/saveconfig.c     | 2 +-
>  src/tests/startone.c       | 2 +-
>  9 files changed, 10 insertions(+), 12 deletions(-)
> 
> diff --git a/configure.ac b/configure.ac
> index 65751ce..4567001 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -218,6 +218,7 @@ AS_AC_EXPAND(BINDIR, "$bindir")
>  AS_AC_EXPAND(LIBEXECDIR, "$libexecdir")
>  AS_AC_EXPAND(INCLUDEDIR, "$includedir")
>  AS_AC_EXPAND(SYSCONFDIR, "$sysconfdir")
> +AS_AC_EXPAND(LXC_DEFAULT_CONFIG, "$sysconfdir/lxc/default.conf")
>  AS_AC_EXPAND(DATADIR, "$datadir")
>  AS_AC_EXPAND(LOCALSTATEDIR, "$localstatedir")
>  AS_AC_EXPAND(DOCDIR, "$docdir")
> diff --git a/src/lxc/Makefile.am b/src/lxc/Makefile.am
> index 70fcd25..18469a1 100644
> --- a/src/lxc/Makefile.am
> +++ b/src/lxc/Makefile.am
> @@ -94,7 +94,8 @@ AM_CFLAGS=-I$(top_srcdir)/src \
>       -DLXC_GLOBAL_CONF=\"$(LXC_GLOBAL_CONF)\" \
>       -DLXCINITDIR=\"$(LXCINITDIR)\" \
>       -DLXCTEMPLATEDIR=\"$(LXCTEMPLATEDIR)\" \
> -     -DLOGPATH=\"$(LOGPATH)\"
> +     -DLOGPATH=\"$(LOGPATH)\" \
> +     -DLXC_DEFAULT_CONFIG=\"$(LXC_DEFAULT_CONFIG)\"
>  
>  if ENABLE_APPARMOR
>  AM_CFLAGS += -DHAVE_APPARMOR
> diff --git a/src/lxc/lxc_create.c b/src/lxc/lxc_create.c
> index 2613604..9176d1f 100644
> --- a/src/lxc/lxc_create.c
> +++ b/src/lxc/lxc_create.c
> @@ -135,8 +135,6 @@ bool validate_bdev_args(struct lxc_arguments *a)
>       return true;
>  }
>  
> -/* grab this through autoconf from @config-path@ ? */
> -#define DEFAULT_CONFIG "/etc/lxc/default.conf"
>  int main(int argc, char *argv[])
>  {
>       struct lxc_container *c;
> @@ -174,7 +172,7 @@ int main(int argc, char *argv[])
>       if (my_args.configfile)
>               c->load_config(c, my_args.configfile);
>       else
> -             c->load_config(c, DEFAULT_CONFIG);
> +             c->load_config(c, LXC_DEFAULT_CONFIG);
>  
>       if (strcmp(my_args.bdevtype, "zfs") == 0) {
>               if (my_args.zfsroot)
> diff --git a/src/lxc/lxccontainer.c b/src/lxc/lxccontainer.c
> index 24b6008..61e1327 100644
> --- a/src/lxc/lxccontainer.c
> +++ b/src/lxc/lxccontainer.c
> @@ -1158,9 +1158,6 @@ static int lxcapi_get_keys(struct lxc_container *c, 
> const char *key, char *retv,
>       return ret;
>  }
>  
> -
> -/* default config file - should probably come through autoconf */
> -#define LXC_DEFAULT_CONFIG "/etc/lxc/default.conf"
>  static bool lxcapi_save_config(struct lxc_container *c, const char *alt_file)
>  {
>       FILE *fout;
> diff --git a/src/tests/Makefile.am b/src/tests/Makefile.am
> index ba24d2c..be036e3 100644
> --- a/src/tests/Makefile.am
> +++ b/src/tests/Makefile.am
> @@ -20,7 +20,8 @@ AM_CFLAGS=-I$(top_srcdir)/src \
>       -DLXCROOTFSMOUNT=\"$(LXCROOTFSMOUNT)\" \
>       -DLXCPATH=\"$(LXCPATH)\" \
>       -DLXC_GLOBAL_CONF=\"$(LXC_GLOBAL_CONF)\" \
> -     -DLXCINITDIR=\"$(LXCINITDIR)\"
> +     -DLXCINITDIR=\"$(LXCINITDIR)\" \
> +     -DLXC_DEFAULT_CONFIG=\"$(LXC_DEFAULT_CONFIG)\"
>  
>  bin_PROGRAMS = lxc-test-containertests lxc-test-locktests lxc-test-startone \
>       lxc-test-destroytest lxc-test-saveconfig lxc-test-createtest \
> diff --git a/src/tests/containertests.c b/src/tests/containertests.c
> index 8868faa..8d1658f 100644
> --- a/src/tests/containertests.c
> +++ b/src/tests/containertests.c
> @@ -71,7 +71,7 @@ static int create_busybox(void)
>               return -1;
>       }
>       if (pid == 0) {
> -             ret = execlp("lxc-create", "lxc-create", "-t", "busybox", "-f", 
> "/etc/lxc/default.conf", "-n", MYNAME, NULL);
> +             ret = execlp("lxc-create", "lxc-create", "-t", "busybox", "-f", 
> LXC_DEFAULT_CONFIG, "-n", MYNAME, NULL);
>               // Should not return
>               perror("execl");
>               exit(1);
> diff --git a/src/tests/destroytest.c b/src/tests/destroytest.c
> index 0552b4c..428d09e 100644
> --- a/src/tests/destroytest.c
> +++ b/src/tests/destroytest.c
> @@ -38,7 +38,7 @@ static int create_ubuntu(void)
>               return -1;
>       }
>       if (pid == 0) {
> -             ret = execlp("lxc-create", "lxc-create", "-t", "ubuntu", "-f", 
> "/etc/lxc/default.conf", "-n", MYNAME, NULL);
> +             ret = execlp("lxc-create", "lxc-create", "-t", "ubuntu", "-f", 
> LXC_DEFAULT_CONFIG, "-n", MYNAME, NULL);
>               // Should not return
>               perror("execl");
>               exit(1);
> diff --git a/src/tests/saveconfig.c b/src/tests/saveconfig.c
> index fa84e32..649573f 100644
> --- a/src/tests/saveconfig.c
> +++ b/src/tests/saveconfig.c
> @@ -38,7 +38,7 @@ static int create_ubuntu(void)
>               return -1;
>       }
>       if (pid == 0) {
> -             ret = execlp("lxc-create", "lxc-create", "-t", "ubuntu", "-f", 
> "/etc/lxc/default.conf", "-n", MYNAME, NULL);
> +             ret = execlp("lxc-create", "lxc-create", "-t", "ubuntu", "-f", 
> LXC_DEFAULT_CONFIG, "-n", MYNAME, NULL);
>               // Should not return
>               perror("execl");
>               exit(1);
> diff --git a/src/tests/startone.c b/src/tests/startone.c
> index 591399d..d781e75 100644
> --- a/src/tests/startone.c
> +++ b/src/tests/startone.c
> @@ -70,7 +70,7 @@ static int create_ubuntu(void)
>               return -1;
>       }
>       if (pid == 0) {
> -             ret = execlp("lxc-create", "lxc-create", "-t", "ubuntu", "-f", 
> "/etc/lxc/default.conf", "-n", MYNAME, NULL);
> +             ret = execlp("lxc-create", "lxc-create", "-t", "ubuntu", "-f", 
> LXC_DEFAULT_CONFIG, "-n", MYNAME, NULL);
>               // Should not return
>               perror("execl");
>               exit(1);
> 


-- 
Stéphane Graber
Ubuntu developer
http://www.ubuntu.com

Attachment: signature.asc
Description: OpenPGP digital signature

------------------------------------------------------------------------------
Get 100% visibility into Java/.NET code with AppDynamics Lite
It's a free troubleshooting tool designed for production
Get down to code-level detail for bottlenecks, with <2% overhead.
Download for free and get started troubleshooting in minutes.
http://p.sf.net/sfu/appdyn_d2d_ap2
_______________________________________________
Lxc-devel mailing list
Lxc-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/lxc-devel

Reply via email to