[前][次][番号順一覧][スレッド一覧]

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/

[前][次][番号順一覧][スレッド一覧]