Quoting Christian Franke <christian.fra...@t-online.de>:

Good idea. Patch works on Cygwin. Does it work on Linux, FreeBSD, ... ?

It works on Linux.  I assume it should work on every OS that has GNU ld.

The variable TARGET_IMG_LDFLAGS_AC can also be removed, see attached patch.

I think we can do a big cleanup. The start address could be specified in the linker script. Some tests will become unnecessary. We should be able to move a lot of code from the configure script into the makefiles.

+/* Linker script to create grub .img files on Cygwin.  */

This comment is out of sync

I know.

VARIABLE(grub_end_addr)
-       .long   END_SYMBOL
+       .long   __bss_end__

This variable isn't used at all. In my Apple's CC patch I just remove it

OK, let's remove it.

'grub_start_addr' and '.globl start, _start' are also not used.

Patches are welcome.

--
Regards,
Pavel Roskin


_______________________________________________
Grub-devel mailing list
Grub-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/grub-devel

Reply via email to