Re: [lxc-devel] [PATCH] fix "make rpm"

2012-10-19 Thread Serge Hallyn
Quoting Dwight Engen (dwight.en...@oracle.com):
> 
> RPM doesn't like "-" in the version number and gives:
> "error: line 24: Illegal char '-' in: Version: 0.8.0-rc2"
> Other packages (bind-utils for example) have used . instead
> of - as a seperator.
> 
> Signed-off-by: Dwight Engen 

Acked-by: Serge E. Hallyn 

(and pushed to github)

> ---
>  configure.ac |2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/configure.ac b/configure.ac
> index e7356eb..403d549 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -1,7 +1,7 @@
>  #   -*- Autoconf -*-
>  # Process this file with autoconf to produce a configure script.
>  
> -AC_INIT([lxc], [0.8.0-rc2])
> +AC_INIT([lxc], [0.8.0.rc2])
>  
>  AC_CONFIG_SRCDIR([configure.ac])
>  AC_CONFIG_AUX_DIR([config])
> -- 
> 1.7.1
> 
> 
> --
> Don't let slow site performance ruin your business. Deploy New Relic APM
> Deploy New Relic app performance management and know exactly
> what is happening inside your Ruby, Python, PHP, Java, and .NET app
> Try New Relic at no cost today and get our sweet Data Nerd shirt too!
> http://p.sf.net/sfu/newrelic-dev2dev
> ___
> Lxc-devel mailing list
> Lxc-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/lxc-devel

--
Everyone hates slow websites. So do we.
Make your web apps faster with AppDynamics
Download AppDynamics Lite for free today:
http://p.sf.net/sfu/appdyn_sfd2d_oct
___
Lxc-devel mailing list
Lxc-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/lxc-devel


Re: [lxc-devel] [PATCH 5/5 v2] Add distro config file /etc/lxc/lxc.conf

2012-10-19 Thread Serge Hallyn
Quoting Dwight Engen (dwight.en...@oracle.com):
> On Thu, 18 Oct 2012 10:34:16 -0500
> Serge Hallyn  wrote:
> 
> > Quoting Dwight Engen (dwight.en...@oracle.com):
> > > This allows a distro to put the distro specific default network
> > > configuration (for example bridge device, link type), or other lxc
> > > configuration in the case that -f is not passed by the user to
> > > lxc-create, in which case lxc-create will use the distro conf file
> > > as the basis for the containers config.
> > > 
> > > Signed-off-by: Dwight Engen 
> > 
> > Thanks much for doing this, Dwight.  I'm pushing this to my github
> > tree.  I do have one question below (which, if I'm not wrong, would
> > only result in a 1-line patch).
> > 
> > Acked-by: Serge E. Hallyn 
> 
> See below for patch. BTW- Are you okay with this being in the config
> directory with the autoconf machinery also? I think its kind of icky,

Eh, I'm fine with it.

> and I can move the autoconf stuff to build-aux if you want.

If it leaves you with less of a bad taste when you look at it then
that's fine, but either way.

I've pulled your new patch, and Stephane has pushed them to
github.com/lxc/lxc.git - thanks!

-serge

--
Everyone hates slow websites. So do we.
Make your web apps faster with AppDynamics
Download AppDynamics Lite for free today:
http://p.sf.net/sfu/appdyn_sfd2d_oct
___
Lxc-devel mailing list
Lxc-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/lxc-devel