Quoting Frederic Crozat (fcro...@suse.com):
> Le mercredi 24 avril 2013 à 08:55 -0500, Serge Hallyn a écrit :
> > Quoting Frederic Crozat (fcro...@suse.com):
> > > Le mardi 23 avril 2013 à 10:09 -0500, Serge Hallyn a écrit :
> > > > Quoting Frederic Crozat (fcro...@suse.com):
> > > 
> > > > Thanks - clearly my patch needed some love :)  Just a few comments 
> > > > below.
> > > 
> > > hehe :)
> > > > 
> > > > > +
> > > > > +     /* check if directory exists, and create it otherwise */
> > > > > +     if (access(path, F_OK)) {
> > > > > +             ret = mkdir(path, 0755);
> > > > > +             if (ret == -1 && errno == EEXIST) {
> > > > 
> > > > Did you mean errno != EEXIST?
> > > 
> > > I blindly copied code from lxc_one_cgroup_create (now
> > > create_lxcgroups :)
> > 
> > That has:
> > 
> >                         if (ret == -1 && errno != EEXIST) {
> 
> My copy didn't, because I didn't had
> d7436fc1dfe0323fb5e6905457f342da5ed8ca97 backported..
> 
> I'll fix that..

Thanks, ack with that fixed.

------------------------------------------------------------------------------
Try New Relic Now & We'll Send You this Cool Shirt
New Relic is the only SaaS-based application performance monitoring service 
that delivers powerful full stack analytics. Optimize and monitor your
browser, app, & servers with just a few lines of code. Try New Relic
and get this awesome Nerd Life shirt! http://p.sf.net/sfu/newrelic_d2d_apr
_______________________________________________
Lxc-users mailing list
Lxc-users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/lxc-users

Reply via email to