This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: PATCH: PR testsuite/12040: GDB Fortran tests use g77 instead of gfortran
- From: "H.J. Lu" <hjl dot tools at gmail dot com>
- To: Tom Tromey <tromey at redhat dot com>
- Cc: GDB <gdb-patches at sourceware dot org>
- Date: Tue, 28 Jun 2011 12:55:55 -0700
- Subject: Re: PATCH: PR testsuite/12040: GDB Fortran tests use g77 instead of gfortran
- References: <20100920135856.GA27953@intel.com> <m3hb797nqu.fsf@fleche.redhat.com>
On Tue, Jun 28, 2011 at 12:35 PM, Tom Tromey <tromey@redhat.com> wrote:
>>>>>> "HJ" == H J Lu <hongjiu.lu@intel.com> writes:
>
> HJ> This isn't a proper fix since it depends on we load lib/ada.exp before
> HJ> any Fortran tests. ?I think ada.exp is the wrong place to provide
> HJ> GDB functions. ?It should be placed in something like gdb-support.exp and
> HJ> ada.exp/fortran.exp can just load it.
>
> I'm sorry that it took so long for me to review this patch.
>
> HJ> ? ? PR testsuite/12040
> HJ> ? ? * lib/ada.exp (gdb_find_gfortran): New.
> HJ> ? ? (gdb_default_target_compile): Use it.
>
> I would like something like this to go in, but I don't think this will
> have the desired effect, since ada.exp is not always loaded.
>
> Instead I think these procs must be moved elsewhere so they are always
> used.
>
> I sent a patch to dejagnu to allow f90 today; so perhaps the result
> could be based on that. ?I can do the work if you like, just let me
> know.
Please do.
Thanks.
--
H.J.