Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Andrew Cagney <ac131313@cygnus.com>
To: Hilfinger@gnat.com
Cc: Aidan Skinner <aidan@velvet.net>, gdb-patches@sources.redhat.com
Subject: Re: LEX vs FLEX; Was: [PATCH] Basic Ada files
Date: Thu, 23 May 2002 21:46:00 -0000	[thread overview]
Message-ID: <3CEDB860.9060308@cygnus.com> (raw)
In-Reply-To: <200205240300.UAA23495@tully.CS.Berkeley.EDU>

> (I note you're not the author of this)  I'm not exactly comfortable with 
>> making  FLEX a condition of being able to build GDB - while the above 
>> tries to hide it, the dependency still exists.  I guess we'll need to 
>> come back to that later.
> 
> 
> Andrew,
> 
> I am the author of that, so I suppose I should jump in.  I'm not quite clear
> on your objection here.  Is it 

Just FYI, the current objective is to get the files under CVS, and then 
get all relevant CORE-gdb changes and other problems resolved.

My request to Aidan was to just fix the Makefile.in patch submition so 
we can table it.  The Makefile.in change will then be committed last.

Any way for reference:

> * the dependence on flex as opposed to lex?

yes (but as a reservation, not an objection, I don't even know how 
pratical it is to get the code build using lex).

> * the dependence on either lex or flex (unlikely given the
>   dependencies on yacc)?

no

> * the option to use the .c code and NOT depend on (f)lex at all?

I suspect it will need changes.  Compare it to

c-exp.tab.c: c-exp.y
         $(SHELL) $(YLWRAP) "$(YACC)" $(srcdir)/c-exp.y  y.tab.c 
c-exp.tmp -- $(Y
FLAGS)
	....

However, like I said, all of this can be returned to later.  I'm not 
asking Aidan to fix it now.  I'm just looking for a patch sufficent to 
build/test the Ada files as Aidan commits them.

Andrew



  reply	other threads:[~2002-05-24  3:49 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2002-05-22 11:54 Aidan Skinner
2002-05-23 19:59 ` LEX vs FLEX; Was: " Andrew Cagney
2002-05-23 20:01   ` Paul Hilfinger
2002-05-23 21:46     ` Andrew Cagney [this message]
2002-05-24 12:41       ` Michael Snyder
2002-05-25 11:20   ` Aidan Skinner
2002-05-29 21:56     ` Andrew Cagney
2002-05-31 12:42       ` Aidan Skinner

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=3CEDB860.9060308@cygnus.com \
    --to=ac131313@cygnus.com \
    --cc=Hilfinger@gnat.com \
    --cc=aidan@velvet.net \
    --cc=gdb-patches@sources.redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox