This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Re: [PATCH 0/2] AArch64 glibc port
On Mon, 1 Oct 2012, Carlos O'Donell wrote:
> (2) Split up your patches.
>
> One huge patch is impossible to review.
>
> I can see about 25-30 groupings of files that relate and your patch should
> be split up along those lines.
No, absolutely do not split it up artificially. Patch submissions should
be self-contained. The port (the new sysdeps files) should be submitted
as a single patch, to libc-ports only (not libc-alpha), with appropriate
rationale for any design choices it seems appropriate to draw attention
to. One patch is much easier to review, and spot whether anything is
missing in it, than 25-30.
The only things to split up are any libc changes required (generally,
anything outside of ports/sysdeps/.../aarch64/), where each logical change
should be sent in its own self-contained patch submission with its own
self-contained rationale.
(If any bits of the port are *not needed at all* for a functional port -
if they are purely optimized functions - they can be omitted from the
submission and dealt with later afer the port is in. But still don't send
multiple ports patches in the initial submission.)
--
Joseph S. Myers
joseph@codesourcery.com