* [RFA] [3/4] Mingw64 gdbserver support
@ 2010-04-19 22:58 Pierre Muller
2010-04-19 23:30 ` Pedro Alves
0 siblings, 1 reply; 4+ messages in thread
From: Pierre Muller @ 2010-04-19 22:58 UTC (permalink / raw)
To: gdb-patches; +Cc: 'Pedro Alves'
gdbserver/configure.srv part of the patch.
Pierre Muller
Pascal language support maintainer for GDB
2010-04-19 Pierre Muller <muller@ics.u-strasbg.fr>
* configure.srv (x86_64-*-mingw*): New configuration for Windows
64-bit executables.
Set srv_mingw64 variable to `yes'.
Index: gdbserver/configure.srv
===================================================================
RCS file: /cvs/src/src/gdb/gdbserver/configure.srv,v
retrieving revision 1.54
diff -u -p -r1.54 configure.srv
--- gdbserver/configure.srv 19 Apr 2010 14:08:05 -0000 1.54
+++ gdbserver/configure.srv 19 Apr 2010 22:42:53 -0000
@@ -231,6 +231,13 @@ case "${target}" in
srv_linux_regsets=yes
srv_linux_thread_db=yes
;;
+ x86_64-w64-mingw*) srv_regobj="$srv_amd64_regobj"
+ srv_tgtobj="i386-low.o i387-fp.o win32-low.o
win32-i386-low.o"
+ srv_xmlfiles="$srv_i386_xmlfiles
$srv_amd64_xmlfiles"
+ srv_mingw64=yes
+ srv_mingw=yes
+ ;;
+
xscale*-*-linux*) srv_regobj=reg-arm.o
srv_tgtobj="linux-low.o linux-arm-low.o"
srv_linux_usrregs=yes
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [RFA] [3/4] Mingw64 gdbserver support
2010-04-19 22:58 [RFA] [3/4] Mingw64 gdbserver support Pierre Muller
@ 2010-04-19 23:30 ` Pedro Alves
2010-04-19 23:41 ` Pierre Muller
0 siblings, 1 reply; 4+ messages in thread
From: Pedro Alves @ 2010-04-19 23:30 UTC (permalink / raw)
To: Pierre Muller; +Cc: gdb-patches
On Monday 19 April 2010 23:58:55, Pierre Muller wrote:
> gdbserver/configure.srv part of the patch.
>
>
>
> Pierre Muller
> Pascal language support maintainer for GDB
>
> 2010-04-19 Pierre Muller <muller@ics.u-strasbg.fr>
>
> * configure.srv (x86_64-*-mingw*): New configuration for Windows
> 64-bit executables.
Weren't you going to s/x86_64-w64-mingw/x86_64-*-mingw/ ? The ChangeLog
mentions the latter, but the patch does the former.
> Set srv_mingw64 variable to `yes'.
Why is this needed? I don't see anywhere in the patch series
using it. Wasn't this needed in the older version to have configure.ac
link with -lws2_32 for this config only, but it's not needed anymore?
>
> Index: gdbserver/configure.srv
> ===================================================================
> RCS file: /cvs/src/src/gdb/gdbserver/configure.srv,v
> retrieving revision 1.54
> diff -u -p -r1.54 configure.srv
> --- gdbserver/configure.srv 19 Apr 2010 14:08:05 -0000 1.54
> +++ gdbserver/configure.srv 19 Apr 2010 22:42:53 -0000
> @@ -231,6 +231,13 @@ case "${target}" in
> srv_linux_regsets=yes
> srv_linux_thread_db=yes
> ;;
> + x86_64-w64-mingw*) srv_regobj="$srv_amd64_regobj"
> + srv_tgtobj="i386-low.o i387-fp.o win32-low.o
> win32-i386-low.o"
> + srv_xmlfiles="$srv_i386_xmlfiles
> $srv_amd64_xmlfiles"
> + srv_mingw64=yes
> + srv_mingw=yes
> + ;;
> +
> xscale*-*-linux*) srv_regobj=reg-arm.o
> srv_tgtobj="linux-low.o linux-arm-low.o"
> srv_linux_usrregs=yes
>
--
Pedro Alves
^ permalink raw reply [flat|nested] 4+ messages in thread* RE: [RFA] [3/4] Mingw64 gdbserver support
2010-04-19 23:30 ` Pedro Alves
@ 2010-04-19 23:41 ` Pierre Muller
2010-04-19 23:56 ` Pedro Alves
0 siblings, 1 reply; 4+ messages in thread
From: Pierre Muller @ 2010-04-19 23:41 UTC (permalink / raw)
To: 'Pedro Alves'; +Cc: gdb-patches
> -----Message d'origine-----
> De : Pedro Alves [mailto:pedro@codesourcery.com]
> Envoyé : Tuesday, April 20, 2010 1:30 AM
> À : Pierre Muller
> Cc : gdb-patches@sourceware.org
> Objet : Re: [RFA] [3/4] Mingw64 gdbserver support
>
> On Monday 19 April 2010 23:58:55, Pierre Muller wrote:
> > gdbserver/configure.srv part of the patch.
> >
> >
> >
> > Pierre Muller
> > Pascal language support maintainer for GDB
> >
> > 2010-04-19 Pierre Muller <muller@ics.u-strasbg.fr>
> >
> > * configure.srv (x86_64-*-mingw*): New configuration for
> Windows
> > 64-bit executables.
>
> Weren't you going to s/x86_64-w64-mingw/x86_64-*-mingw/ ? The
> ChangeLog
> mentions the latter, but the patch does the former.
Whooops, sorry for this.
> > Set srv_mingw64 variable to `yes'.
>
> Why is this needed? I don't see anywhere in the patch series
> using it. Wasn't this needed in the older version to have configure.ac
> link with -lws2_32 for this config only, but it's not needed anymore?
You are perfectly right.
Still simpler!
2010-04-19 Pierre Muller <muller@ics.u-strasbg.fr>
* configure.srv (x86_64-*-mingw*): New configuration for Windows
64-bit executables.
Index: gdbserver/configure.srv
===================================================================
RCS file: /cvs/src/src/gdb/gdbserver/configure.srv,v
retrieving revision 1.54
diff -u -p -r1.54 configure.srv
--- gdbserver/configure.srv 19 Apr 2010 14:08:05 -0000 1.54
+++ gdbserver/configure.srv 19 Apr 2010 22:42:53 -0000
@@ -231,6 +231,12 @@ case "${target}" in
srv_linux_regsets=yes
srv_linux_thread_db=yes
;;
+ x86_64-*-mingw*) srv_regobj="$srv_amd64_regobj"
+ srv_tgtobj="i386-low.o i387-fp.o win32-low.o
win32-i386-low.o"
+ srv_xmlfiles="$srv_i386_xmlfiles
$srv_amd64_xmlfiles"
+ srv_mingw=yes
+ ;;
+
xscale*-*-linux*) srv_regobj=reg-arm.o
srv_tgtobj="linux-low.o linux-arm-low.o"
srv_linux_usrregs=yes
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2010-04-19 23:56 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-04-19 22:58 [RFA] [3/4] Mingw64 gdbserver support Pierre Muller
2010-04-19 23:30 ` Pedro Alves
2010-04-19 23:41 ` Pierre Muller
2010-04-19 23:56 ` Pedro Alves
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox