dak | 9 Aug 2012 20:32
Picon
Picon

Brain surgery on the build system, first stage (issue 6446096)


http://codereview.appspot.com/6446096/diff/2001/aclocal.m4
File aclocal.m4 (right):

http://codereview.appspot.com/6446096/diff/2001/aclocal.m4#newcode1350
aclocal.m4:1350: +    if test "$CYGWIN$MINGW32" = "nono"; then
This does not look like aclocal.m4 at all, more like the output of git
log -p.  What is up with this?

http://codereview.appspot.com/6446096/
john.mandereau | 9 Aug 2012 23:06
Picon

Re: Brain surgery on the build system, first stage (issue 6446096)

Reviewers: dak,

http://codereview.appspot.com/6446096/diff/2001/aclocal.m4
File aclocal.m4 (right):

http://codereview.appspot.com/6446096/diff/2001/aclocal.m4#newcode1350
aclocal.m4:1350: +    if test "$CYGWIN$MINGW32" = "nono"; then
On 2012/08/09 18:32:28, dak wrote:
> This does not look like aclocal.m4 at all, more like the output of git
log -p.
> What is up with this?

This looks like a bug in git-cl and/or side-by-side view or Rietveld.
Unified diff view is correct, though.

Description:
Brain surgery on the build system, first stage

Sorry to do this just before 2.16 branching out, but if you remember
nightmares about non-distributed files and the like because of the
desire to include last-minute changes to translations and whatnot, IMO
the part of this patch regarding "make dist" is really worth the
trouble.  If you like, I can also include a merge of stepmake/stepmake
and make/, but I thought this would really clutter the review of other
changes, and this merge is trivial if you compare with the patch of
this issue, so that merge will be submitted for review after this
review issue settled down.

Commits are viewable separately on dev/jmandereau:
http://git.savannah.gnu.org/gitweb/?p=lilypond.git;a=shortlog;h=refs/heads/dev/jmandereau
(Continue reading)


Gmane