Re: [PATCH] sun partition map support

2005-02-21 Thread Yoshinori K. Okuji
On Monday 21 February 2005 20:25, Vincent Pelletier wrote:
> Ping ? :)

I want to ping you as well.

Okuji


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


Re: [PATCH] sun partition map support

2005-02-21 Thread Marco Gerards
Vincent Pelletier <[EMAIL PROTECTED]> writes:

> Marco Gerards wrote:
> | This patch will be committed on Friday, if I don't hear anything
> | before then.
>
> Ping ? :)

Whoops!  I have committed your patch.  Thanks again and sorry for the
delay.

--
Marco




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


[PATCH] dprintf implementation

2005-02-21 Thread Vincent Pelletier
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
Hello.
I haven't worked on grub2 for 2 weeks now, sorry.
Here is the dprintf function I worked on 2 weeks ago, which I fixed this
evening.
Vincent Pelletier
2005-02-21  Vincent Pelletier  <[EMAIL PROTECTED]>
* include/grub/misc.h (grub_dprintf): New macro.
(grub_real_dprintf): New prototype.
* kern/misc.c (grub_real_dprintf): New function.
Index: kern/misc.c
===
RCS file: /cvsroot/grub/grub2/kern/misc.c,v
retrieving revision 1.18
diff -u -p -r1.18 misc.c
- --- kern/misc.c   19 Feb 2005 20:56:07 -  1.18
+++ kern/misc.c 21 Feb 2005 20:38:38 -
@@ -128,6 +128,18 @@ grub_printf (const char *fmt, ...)
~   return ret;
~ }
+void
+grub_real_dprintf(const char *file, const int line, const char *title,
+  const char *fmt, ...)
+{
+  va_list args;
+
+  grub_printf ("%s,%d (%s): ", file, line, title);
+  va_start (args, fmt);
+  grub_vprintf (fmt, args);
+  va_end (args);
+}
+
~ int
~ grub_vprintf (const char *fmt, va_list args)
~ {
Index: include/grub/misc.h
===
RCS file: /cvsroot/grub/grub2/include/grub/misc.h,v
retrieving revision 1.12
diff -u -p -r1.12 misc.h
- --- include/grub/misc.h   29 Jan 2005 22:01:53 -  1.12
+++ include/grub/misc.h 21 Feb 2005 20:38:38 -
@@ -26,6 +26,7 @@
~ #include 
~ #include 
+#define grub_dprintf(title,fmt,args...)
grub_real_dprintf(__FILE__,__LINE__,title,fmt,args);
~ /* XXX: If grub_memmove is too slow, we must implement grub_memcpy.  */
~ #define grub_memcpy(d,s,n)grub_memmove ((d), (s), (n))
@@ -58,6 +59,10 @@ char *EXPORT_FUNC(grub_strndup) (const c
~ void *EXPORT_FUNC(grub_memset) (void *s, int c, grub_size_t n);
~ grub_size_t EXPORT_FUNC(grub_strlen) (const char *s);
~ int EXPORT_FUNC(grub_printf) (const char *fmt, ...) __attribute__
((format (printf, 1, 2)));
+void EXPORT_FUNC(grub_real_dprintf) (const char *file,
+ const int line,
+ const char *title,
+ const char *fmt, ...)
__attribute__ ((format (printf, 4, 5)));
~ int EXPORT_FUNC(grub_vprintf) (const char *fmt, va_list args);
~ int EXPORT_FUNC(grub_sprintf) (char *str, const char *fmt, ...)
__attribute__ ((format (printf, 2, 3)));
~ int EXPORT_FUNC(grub_vsprintf) (char *str, const char *fmt, va_list args);
-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.0 (GNU/Linux)
Comment: Using GnuPG with Thunderbird - http://enigmail.mozdev.org
iD8DBQFCGkzqFEQoKRQyjtURAokmAJ9Mirp8xseh2AwUJOl+tykvKm/lHgCeM07L
2pNw3UbeFYot+LOR4vDFWH4=
=+3BC
-END PGP SIGNATURE-

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


Re: [PATCH] sun partition map support

2005-02-21 Thread Vincent Pelletier
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
Marco Gerards wrote:
| This patch will be committed on Friday, if I don't hear anything
| before then.
Ping ? :)
Vincent Pelletier
-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.0 (GNU/Linux)
Comment: Using GnuPG with Thunderbird - http://enigmail.mozdev.org
iD8DBQFCGjW0FEQoKRQyjtURAi8iAJ49gGmU4MEL5psu11lAq69Exefh6QCfUomn
CVvkdQ388bGdAg1LtTreI+U=
=iIWb
-END PGP SIGNATURE-

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


Re: [patch] set prefix on PPC

2005-02-21 Thread Marco Gerards
Hollis Blanchard <[EMAIL PROTECTED]> writes:

> This code sets the GRUB "prefix" environment variable from the OF
> /chosen/bootpath property. It must therefore translate between the two
> syntaxes.

Sven just told me bootpath does not work on the pegasos II, although
it will be fixed.  So the argument will still be required, I guess.  I
will send in a patch for that later (one that will work with this
patch applied).

> I believe most of this code could be moved to a kern/ieee1275/device.c
> file, but that can wait until there is another supported OF platform.

Right.  A lot of code can be moved and shared.

> I have verified that this works (a config file is loaded) from disk, and
> netbooting is not broken. The subdirectory stuff works as well.

Ok.  I still have to test it... :/

> 2005-02-13  Hollis Blanchard  <[EMAIL PROTECTED]>

[...]

>   (GRUB_PARSE_PARTITION): Likewise.

Just make this a new enum.  So name the enum and say "New enum
`foo'.".  See my comment below.


> Index: kern/powerpc/ieee1275/init.c

[...]

> +/* Translate an OF filesystem path (separated by backslashes), into a GRUB
> +   path (separated by forward slashes).  */
> +static void
> +translate_path (char *filepath)

Can you add a prefix?

> +static void
> +grub_set_prefix (void)
> +{
> +  char bootpath[64]; /* XXX check length */

You could get the property length and use that.  It would be clearer.
See my patch for passing the prefix as an argument for an example of
this.  I think this was done like this before, but that should be
changed as well. :)

> Index: kern/powerpc/ieee1275/openfw.c

[...]

> +enum {

Please put the { on a new line.

> +  GRUB_PARSE_FILENAME,
> +  GRUB_PARSE_PARTITION,
> +};

I think there can be some problems with unnamed enums.  I am not sure
though...

> +
>  /* Walk children of 'devpath', calling hook for each.  */
>  grub_err_t
>  grub_children_iterate (char *devpath,
> @@ -64,7 +69,7 @@ grub_children_iterate (char *devpath,
>if (actual == -1)
>   continue;
>  
> -  grub_sprintf(fullname, "%s/%s", devpath, childname);
> +  grub_sprintf (fullname, "%s/%s", devpath, childname);
>  
>alias.type = childtype;
>alias.path = childpath;
> @@ -199,3 +204,133 @@ grub_claimmap (grub_addr_t addr, grub_si
>  
>return 0;
>  }
> +
> +/* Get the device arguments of the Open Firmware device specifier `path'.  */
> +static char *

Device arguments are like the partition number or so?

> +static char *
> +grub_ieee1275_parse_args (const char *path, int field)
> +{
> +  char type[64]; /* XXX check size.  */

Same stuff as before. :)

> +  char *device = grub_ieee1275_get_devname (path);
> +  char *args = grub_ieee1275_get_devargs (path);
> +  char *ret = 0;
> +  grub_ieee1275_phandle_t dev;
> +
> +  if (!args)
> +/* Shouldn't happen.  */
> +return 0;

If I understood your code correctly it can't happen at all, in that
case the test would be useless.

> +
> +  /* We need to know what type of device it is in order to parse the full
> + file path properly.  */
> +  if (grub_ieee1275_finddevice (device, &dev))
> +{
> +  grub_error (GRUB_ERR_UNKNOWN_DEVICE, "Device %s not found\n", device);
> +  goto out;
> +}
> +  if (grub_ieee1275_get_property (dev, "device_type", &type, sizeof type, 0))
> +{
> +  grub_error (GRUB_ERR_UNKNOWN_DEVICE,
> +   "Device %s lacks a device_type property\n", device);
> +  goto out;
> +}
> +
> +  if (!grub_strcmp ("block", type))
> +{
> +  /* Example: "disk:,".  */

Is it always like this without exceptions?  Will this always be an
alias or will this be copied from "boot"?

> +   ret = grub_malloc (grub_strlen (filepath) + 1);
> +   /* Make sure filepath has leading backslash.  */
> +   if (filepath[0] != '\\')
> + grub_sprintf (ret, "\\%s", filepath);
> +   else
> + grub_strcpy (ret, filepath);

Why is this required?

> +out:

Please use "fail:", just to be consistent.

> +char *
> +grub_ieee1275_get_dev_encoding (const char *path)
> +{
> +  char *device = grub_ieee1275_get_devname (path);
> +  char *partition = grub_ieee1275_parse_args (path, GRUB_PARSE_PARTITION);
> +  char *encoding;
> +
> +  if (partition)
> +{
> +  unsigned int partno = grub_strtoul (partition, 0, 0);
> +  partno--; /* GRUB partition numbering is 0-based.  */

Right.  But how can you be sure both match?

> +
> +  /* XXX Assume partno will only require two bytes? */
> +  encoding = grub_malloc (grub_strlen (device) + 5);

Can't you just calculate this?  Otherwise you can better allocate a
few bytes too much.

Thanks,
Marco



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


Re: [patch] PPC build fixes

2005-02-21 Thread Marco Gerards
Hollis Blanchard <[EMAIL PROTECTED]> writes:

> On Feb 20, 2005, at 7:39 AM, Yoshinori K. Okuji wrote:
>
>> On Sunday 20 February 2005 00:34, Hollis Blanchard wrote:
>>> - grub_reboot and grub_halt in util/i386/pc/misc.c are not
>>>   architecture-specific, so have been moved to util/grub-emu.c.
>>
>> grub_halt is clearly arch-specific. Look at the definition grub_halt in
>> PC. It takes one argument, while the PPC version does not. I expect
>> that the same thing might happen in grub_reboot as well.
>
> Oops, I meant to convert that into using a more generic "flags"
> argument, and then "no_apm" becomes one bit in it. How would that be?

And how would the parser work?  What if more options have to be added
for other archs?  I do not have a strong objection against using
flags, I am just afraid it might be inadequate and that the current
code is easier to maintain.

Thanks,
Marco



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


Re: [patch] PPC build fixes

2005-02-21 Thread Marco Gerards
Hollis Blanchard <[EMAIL PROTECTED]> writes:

> This patch fixes PowerPC build breakage from the recent grub-emu changes. Of
> note:
>
> - grub_reboot and grub_halt in util/i386/pc/misc.c are not
>   architecture-specific, so have been moved to util/grub-emu.c.
>
> - commands/ieee1275/halt.c and reboot.c are no longer
>   architecture-specific either. This is because we now want those
>   commands to be loaded in grub-emu, and of course grub-emu does not
>   emulate Open Firmware. Accordingly, these files are moved up to the
>   commands directory. Since i386 needs a special argument ("no-apm"), it
>   still uses its version in commands/i386/pc.
>
> - since grub_machine_fini isn't called yet, I haven't yet figured out
>   how to release the memory indicated in the comments.
>
> OK?
>
> -Hollis
>
> 2005-02-19  Hollis Blanchard  <[EMAIL PROTECTED]>
>
>   * commands/ieee1275/halt.c (grub_cmd_halt): Call grub_halt.
>   Move file...
>   * commands/halt.c: ... to here.
>   * commands/ieee1275/reboot.c (grub_cmd_reboot): Call grub_reboot.
>   Move file...
>   * commands/reboot.c: ... to here.

If you do this, you could delete the PC specific reboot and halt
commands.  Personally I like separate reboot and halt commands.  If
you do it like this it will become hard/ugly to make platform specific
changes, I think.

>   * disk/powerpc/ieee1275/ofdisk.c (grub_ofdisk_fini): New
>   function.
>   * include/grub/powerpc/ieee1275/ieee1275.h (grub_reboot): Add
>   prototype.

Better put this in `include/grub/powerpc/ieee1275/init.h', just like
it is done for the PC.  Perhaps it would be even better to have a new
header file for such things.

>   (grub_halt): Likewise.
>   * kern/powerpc/ieee1275/init.c (heap_start): Make global.
>   (heap_len): Likewise.

`New variable.' would be more appropriate, I think.  Can you put a
prefix before the function name?

>   (grub_machine_fini): New function.

It's really nice that we have this now. :)

>   * kern/powerpc/ieee1275/openfw.c (grub_reboot): New function.
>   (grub_halt): Likewise.
>   * term/powerpc/ieee1275/ofconsole.c (grub_ofconsole_fini): New
>   function.

You have added grub_console_fini, not grub_ofconsole_fini.

>  void
> +grub_machine_fini (void)
> +{
> +  grub_ofdisk_fini ();
> +  grub_console_fini ();
> +
> +  grub_ieee1275_release (heap_start, heap_len);
> +  /* XXX Release memory claimed in 'linux' and 'initrd' commands.  */
> +  /* XXX Release memory claimed for Old World firmware.  */
> +}

Why would you release the memory for linux and initrd?  IIRC there is
a callback function.

I think it should work like this:

In case someone quits GRUB:

- longjump (?)
- Release the memory for the loader (use grub_loader_unset).
- Call grub_console_fini.
- Call grub_ofdisk_fini.
- Release the heap.

In case someone starts a loader the same should be done, except
calling grub_loader_unset.  After that the OS should be loaded.  I
wonder what should happen when the OS can not be loaded.  Perhaps GRUB
just has to fail or so.

Thanks,
Marco



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


Re: Fosdem [was: menu entry editor]

2005-02-21 Thread marco
Nico -telmich- Schottelius <[EMAIL PROTECTED]> writes:

> marco [Sun, Feb 20, 2005 at 10:18:31PM +0100]:
>> "Yoshinori K. Okuji" <[EMAIL PROTECTED]> writes:
>> 
>> > On Sunday 20 February 2005 18:45, Marco Gerards wrote:
>> >> If it is possible, I will try to test this before I go to FOSDEM next
>> >> week...
>> >
>> > BTW, do you know which room is booked for GNU Hurd? I don't find the 
>> > information on the web site. I will ask someone when I reach there, but 
>> > I am very afraid of losing my way.
>> 
>> In room AW1.115, see http://www.fosdem.org/2005/index/schedule.  I
>> will give a small talk about GRUB 2 there.  It will be mainly a
>> technical talk.
>
> Will your speech be available as video for download?

No.  It is not an official talk.

--
Marco



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


Re: build error on ppc

2005-02-21 Thread Hollis Blanchard
On Feb 21, 2005, at 3:58 AM, Nico -telmich- Schottelius wrote:
I just wanted to compile grub2/cvs, but configure fails:
[10:31] ei:grub2% ./configure --prefix=/usr/packages/grub2 && make
[...]
checking for objcopy... objcopy
checking whether objcopy works for absolute addresses... configure: 
error: gcc cann
ot link at address 2000
[10:31] ei:grub2%

Calling
[10:49] ei:grub2% autoconf && autoheader
does not fix that.
After commenting out
#grub_PROG_OBJCOPY_ABSOLUTE
in configure.ac, autoconf && autoheader, ./configure works fine.
That test passes for me. I'm slightly curious about what exactly it's 
doing and why it's there, but have never looked into it.

...
checking whether objcopy works for absolute addresses... yes
...
objcopy --version says "GNU objcopy 2.15" for me.
But then make fails:
-- snip --
gcc -o grub-emu grub_emu-kern_main.o grub_emu-kern_device.o 
grub_emu-kern_disk.o grub_emu-kern_dl.o grub_emu-kern_file.o 
grub_emu-kern_fs.o grub_emu-kern_err.o grub_emu-kern_misc.o 
grub_emu-kern_loader.o grub_emu-kern_rescue.o grub_emu-kern_term.o 
grub_emu-partmap_amiga.o grub_emu-partmap_pc.o 
grub_emu-partmap_apple.o grub_emu-fs_fshelp.o 
grub_emu-util_i386_pc_biosdisk.o grub_emu-fs_fat.o grub_emu-fs_ext2.o 
grub_emu-fs_ufs.o grub_emu-fs_minix.o grub_emu-fs_hfs.o 
grub_emu-fs_jfs.o grub_emu-fs_iso9660.o grub_emu-normal_cmdline.o 
grub_emu-normal_command.o grub_emu-normal_main.o 
grub_emu-normal_menu.o grub_emu-normal_arg.o grub_emu-kern_partition.o 
grub_emu-util_console.o grub_emu-util_grub_emu.o grub_emu-util_misc.o 
grub_emu-util_i386_pc_getroot.o grub_emu-kern_env.o 
grub_emu-disk_loopback.o grub_emu-commands_ls.o 
grub_emu-commands_help.o grub_emu-commands_terminal.o 
grub_emu-commands_boot.o grub_emu-commands_cmp.o 
grub_emu-commands_cat.o  -lncurses
grub_emu-normal_menu.o(.text+0x944): In function `grub_menu_run':
normal/menu.c:296: undefined reference to `grub_menu_entry_run'
grub_emu-util_grub_emu.o(.text+0x414): In function `main':
util/grub-emu.c:191: undefined reference to `grub_halt_init'
grub_emu-util_grub_emu.o(.text+0x418):util/grub-emu.c:192: undefined 
reference to `grub_reboot_init'
grub_emu-util_grub_emu.o(.text+0x430):util/grub-emu.c:201: undefined 
reference to `grub_reboot_fini'
grub_emu-util_grub_emu.o(.text+0x434):util/grub-emu.c:202: undefined 
reference to `grub_halt_fini'
collect2: ld returned 1 exit status
make: *** [grub-emu] Error 1
For a temporary fix, you can apply the patch I sent out 19 Feb 2005, in 
a mail titled "PPC build fixes."

However as you can see from that thread, there is one more addition I 
need to make before I can commit the patch, and I won't have time for 
that tonight. So it will be at least another day or two before CVS will 
build for PPC.

-Hollis

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


x86 serial support

2005-02-21 Thread Omniflux
Here is a patch to add serial support to x86.
It is a porting of grub-legacy's serial support, copies of grub-legacy's 
tparms.{c,h} and terminfo.{c.h} and modifications based on grub2's 
term/i386/pc/vga.c, disk/loopback.c (to add a grub command), and 
util/console.c

It still needs a lot of work (it does not even let you specify serial 
port settings yet), but I don't want to spend any more time on it if it 
is not going to be used.

If this is headed in the right direction, and likely to be accepted for 
merging with a little more work, I'll keep working on it.

Please let me know.
--
Omniflux
diff -uNr grub2/conf/i386-pc.mk grub2.newserial/conf/i386-pc.mk
--- grub2/conf/i386-pc.mk   2005-02-19 20:56:06.0 +
+++ grub2.newserial/conf/i386-pc.mk 2005-02-21 12:20:18.044758032 +
@@ -253,7 +253,7 @@
file.h fs.h kernel.h loader.h misc.h mm.h net.h rescue.h symbol.h \
term.h types.h machine/biosdisk.h machine/boot.h \
machine/console.h machine/init.h machine/memory.h \
-   machine/loader.h partition.h pc_partition.h machine/time.h machine/vga.h
+   machine/loader.h partition.h pc_partition.h machine/time.h 
machine/vga.h machine/serial.h
 kernel_img_CFLAGS = $(COMMON_CFLAGS)
 kernel_img_ASFLAGS = $(COMMON_ASFLAGS)
 kernel_img_LDFLAGS = -nostdlib -Wl,-N,-Ttext,8200
@@ -902,7 +902,7 @@
 pkgdata_MODULES = _chain.mod _linux.mod linux.mod fat.mod ufs.mod ext2.mod 
minix.mod \
hfs.mod jfs.mod normal.mod hello.mod vga.mod font.mod _multiboot.mod 
ls.mod \
boot.mod cmp.mod cat.mod terminal.mod fshelp.mod chain.mod 
multiboot.mod \
-   amiga.mod apple.mod pc.mod loopback.mod reboot.mod halt.mod help.mod
+   amiga.mod apple.mod pc.mod loopback.mod reboot.mod halt.mod help.mod 
serial.mod
 
 # For _chain.mod.
 _chain_mod_SOURCES = loader/i386/pc/chainloader.c
@@ -1817,6 +1817,51 @@
 
 vga_mod_CFLAGS = $(COMMON_CFLAGS)
 
+# For serial.mod.
+serial_mod_SOURCES = term/i386/pc/serial.c term/tparm.c term/terminfo.c
+CLEANFILES += serial.mod mod-serial.o mod-serial.c pre-serial.o 
serial_mod-term_i386_pc_serial.o def-serial.lst und-serial.lst
+MOSTLYCLEANFILES += serial_mod-term_i386_pc_serial.d
+DEFSYMFILES += def-serial.lst
+UNDSYMFILES += und-serial.lst
+
+serial.mod: pre-serial.o mod-serial.o
+   -rm -f $@
+   $(LD) -r -d -o $@ $^
+   $(STRIP) --strip-unneeded -K grub_mod_init -K grub_mod_fini -R .note -R 
.comment $@
+
+pre-serial.o: serial_mod-term_i386_pc_serial.o 
serial_mod-term_i386_pc_terminfo.o serial_mod-term_i386_pc_tparm.o
+   -rm -f $@
+   $(LD) -r -d -o $@ $^
+
+mod-serial.o: mod-serial.c
+   $(CC) $(CPPFLAGS) $(CFLAGS) $(serial_mod_CFLAGS) -c -o $@ $<
+
+mod-serial.c: moddep.lst genmodsrc.sh
+   sh $(srcdir)/genmodsrc.sh 'serial' $< > $@ || (rm -f $@; exit 1)
+
+def-serial.lst: pre-serial.o
+   $(NM) -g --defined-only -P -p $< | sed 's/^\([^ ]*\).*/\1 serial/' > $@
+
+und-serial.lst: pre-serial.o
+   echo 'serial' > $@
+   $(NM) -u -P -p $< | cut -f1 -d' ' >> $@
+
+serial_mod-term_i386_pc_serial.o: term/i386/pc/serial.c
+   $(CC) -Iterm/i386/pc -I$(srcdir)/term/i386/pc $(CPPFLAGS) $(CFLAGS) 
$(serial_mod_CFLAGS) -c -o $@ $<
+
+serial_mod-term_i386_pc_serial.d: term/i386/pc/serial.c
+   set -e; $(CC) -Iterm/i386/pc -I$(srcdir)/term/i386/pc $(CPPFLAGS) 
$(CFLAGS) $(serial_mod_CFLAGS) -M $<  | sed 's,serial\.o[ 
:]*,serial_mod-term_i386_pc_serial.o $@ : ,g' > $@; [ -s $@ ] || rm -f $@
+
+serial_mod-term_i386_pc_terminfo.o: term/terminfo.c
+   $(CC) -Iterm -I$(srcdir)/term $(CPPFLAGS) $(CFLAGS) 
$(serial_mod_CFLAGS) -c -o $@ $<
+
+serial_mod-term_i386_pc_tparm.o: term/tparm.c
+   $(CC) -Iterm -I$(srcdir)/term $(CPPFLAGS) $(CFLAGS) 
$(serial_mod_CFLAGS) -c -o $@ $<
+
+-include serial_mod-term_i386_pc_serial.d
+
+serial_mod_CFLAGS = $(COMMON_CFLAGS)
+
 # For font.mod.
 font_mod_SOURCES = font/manager.c
 CLEANFILES += font.mod mod-font.o mod-font.c pre-font.o 
font_mod-font_manager.o def-font.lst und-font.lst
diff -uNr grub2/conf/i386-pc.rmk grub2.newserial/conf/i386-pc.rmk
--- grub2/conf/i386-pc.rmk  2005-02-19 20:56:06.0 +
+++ grub2.newserial/conf/i386-pc.rmk2005-02-21 12:21:03.387864832 +
@@ -33,7 +33,8 @@
file.h fs.h kernel.h loader.h misc.h mm.h net.h rescue.h symbol.h \
term.h types.h machine/biosdisk.h machine/boot.h \
machine/console.h machine/init.h machine/memory.h \
-   machine/loader.h partition.h pc_partition.h machine/time.h machine/vga.h
+   machine/loader.h partition.h pc_partition.h machine/time.h 
machine/vga.h machine/serial.h
+
 kernel_img_CFLAGS = $(COMMON_CFLAGS)
 kernel_img_ASFLAGS = $(COMMON_ASFLAGS)
 kernel_img_LDFLAGS = -nostdlib -Wl,-N,-Ttext,8200
@@ -187,6 +188,10 @@
 vga_mod_SOURCES = term/i386/pc/vga.c
 vga_mod_CFLAGS = $(COMMON_CFLAGS)
 
+# For serial.mod.
+serial_mod_SOURCES = term/i386/pc/serial.c term/tparm.c term/terminfo.c
+serial_mod_CFLAGS = $(COMMON_CFLAGS)
+
 # For font.mod.
 font_mod_SOUR

build error on ppc

2005-02-21 Thread Nico -telmich- Schottelius
Moin!

I just wanted to compile grub2/cvs, but configure fails:

[10:31] ei:grub2% ./configure --prefix=/usr/packages/grub2 && make
[...]
checking for objcopy... objcopy
checking whether objcopy works for absolute addresses... configure: error: gcc 
cann
ot link at address 2000
[10:31] ei:grub2%

Calling 
[10:49] ei:grub2% autoconf && autoheader  

does not fix that.

After commenting out 

#grub_PROG_OBJCOPY_ABSOLUTE

in configure.ac, autoconf && autoheader, ./configure works fine.

But then make fails:

-- snip --

gcc -o grub-emu grub_emu-kern_main.o grub_emu-kern_device.o 
grub_emu-kern_disk.o grub_emu-kern_dl.o grub_emu-kern_file.o grub_emu-kern_fs.o 
grub_emu-kern_err.o grub_emu-kern_misc.o grub_emu-kern_loader.o 
grub_emu-kern_rescue.o grub_emu-kern_term.o grub_emu-partmap_amiga.o 
grub_emu-partmap_pc.o grub_emu-partmap_apple.o grub_emu-fs_fshelp.o 
grub_emu-util_i386_pc_biosdisk.o grub_emu-fs_fat.o grub_emu-fs_ext2.o 
grub_emu-fs_ufs.o grub_emu-fs_minix.o grub_emu-fs_hfs.o grub_emu-fs_jfs.o 
grub_emu-fs_iso9660.o grub_emu-normal_cmdline.o grub_emu-normal_command.o 
grub_emu-normal_main.o grub_emu-normal_menu.o grub_emu-normal_arg.o 
grub_emu-kern_partition.o grub_emu-util_console.o grub_emu-util_grub_emu.o 
grub_emu-util_misc.o grub_emu-util_i386_pc_getroot.o grub_emu-kern_env.o 
grub_emu-disk_loopback.o grub_emu-commands_ls.o grub_emu-commands_help.o 
grub_emu-commands_terminal.o grub_emu-commands_boot.o grub_emu-commands_cmp.o 
grub_emu-commands_cat.o  -lncurses
grub_emu-normal_menu.o(.text+0x944): In function `grub_menu_run':
normal/menu.c:296: undefined reference to `grub_menu_entry_run'
grub_emu-util_grub_emu.o(.text+0x414): In function `main':
util/grub-emu.c:191: undefined reference to `grub_halt_init'
grub_emu-util_grub_emu.o(.text+0x418):util/grub-emu.c:192: undefined reference 
to `grub_reboot_init'
grub_emu-util_grub_emu.o(.text+0x430):util/grub-emu.c:201: undefined reference 
to `grub_reboot_fini'
grub_emu-util_grub_emu.o(.text+0x434):util/grub-emu.c:202: undefined reference 
to `grub_halt_fini'
collect2: ld returned 1 exit status
make: *** [grub-emu] Error 1


-- snap --

I don't know wherefore grub_PROG_OBJCOPY_ABSOLUTE is needed, if it's only
needed on x86 or whatever.

When those bugs are fixed in cvs, I'll recheck and report.

Nico

-- 
Keep it simple & stupid, use what's available.
Please use pgp encryption: 8D0E 27A4 is my id.
http://nico.schotteli.us | http://linux.schottelius.org


pgp5SmX7WMZOl.pgp
Description: PGP signature
___
Grub-devel mailing list
Grub-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/grub-devel


Fosdem [was: menu entry editor]

2005-02-21 Thread Nico -telmich- Schottelius
marco [Sun, Feb 20, 2005 at 10:18:31PM +0100]:
> "Yoshinori K. Okuji" <[EMAIL PROTECTED]> writes:
> 
> > On Sunday 20 February 2005 18:45, Marco Gerards wrote:
> >> If it is possible, I will try to test this before I go to FOSDEM next
> >> week...
> >
> > BTW, do you know which room is booked for GNU Hurd? I don't find the 
> > information on the web site. I will ask someone when I reach there, but 
> > I am very afraid of losing my way.
> 
> In room AW1.115, see http://www.fosdem.org/2005/index/schedule.  I
> will give a small talk about GRUB 2 there.  It will be mainly a
> technical talk.

Will your speech be available as video for download?

Greetings,

Nico

-- 
Keep it simple & stupid, use what's available.
Please use pgp encryption: 8D0E 27A4 is my id.
http://nico.schotteli.us | http://linux.schottelius.org


pgprvSy3sYPRZ.pgp
Description: PGP signature
___
Grub-devel mailing list
Grub-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/grub-devel