Re: 37-unify-handle-dist.patch

[Tom Tromey]

>>>>> "Akim" == Akim Demaille <address@hidden> writes:

Akim>   * dist.am: Remove, merge into...
Akim>   * distdir.am: here.  Use the neat sed snippet for banners.
Akim>   BTW, distcheck is phony.
Akim>   * automake.in (&handle_dist_worker): Remove, merge into...
Akim>   (&handle_dist): this.
Akim>   (&file_contents): Strip leading new lines.
Akim>   Be sure to end comments with new lines.

Ok.

BTW it is hard to read big rearrangement patches.  It is made worse
because sometimes they include actual code changes as well.  For
instance this patch wasn't just a rearrangement, it included semantic
differences in file_contents.

I'd prefer to see rearrangements separate from semantic changes.  Then
hopefully I can approve rearrangements without trying to read them in
detail.

Tom