Re: The direction linux is taking

Dave Jones (davej@suse.de)
Sat, 29 Dec 2001 21:04:07 +0100 (CET)


On Sat, 29 Dec 2001, Larry McVoy wrote:

> One way to quantify this is to ask Linus, Alan, Marcelo, et al, how much
> time they spend merging, i.e., how often do they get patch rejects?
> Regardless of the answer, it will be interesting. If it is a lot,
> then the patchbot idea has marginal usefulness. If it is none at all,
> then that says development is serialized, which means we may be leaving
> a lot of progress on the floor.

Bringing the 2.4 fixes forward to 2.5 has been brought about a couple
of head-scratching moments when I've had what appear to be parallel
development of the same piece of code in both trees, it's not so much the
rejects, but a case of 'has this already been fixed in a different way'
or 'is this necessary and does it make sense in 2.5?'.

> I wouldn't be surprised if the serialized case is the answer, or close
> to it.

For some things I think it's definitly the right approach.
A single patch with a dozen peoples changes to the same piece of code
would be a royal pita if it then turns out 1 of them is has problems.

Likewise, Alans patches always seemed to isolate anything which could
make diagnosing problems into seperate releases, eg no VFS & IDE updates
in the same patch unless blindingly obviously correct.

> Anyway, I'm interested to see if there are screams of "all I ever do is
> merge and I hate it" or "merging? what's that?".

I've only been keeping this tree since the beginning of the month,
so I'm still trying to find my feet a little, but so far merging is
pretty straightforward and usually painless.

The procedure when Linus/Marcelo release a new patch usually goes..

1. edit the patch to remove any bits that don't make sense
(eg, I have newer/better version in my tree)
2. cat ../patch-2.5.x | patch -p1 --dry-run
3. edit the patch to remove already present hunks.
4. manually fix up rejects in my tree, and remove reject hunk
from the diff.
5. back to (1) until no rejects.
6. cat ../patch-2.5.x | patch -p1
7. testing..
8. Create new diff, and give it a quick readthrough.

Out of all this the initial patch review (step #1) and the final
lookover are by far the most time consuming, and I don't think any
automated tool could speed this up and give me the same level of
understanding over what I'm merging.

Dave.

-- 
| Dave Jones.        http://www.codemonkey.org.uk
| SuSE Labs

- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/