[Qemu-devel] PATCH: fix configure for cygwin

[Qemu-devel] PATCH: fix configure for cygwin

[Tristan Gingold]

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

Hi,


this patch fixes two minors issue for cygwin:
* --disable-kqemu didn't work for this configuration (because  
overridden)
* -mno-cygwin was missing in the final link.

Tristan.

[-- Attachment #2: qemu-mingw.diff --]
[-- Type: application/octet-stream, Size: 2014 bytes --]

Index: Makefile.target
===================================================================
RCS file: /sources/qemu/qemu/Makefile.target,v
retrieving revision 1.186
diff -c -p -r1.186 Makefile.target
*** Makefile.target	30 Jun 2007 17:32:17 -0000	1.186
--- Makefile.target	3 Jul 2007 12:58:09 -0000
*************** tcp_subr.o tcp_timer.o udp.o bootp.o deb
*** 494,500 ****
  VL_OBJS+=$(addprefix slirp/, $(SLIRP_OBJS))
  endif
  
! VL_LDFLAGS=
  VL_LIBS=$(AIOLIBS)
  # specific flags are needed for non soft mmu emulator
  ifdef CONFIG_STATIC
--- 494,500 ----
  VL_OBJS+=$(addprefix slirp/, $(SLIRP_OBJS))
  endif
  
! VL_LDFLAGS=$(VL_OS_LDFLAGS)
  VL_LIBS=$(AIOLIBS)
  # specific flags are needed for non soft mmu emulator
  ifdef CONFIG_STATIC
Index: configure
===================================================================
RCS file: /sources/qemu/qemu/configure,v
retrieving revision 1.146
diff -c -p -r1.146 configure
*** configure	23 Jun 2007 16:03:35 -0000	1.146
--- configure	3 Jul 2007 12:58:09 -0000
*************** case $targetos in
*** 108,116 ****
--- 108,123 ----
  CYGWIN*)
  mingw32="yes"
  OS_CFLAGS="-mno-cygwin"
+ VL_OS_LDFLAGS="-mno-cygwin"
+ if [ "$cpu" = "i386" ] ; then
+     kqemu="yes"
+ fi
  ;;
  MINGW32*)
  mingw32="yes"
+ if [ "$cpu" = "i386" ] ; then
+     kqemu="yes"
+ fi
  ;;
  GNU/kFreeBSD)
  oss="yes"
*************** if test "$mingw32" = "yes" ; then
*** 390,398 ****
      linux="no"
      EXESUF=".exe"
      oss="no"
-     if [ "$cpu" = "i386" ] ; then
-         kqemu="yes"
-     fi
  fi
  
  # Check for gcc4, error if pre-gcc4 
--- 397,402 ----
*************** echo "AR=$ar" >> $config_mak
*** 714,719 ****
--- 718,724 ----
  echo "STRIP=$strip -s -R .comment -R .note" >> $config_mak
  echo "OS_CFLAGS=$OS_CFLAGS" >> $config_mak
  echo "OS_LDFLAGS=$OS_LDFLAGS" >> $config_mak
+ echo "VL_OS_LDFLAGS=$VL_OS_LDFLAGS" >> $config_mak
  echo "ARCH_CFLAGS=$ARCH_CFLAGS" >> $config_mak
  echo "ARCH_LDFLAGS=$ARCH_LDFLAGS" >> $config_mak
  echo "CFLAGS=$CFLAGS" >> $config_mak

[Qemu-devel] Ping: PATCH: fix configure for cygwin

[Tristan Gingold]

Hi,

any comment/review on this patch ?

Is there any interest for Win32 ?  I have patches (serial input from  
console) to port to the current version.

TIA,
Tristan.

On Jul 3, 2007, at 3:03 PM, Tristan Gingold wrote:
> this patch fixes two minors issue for cygwin:
> * --disable-kqemu didn't work for this configuration (because  
> overridden)
> * -mno-cygwin was missing in the final link.
>
> Tristan.
> <qemu-mingw.diff>

Re: [Qemu-devel] Ping: PATCH: fix configure for cygwin

[Thiemo Seufer]

Tristan Gingold wrote:
> Hi,
>
> any comment/review on this patch ?
>
> Is there any interest for Win32 ?  I have patches (serial input from 
> console) to port to the current version.

Committed.


Thiemo

Re: [Qemu-devel] Ping: PATCH: fix configure for cygwin

[Tristan Gingold]

On Jul 12, 2007, at 11:30 AM, Thiemo Seufer wrote:

> Tristan Gingold wrote:
>> Hi,
>>
>> any comment/review on this patch ?
>>
>> Is there any interest for Win32 ?  I have patches (serial input from
>> console) to port to the current version.
>
> Committed.

Thanks!