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

[Joel Sherrill <joel.sherrill@oarcorp.com>]

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