Removing the original params.c parser

Andrew Bartlett abartlet at samba.org
Fri Aug 15 14:27:48 MDT 2014


On Fri, 2014-08-15 at 20:34 +0200, Michael Adam wrote:
> On 2014-08-15 at 18:24 +0200, Volker Lendecke wrote:
> > On Fri, Aug 15, 2014 at 09:22:22AM -0700, Jeremy Allison wrote:
> > > On Fri, Aug 15, 2014 at 06:21:14PM +0200, Volker Lendecke wrote:
> > > > On Fri, Aug 15, 2014 at 11:12:25AM -0500, Christopher R. Hertel wrote:
> > > > > To be honest, I haven't actually reviewed the code itself, just the
> > > > > discussion of the discussion of the patch.  :) :) :)
> > > > > 
> > > > > "Chaos, panic, and disorder...my work here is done."  No further
> > > > > interference from my corner.  Go for it, Jeremy.
> > > > 
> > > > Let's wait for Andrew to re-read this discussion before we
> > > > do ANYTHING. Then we can decide which version of the code is
> > > > better and we can carefully carve the words that put the
> > > > R.I.P. on either version.
> > > 
> > > Too late, already pushed it with a mildly updated
> > > commit message of : "lib: Remove parser code, now replaced with tini parser."
> > 
> > Just push current master over your tree. This will reliably
> > kill the current autobuild.
> 
> Could we please just stop this silly discussion!?
> 
> The patch that you originally proposed is now on its way
> to master. The concerns that Andrew hat raised in his mail
> have been cleared by Chris himself. And Andrew didn't give a
> formal NACK, so Jeremy pushing would have been fine anyways!
> So I see no point in still complaining or asking for reverts
> or further delays. It's wasted energy.

Thanks all!

I'm glad to see this discussed properly, and that we now have a clearer
commit message. 

Andrew Bartlett

-- 
Andrew Bartlett                       http://samba.org/~abartlet/
Authentication Developer, Samba Team  http://samba.org
Samba Developer, Catalyst IT          http://catalyst.net.nz/services/samba




More information about the samba-technical mailing list