ruby-changes:46701
From: mame <ko1@a...>
Date: Sat, 20 May 2017 21:23:34 +0900 (JST)
Subject: [ruby-changes:46701] mame:r58816 (trunk): ripper.c should have `#line "ripper.c"`, not `#line "parse.c"`
mame 2017-05-20 21:23:27 +0900 (Sat, 20 May 2017) New Revision: 58816 https://svn.ruby-lang.org/cgi-bin/viewvc.cgi?view=revision&revision=58816 Log: ripper.c should have `#line "ripper.c"`, not `#line "parse.c"` The order of sed commands in Makefile of ripper was wrong: it tries to replace `y.tab.c` with `ripper.c`, but before that, ytab.sed replaced `y.tab.c` with `parse.c`, which led to a wrong result. Modified files: trunk/ext/ripper/depend Index: ext/ripper/depend =================================================================== --- ext/ripper/depend (revision 58815) +++ ext/ripper/depend (revision 58816) @@ -12,7 +12,7 @@ ripper.o: ripper.c https://github.com/ruby/ruby/blob/trunk/ext/ripper/depend#L12 .y.c: $(ECHO) compiling compiler $< $(Q) $(BISON) -t -v -oy.tab.c $< - $(Q) sed -f $(top_srcdir)/tool/ytab.sed -e "/^#/s!y\.tab\.c!$@!" y.tab.c > $@ + $(Q) sed -e "/^#/s!y\.tab\.c!$@!" -f $(top_srcdir)/tool/ytab.sed y.tab.c > $@ @$(RM) y.tab.c all: check -- ML: ruby-changes@q... Info: http://www.atdot.net/~ko1/quickml/