<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.0 TRANSITIONAL//EN"> <HTML> <HEAD> <META HTTP-EQUIV="Content-Type" CONTENT="text/html; CHARSET=UTF-8"> <META NAME="GENERATOR" CONTENT="GtkHTML/3.32.2"> </HEAD> <BODY> On Tue, 2011-06-14 at 13:51 +1000, Peter Hutterer wrote: <BLOCKQUOTE TYPE=CITE> <PRE> On Mon, Jun 13, 2011 at 07:43:26PM -0400, Gaetan Nadon wrote: > Aside from one packaging error, mostly adding confusion and causing extra work. tbh, I'd rather prefer reverting the changes and then implementing your changes on top. In your patch 4/5 we'd restore the behaviour to before 39afe69ad7d2 but it's not obvious in the history. A "Revert blah bla" is more visible. The other patches can then be pushed on top. </PRE> </BLOCKQUOTE> I think that's the better approach as well. It would also highlight and document<BR> the desirable changes. <BLOCKQUOTE TYPE=CITE> <PRE> Would still like to hear back from Diego since he may have some extra arguments pro/con the whole thing. Right now, I'm quite willing to sacrifice build-time for maintainer time though. </PRE> </BLOCKQUOTE> <PRE> I would too. There is some guess work on my part due to changes not being explained in the commit text. I am sure the bottom line will be on the plus side. </PRE> <BLOCKQUOTE TYPE=CITE> <PRE> Cheers, Peter > > Gaetan Nadon (5): > Use appropriate Autoconf statements to check for prerequisites > Do not install ChangeLog and README with documentation > man: there is no need to create the "man" source directory > man: restore the standard man pages makefile > Regroup the distribution makefile rules together > > Makefile.am | 42 ++++++++---------------------------------- > configure.ac | 6 ++++-- > man/Makefile.am | 47 +++++++++++++++++++++++++++++++++++++++++++++++ > 3 files changed, 59 insertions(+), 36 deletions(-) > create mode 100644 man/Makefile.am > > -- > 1.7.4.1 > > _______________________________________________ > <A HREF="mailto:xorg-devel@lists.x.org">xorg-devel@lists.x.org</A>: X.Org development > Archives: <A HREF="http://lists.x.org/archives/xorg-devel">http://lists.x.org/archives/xorg-devel</A> > Info: <A HREF="http://lists.x.org/mailman/listinfo/xorg-devel">http://lists.x.org/mailman/listinfo/xorg-devel</A> > </PRE> </BLOCKQUOTE> <BR> </BODY> </HTML>