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