Re: PATCH: PR testsuite/12040: GDB Fortran tests use g77 instead of gfortran

H.J. Lu
2011-06-28T19:56:10Z

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.