patch: Windows CE on x86

patch: Windows CE on x86

[Danny Backx]

[-- Attachment #1: Type: text/plain, Size: 183 bytes --]

The patch attached is a trivial one to add the target(s) for building
gdbserver for a x86 CE target.

	Danny
-- 
Danny Backx ; danny.backx - at - scarlet.be ; http://danny.backx.info

[-- Attachment #2: gdbserver-patch --]
[-- Type: text/x-patch, Size: 1411 bytes --]

Index: ChangeLog
===================================================================
RCS file: /cvs/src/src/gdb/gdbserver/ChangeLog,v
retrieving revision 1.260
diff -u -r1.260 ChangeLog
--- ChangeLog	3 Apr 2009 20:15:51 -0000	1.260
+++ ChangeLog	17 Apr 2009 18:48:10 -0000
@@ -1,3 +1,7 @@
+2009-04-17  Danny Backx  <dannybackx@users.sourceforge.net>
+
+	* configure.srv : Add x86 Windows CE target.
+
 2009-04-03  Ulrich Weigand  <uweigand@de.ibm.com>
 
 	* inferiors.c (get_thread_process): Make global.
Index: configure.srv
===================================================================
RCS file: /cvs/src/src/gdb/gdbserver/configure.srv,v
retrieving revision 1.39
diff -u -r1.39 configure.srv
--- configure.srv	18 Nov 2008 21:48:48 -0000	1.39
+++ configure.srv	17 Apr 2009 18:48:10 -0000
@@ -61,6 +61,15 @@
 			srv_linux_regsets=yes
 			srv_linux_thread_db=yes
 			;;
+  i[34567]86-*-mingw*ce*)
+                      srv_regobj=reg-i386.o
+                      srv_tgtobj="win32-low.o win32-i386-low.o"
+                      srv_tgtobj="${srv_tgtobj} wincecompat.o"
+                      # hostio_last_error implementation is in win32-low.c
+                      srv_hostio_err_objs=""
+                      srv_mingw=yes
+                      srv_mingwce=yes
+                      ;;
   i[34567]86-*-mingw*)	srv_regobj=reg-i386.o
 			srv_tgtobj="win32-low.o win32-i386-low.o"
 			srv_mingw=yes

Re: patch: Windows CE on x86

[Pedro Alves]

On Friday 17 April 2009 19:50:29, Danny Backx wrote:
> The patch attached is a trivial one to add the target(s) for building
> gdbserver for a x86 CE target.

Oh, exciting!  No other changes required?

I know that this is mostly copy&paste from the arm-mingw32ce
configuration, so I think it can pass by without a copyright
assignment.  In any case, do you have one in place with the FSF?

Just a small curiosity question before I apply this, is
there a reason this is:

> +  i[34567]86-*-mingw*ce*)

Instead of:

> + i[34567]86-*-mingw32ce*)

?

-- 
Pedro Alves

Re: patch: Windows CE on x86

[Danny Backx]

On Fri, 2009-04-17 at 20:44 +0100, Pedro Alves wrote:
> On Friday 17 April 2009 19:50:29, Danny Backx wrote:
> > The patch attached is a trivial one to add the target(s) for building
> > gdbserver for a x86 CE target.
> 
> Oh, exciting!  No other changes required?

No other changes : a clean build after this change.

> I know that this is mostly copy&paste from the arm-mingw32ce
> configuration, so I think it can pass by without a copyright
> assignment.  In any case, do you have one in place with the FSF?

I've always been lazy :-)
I'll get it done now though.

> Just a small curiosity question before I apply this, is
> there a reason this is:

An attempt on my part to generalize further. Feel free to remove convert
it to mingw32ce if you think that's better.

	Danny

> > +  i[34567]86-*-mingw*ce*)
> 
> Instead of:
> 
> > + i[34567]86-*-mingw32ce*)
> 
> ?
> 
-- 
Danny Backx ; danny.backx - at - scarlet.be ; http://danny.backx.info

Re: patch: Windows CE on x86

[Pedro Alves]

On Saturday 18 April 2009 05:59:08, Danny Backx wrote:
> On Fri, 2009-04-17 at 20:44 +0100, Pedro Alves wrote:
> 
> > I know that this is mostly copy&paste from the arm-mingw32ce
> > configuration, so I think it can pass by without a copyright
> > assignment.  In any case, do you have one in place with the FSF?
> 
> I've always been lazy :-)
> I'll get it done now though.

Thanks, that's great.  I don't think I know if I have access to
the forms.  Could someone please send him the forms?

> > Just a small curiosity question before I apply this, is
> > there a reason this is:
> 
> An attempt on my part to generalize further. Feel free to remove convert
> it to mingw32ce if you think that's better.

I've committed the patch with that change.  Thanks!

-- 
Pedro Alves

Re: patch: Windows CE on x86

[Pedro Alves]

On Friday 17 April 2009 19:50:29, Danny Backx wrote:
> The patch attached is a trivial one to add the target(s) for building
> gdbserver for a x86 CE target.

Pedro Alves wrote:
> I've committed the patch with that change.  Thanks!

New configuration support, probably should get a NEWS entry.  I can
take care of it.

Eli, is this OK?  There were no changes to GDB
itself, just configuration goo changes to gdbserver/ .

-- 
Pedro Alves

2009-04-19  Pedro Alves  <pedro@codesourcery.com>

	* NEWS: Mention gdbserver support for x86 Windows CE.

---
 gdb/NEWS |    3 +++
 1 file changed, 3 insertions(+)

Index: src/gdb/NEWS
===================================================================
--- src.orig/gdb/NEWS	2009-04-19 19:01:47.000000000 +0100
+++ src/gdb/NEWS	2009-04-19 19:04:34.000000000 +0100
@@ -295,6 +295,9 @@ x86_64 MinGW			x86_64-*-mingw*
 x86 DICOS			i[34567]86-*-dicos*
 x86_64 DICOS		        x86_64-*-dicos*
 
+* The GDB remote stub, gdbserver, now supports x86 Windows CE
+  (mingw32ce) debugging.
+
 * Removed commands
 
 catch load

Re: patch: Windows CE on x86

[Eli Zaretskii]

> From: Pedro Alves <pedro@codesourcery.com>
> Date: Sun, 19 Apr 2009 19:10:29 +0100
> Cc: danny.backx@scarlet.be,
>  Eli Zaretskii <eliz@gnu.org>
> 
> New configuration support, probably should get a NEWS entry.  I can
> take care of it.
> 
> Eli, is this OK?

Yes, thanks.

Re: patch: Windows CE on x86

[Pedro Alves]

On Sunday 19 April 2009 20:49:22, Eli Zaretskii wrote:
> > From: Pedro Alves <pedro@codesourcery.com>
> > Eli, is this OK?
> Yes, thanks.

Great.  Checked in.

-- 
Pedro Alves