- Misha; I told him tests were ok to submit during a code freeze; this is true, given that Alexandre accepted tests as last as last Friday. I should have told him that tests which add stubs probably aren't ok, but he learned that as soon as he submitted. http://www.winehq.org/pipermail/wine-devel/2010-June/084632.html so that seems fine.
From the look of things Misha will be ok, but I imagine he would have been anyway.
But there are also cases like e.g. http://bugs.winehq.org/show_bug.cgi?id=21233#c8 where you completely miss the real issues in the patch, and instead recommend referring to a bug number in the commit log,
Probably not my place to wade in here, but that's not true, he didn't mention the commit log: "Be sure to mention in the post that it fixes http://bugs.winehq.org/show_bug.cgi?id=21233" <http://bugs.winehq.org/show_bug.cgi?id=21233> I'm pretty sure he meant in the email when he emails wine-patches rather than in the commit log. I've got to say Henri, I think you are being a little unfair here. Yes, possibly Dan has given flawed advice in the past, but it's nothing that a private email or chat on IRC wouldn't have solved. Launching into a tirade against him publically isn't helping anything, all these instances are Dan trying to help out, quite obviously not to be "insidious" or to harm the project. Luke.