[lxc-devel] [PATCH 1/2] Revert "utils: reimplement/fix mkdir_p()"

[Stéphane Graber]

On 04/18/2013 07:14 PM, richard -rw- weinberger wrote:
> On Thu, Apr 18, 2013 at 10:48 AM, Stéphane Graber <stgraber at ubuntu.com> wrote:
>> On 04/18/2013 10:34 AM, Stéphane Graber wrote:
>>> On 04/18/2013 10:32 AM, Stéphane Graber wrote:
>>>> This reverts commit 8de4140644f01180f2fdab55b0ab0f13d1c761c6.
>>>>
>>>> This commit was preventing container startup on my machine, making them
>>>> all fail with various "No such file or directory" errors.
>>>>
>>>> Signed-off-by: Stéphane Graber <stgraber at ubuntu.com>
>>>
>>> Was directly applied to the staging branch as the issue was preventing
>>> regular use of LXC.
>>
>> Hmm, with the recent round of moderation on the list, it looks like I
>> may have applied the wrong version of the patch.
>>
>> There's two identically named patch on the ML but with different content.
>> I'll test the second one and see if that solves my issues.
>>
>> In the future, please include "v2" in the name of the second patch so
>> that people trying to figure out what to apply to staging don't get
>> confused ;)
> 
> 1. Why have you even applied the first version of my patch?

Because I saw an Ack for a patch with a title that exactly matched one I
had in my history.

> It looks like you applied while it was under review.

No, I applied it after Serge's ack.

> 2. Where did Serge's Ack came from? He did never Ack my first patch.

He only acked the second patch which made it extremely late to the
mailing-list (Serge got it early as he was directly CCed on it).

> 3. If you encounter problems with one of my patches the lest thing you
> can do is CC'ing me.

We don't have any real policy that I know of for this mailing-list so I
go with the one that's the least annoying for everyone involved which is
to always send to the mailing-list and to nobody else (unless someone is
clearly not subscribed to it). That way everyone receives the e-mail at
the same time.

Keep in mind that CCing people is what caused that mess to start with
(that and you not marking the second submit as "v2" which would have
made it trivial to detect).


> --
> Thanks,
> //richard
> 


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

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 901 bytes
Desc: OpenPGP digital signature
URL: <http://lists.linuxcontainers.org/pipermail/lxc-devel/attachments/20130418/61a61691/attachment.pgp>