Skip to contentSkip to conversation

Fix grub-emu on Cygwin

PATCH Applied Christian Franke 2 participants 2 messages Apr 19, 2010 – Apr 24, 2010

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 Fix grub-emu on Cygwin

This fixes build of grub-emu on Cygwin, works also for --enable-grub-emu-modules.

Signature · 2 lines
Regards,
Christian Franke

=== modified file 'ChangeLog'

ChangeLog +7 -0
@@ -1,3 +1,10 @@
+2010-04-19  Christian Franke  <[email protected]>
+
+	* configure.ac: Do not CHECK_BSS_START_SYMBOL
+	and CHECK_END_SYMBOL if grub-emu is built.
+	Unset TARGET_OBJ2ELF if grub-emu is built
+	without module support.
+
 2010-04-19  BVK Chaitanya  <[email protected]>
 
 	Fix emu build with grub-emu-pci and grub-emu-modules.

=== modified file 'configure.ac'

configure.ac +11 -5
@@ -513,11 +513,11 @@
 fi
 grub_PROG_LD_BUILD_ID_NONE
 if test "x$target_cpu" = xi386; then
-  if test ! -z "$TARGET_IMG_LDSCRIPT"; then
-    # Check symbols provided by linker script.
-    CFLAGS="$TARGET_CFLAGS -nostdlib ${TARGET_IMG_LDFLAGS_AC}8000,--defsym,___main=0x8100"
-  fi
-  if test "x$TARGET_APPLE_CC" != x1 ; then
+  if test "$platform" != emu && test "x$TARGET_APPLE_CC" != x1 ; then
+    if test ! -z "$TARGET_IMG_LDSCRIPT"; then
+      # Check symbols provided by linker script.
+      CFLAGS="$TARGET_CFLAGS -nostdlib ${TARGET_IMG_LDFLAGS_AC}8000,--defsym,___main=0x8100"
+    fi
     grub_CHECK_BSS_START_SYMBOL
     grub_CHECK_END_SYMBOL
   fi
@@ -637,6 +637,12 @@
 fi
 AC_SUBST(TARGET_NO_MODULES)
 
+if test "$TARGET_NO_MODULES" = yes ; then
+  # Do not convert modules, otherwise linkage may fail (Cygwin only).
+  # FIXME: Should be checked above before TARGET_OBJ2ELF is set first.
+  TARGET_OBJ2ELF=
+fi
+
 if test x"$enable_grub_emu_usb" = xno ; then
   grub_emu_usb_excuse="explicitly disabled"
 fi