This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

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


>>>>> "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.

Tom


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]