Re: [New WinCE support] [patch 4/4] The bulk of the code.

[Pedro Alves <pedro_alves@portugalmail.pt>]

Daniel Jacobowitz wrote:
> On Mon, Mar 19, 2007 at 01:16:15AM +0000, Pedro Alves wrote:
>   
>> ChangeLog

>> 	* wince.c: Remove.
>> 	* wince-stub.h: Remove.
>> 	* wince-stub.c: Remove.

Humm, these files are still used by the current sh and mips WinCE 
ports.  As I
understand it, those ports are scheduled for removal.  I'll post a separate
patch to remove them all.

> This is OK.  Doesn't it deserve a NEWS entry?
>   

Thanks a bunch!

I'll prepare a NEWS entry separately.

> I didn't go over the Windows-specific changes too closely; I trust you
> :-)  I'll try not to break it too badly when working on the Linux ports.
>
>   

Thanks for the trust :)
I've tried to make sure I didn't break Cygwin gdbserver in the process.

>> @@ -52,8 +52,14 @@ case "${target}" in
>>  			srv_linux_regsets=yes
>>  			srv_linux_thread_db=yes
>>  			;;
>> +  arm*-*-mingw32ce*)	srv_regobj=reg-arm.o
>> +			srv_tgtobj="win32-low.o win32-arm-low.o"
>> +			srv_tgtobj="${srv_tgtobj} wincecompat.o"
>> +			srv_mingw=yes
>> +			srv_mingwce=yes
>> +			;;
>>    i[34567]86-*-mingw*)	srv_regobj=reg-i386.o
>> -			srv_tgtobj="win32-low.o"
>> +			srv_tgtobj="win32-low.o win32-i386-low.o"
>>  			srv_mingw=yes
>>  			;;
>>    ia64-*-linux*)	srv_regobj=reg-ia64.o
>>     
>
> Please put arm*-*-mingw32ce* up higher, near other ARM targets - the
> case statement is pretty much sorted.
>
>   

Ok, I'll do that.


> Should you check arm-wince-pe here too, since you did in the directory
> above?  Not that I expect a lot of native arm-wince-pe gdb :-)

I don't think so.  arm-wince-pe is supposed (at least that is what I get 
from the binutils archives
from the time Nick made the original binutils/gcc wince support, and 
from current FSF
arm-wince-pe-gcc) to use MSFT's headers and libs, and as such will not 
be able to build
gdbserver OOTB.  Nevertheless, it should be possible to have an 
arm-wince-pe-gdb
connect to a arm-wince-mingw32ce-gdbserver.

Cheers,
Pedro Alves