Re: [gdb-7.5.91] mips sim fails to build on mingw32

Joel Sherrill
2013-03-17T16:11:06Z

On 3/17/2013 11:02 AM, Joel Brobecker wrote:
>> 2013-03-16 Joel Sherrill <joel.sherrill@oarcorp.com>
>>
>> * configure.ac: Avoid use of dv-sockser on *mingw32*
>> * aclocal.m4: Regenerated.
>> * configure: Regenerated.
> [...]
>> +
>> SIM_AC_OPTION_HARDWARE($hw_enabled,$hw_devices,$hw_extra_devices)
>> -mips_extra_objs="dv-sockser.o"
>> -AC_SUBST(mips_extra_objs)
>> +
>> +mips_extra_objs=
>> +dnl make sure the dv-sockser code can be supported (i.e. windows)
>> +case ${host} in
>> + *mingw32*) ;;
> I would change that to *mingw*. We might get mingw64 someday...
:) The patch I posted simultaneously to address this in common/acinclude.m4
also has mingw32. I will fix it. But that patch needs more review.

cd sim
grep dv-sockser.o */Makefile.in
grep dv-sockser.o */configure.ac

And you will see what I need advice on. The patch in the next email
fixes bfin and mips I think. Scratches problem on other targets.
>
>> + *)
>> + AC_DEFINE_UNQUOTED([HAVE_DV_SOCKSER], 1, [Define if dv-sockser is usable.])
>> + mips_extra_objs="${mips_extra_objs} dv-sockser.o"
>> + ;;
>> +esac
>> +
>> +AC_SUBST([mips_extra_objs], ${mips_extra_objs})
>> +
>> +## mips_extra_objs="dv-sockser.o"
>> +## AC_SUBST(mips_extra_objs)
> Can you also remove the commented-out code?
>

The mips/configure.ac in the new patch relies on common/acinclude.m4 to
tell it whether or to use dv-sockser.o. This centralizes the test for
which hosts
it supports.

--
Joel Sherrill, Ph.D. Director of Research & Development
joel.sherrill@OARcorp.com On-Line Applications Research
Ask me about RTEMS: a free RTOS Huntsville AL 35805
Support Available (256) 722-9985