On Sat,  1 Mar 2014 at 19:13:43 +0100, Christophe wrote:
> 
> > 
> > This serie of patchs is implementing the changes that I have proposed
> > to David's patch:
> > 
> >  - patch 1 / for review: contains the changes against the patch, for
> > easier review;

For some reason this patch did not arrive at the mailing list together with
the others.

> >  - patch 1 / merged: proposes the same changes merged into the original
> > patch, to replace the commit 320430efe5642db07c82d75a6e6311065233d622 to
> > keep a simpler history (again, will be handy at changelog writing time);
> 
> 
> Commit id was meant to be: 4a17aa58626ba27d56fe5a5cbdff3b415c2464b3

This is very kind of you, to propose your fixes to be merged into his
patch.

I was hoping that David would address your review comments in a new
version of his patch, but now we don't need to wait any longer :-)

> >  - patch 2: fixes a small thing seen while making the patch
> > 
> >  - patch 3: more fixes I missed during first review. Could be amended to
> > patch 1 without regret

Thanks a lot Christophe! I'm sure all wmaker users appreciate your
contributions.


-- 
To unsubscribe, send mail to [email protected].

Reply via email to