Hi Serge, On Mon, Mar 18, 2013 at 10:59 PM, Serge Hallyn <serge.hal...@ubuntu.com> wrote: > Quoting Ryota Ozaki (ozaki.ry...@gmail.com): >> When we install lxc by manual (configure; make; make install), >> all files are installed under /usr/local/. Configuration files >> and setting files of containers are stored under /usr/local/ too, >> however, only log files are stored under /var/log/ not >> /usr/local/var/log. >> >> This patch changes the default log path to $localstatedir/log/lxc >> (by default $localstatedir is /usr/local/var) where is an ordinary >> directory, which is probably expected and unsurprising. >> >> Signed-off-by: Ryota Ozaki <ozaki.ry...@gmail.com> > > Is there any distro which does not use /var/log as the standard > log location?
Oh I didn't intend to change the log path for distro packages. Actually when we do "make rpm", the log path will be /var/log/lxc since $localstatedir is set to /var. What I want to change with the patch is a behavior when we run configure without any arguments, which is a procedure that I try to use latest lxc in git for test. > > I don't think treating log files the same way as binaries and > libraries is the right thing here. I got surprised that log files are stored under /var/log because lxc setting files are stored under /usr/local/var/lib (and all other stuffs are also stored under /usr/local). One problem that we use a common log path (/var/log) is that log files for packaged binaries and self-built binaries are mixed. I want to separate them each other. Best regards, ozaki-r > >> --- >> Note that this patch is based on staging of git://github.com/lxc/lxc.git. >> >> configure.ac | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/configure.ac b/configure.ac >> index 0c94d37..62ddb16 100644 >> --- a/configure.ac >> +++ b/configure.ac >> @@ -194,7 +194,7 @@ AM_CONDITIONAL([USE_CONFIGPATH_LOGS], [test >> "$use_configpath_logs" = "yes"]) >> if test "$use_configpath_logs" = "yes"; then >> default_log_path="${with_config_path}" >> else >> - default_log_path="/var/log/lxc" >> + default_log_path="${localstatedir}/log/lxc" >> fi >> >> AC_ARG_WITH([log-path], >> -- >> 1.8.1.4 >> >> >> ------------------------------------------------------------------------------ >> 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_d2d_mar >> _______________________________________________ >> 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_d2d_mar _______________________________________________ Lxc-devel mailing list Lxc-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/lxc-devel