Achim Gratz <strom...@nexgo.de>:
> Gary E. Miller writes:
> > As I said yesterday, I gave up on the top dopwn approach. I have
> > gone to bottom up. Fixing small things. It will prolly take me a week
> > to deal with all the really obvious -Wsign-conversion warnings.
> >
> > I am trying to make all my commits very bytesize, so each is
> > trivial to vet. Feel free to keep an eye on me. :-)
>
> What became of the idea of you doing that in a branch? It would really
> help to fix the constant stream of typos and "whoops" before they hit
> the master branch.
Achim, the house practice here is to develop with small commits on master
rather than branches unless the change is very large and very intrusive.
Yes, this means we have some bobbles and missteps in our trunk that
could have been avoided with a branch-merge approach (and some of
those missteps are mine). That's a tradeoff I cheerfully accept in
order to have everyone's attention constantly focused on the state of
the actual working code. I also think this practice encourages small
commits and staying humble.
That said, your criticism of this particular patch sequence is quite valid
and I will address it in a separate reply.
--
<a href="http://www.catb.org/~esr/";>Eric S. Raymond</a>
Please consider contributing to my Patreon page at https://www.patreon.com/esr
so I can keep the invisible wheels of the Internet turning. Give generously -
the civilization you save might be your own.
_______________________________________________
devel mailing list
devel@ntpsec.org
http://lists.ntpsec.org/mailman/listinfo/devel
