[#6864] ruby 1.8.4 rc breaks alias_method/rails in bad ways — "Ara.T.Howard" <ara.t.howard@...>

20 messages 2005/12/09
[#6870] Re: ruby 1.8.4 rc breaks alias_method/rails in bad ways — =?ISO-8859-15?Q?Florian_Gro=DF?= <florgro@...> 2005/12/12

Ara.T.Howard wrote:

[#6872] Re: ruby 1.8.4 rc breaks alias_method/rails in bad ways — ara.t.howard@... 2005/12/12

On Tue, 13 Dec 2005, [ISO-8859-15] Florian Growrote:

[#6873] Re: ruby 1.8.4 rc breaks alias_method/rails in bad ways — James Edward Gray II <james@...> 2005/12/12

On Dec 12, 2005, at 1:19 PM, ara.t.howard@noaa.gov wrote:

[#6874] Re: ruby 1.8.4 rc breaks alias_method/rails in bad ways — ara.t.howard@... 2005/12/12

On Tue, 13 Dec 2005, James Edward Gray II wrote:

[#6891] Time.utc! and Time.localtime! — Daniel Hobe <hobe@...>

Writing a script yesterday I found out, much to my surprise, that the

16 messages 2005/12/14

[#6918] change to yaml in 1.8.4 — ara.t.howard@...

14 messages 2005/12/16

[#6934] 1.8.x, YAML, and release management — Ryan Davis <ryand-ruby@...>

I'm concerned that 1.8.3's acceptance of non-backwards-compatible

28 messages 2005/12/18

[#6996] Problems building 1.8.4 with VS8 C++ Express Edition (cl 14.00) — Austin Ziegler <halostatue@...>

Visual Studio C++ 2005 Express Edition (VS 8.0)

20 messages 2005/12/27

Re: ruby 1.8.4 preview2

From: "daz" <dooby@...10.karoo.co.uk>
Date: 2005-12-02 06:22:22 UTC
List: ruby-core #6819
From: Nobu


>
> Or:
>
>   $ bison-2.1 -o parse.c parse.y && sed -i '/^    ;$/d' parse.c


+1

If I'm following correctly, 1.9 requires bison (for racc) but 1.8.4
can still use byacc (which is the default (YACC = byacc) in Makefile.sub).
- BTW, I just tried with byacc (to check) and there's no problem.

I (and Curt, probably) had errors from the prepared parse.c in the
distribution which has the bad line.

#-----------

For the case when  YACC = bison:

As my/(all?) Makefile.sub already uses sed, maybe your fix would fit inside?

   .y.c:
    $(YACC) $(YFLAGS) $(<:\=/)
    sed -e  ... y.tab.c > $(@F)
    @del y.tab.c


Single quotes don't work for me.
These are OK:

    sed -e "/^    ;$/d" ifile > ofile
OR:
    sed -e "/^yyparse ()$/{;n;/^ \+;$/d;}" ifile > ofile

I don't know sed, but the second is an attempt to delete the specific
offending line by checking the previous line, moving to the next
and deleting if it matches the empty line.
I guess you won't need it.
What might be interesting is that it took me about 3 hours to work that one line :(


daz





In This Thread