[PATCH] (pxe:server_ip:gateway_ip) parsing wrong?

2010-06-07 Thread Colin Watson
While writing documentation for booting GRUB over the network, I noticed that the code to handle (pxe:server_ip:gateway_ip) device names looks wrong. Shouldn't it be like this? 2010-06-07 Colin Watson * fs/i386/pc/pxe.c (grub_pxe_open): Fix parsing of gateway_ip. === modified file 'f

Re: [PATCH] (pxe:server_ip:gateway_ip) parsing wrong?

2010-06-07 Thread Colin Watson
Oh, and speaking of PXE support, what's the point of the pxe_unload command? As near as I can tell, it seems to be the equivalent of 'rmmod pxe'. Is there some reason one might want to run this if pxe were built into the core image, or is it cruft that we should remove? -- Colin Watson

Re: [PATCH] (pxe:server_ip:gateway_ip) parsing wrong?

2010-06-07 Thread Vladimir 'φ-coder/phcoder' Serbinenko
On 06/07/2010 06:58 PM, Colin Watson wrote: > While writing documentation for booting GRUB over the network, I noticed > that the code to handle (pxe:server_ip:gateway_ip) device names looks > wrong. Shouldn't it be like this? > > Go ahead. > 2010-06-07 Colin Watson > > * fs/i386/pc/px

Re: Vendor power-on button

2010-06-07 Thread Vladimir 'φ-coder/phcoder' Serbinenko
On 06/07/2010 02:15 AM, Henrique Ferreiro wrote: > Hi! > > I have an Asus EeePC 1005PE. If I did it correctly the address would > be 84:1. I am attaching the two files so you can check. > It looks correct. Did you try using GRUB_DEFAULT_BUTTON ? > Regards, > > Henrique. > __

Re: Which partitioning schemes should be supported by GRUB?

2010-06-07 Thread Vladimir 'φ-coder/phcoder' Serbinenko
On 06/06/2010 08:02 PM, Grégoire Sutre wrote: > Hi, > > Tests of GRUB on NetBSD (and FreeBSD) have raised several issues (most > of them reported on the list) regarding partition detection. However, > I have the feeling that some of these issues are not considered as real > issues since the test c

Re: [Patch] Use absolute offset for embedding area in grub-setup (i386-pc)

2010-06-07 Thread Vladimir 'φ-coder/phcoder' Serbinenko
On 06/05/2010 02:22 AM, Grégoire Sutre wrote: > Hi, > > The attached patch uses grub_partition_get_start (p) instead of p->start > in grub-setup's iteration over partitions. While we are at it, also > use grub_partition_get_len (p) instead of p->len. Go ahead > > An alternative would be to assume

Re: [PATCH] Complete 'set root=('

2010-06-07 Thread Vladimir 'φ-coder/phcoder' Serbinenko
Go ahead. On 06/03/2010 02:55 PM, Colin Watson wrote: > While reviewing the Texinfo documentation for references to GRUB Legacy > constructs that needed to be updated, I found some discussion of > completion that used "root (" as an example. For GRUB 2, this > would need to become "set root=(" - b

Re: [PATCH] Fix verbose error output when device-mapper not supported by kernel

2010-06-07 Thread Vladimir 'φ-coder/phcoder' Serbinenko
On 06/03/2010 06:23 PM, Colin Watson wrote: > Following the merge of my dmraid-probe branch, several people have > reported extremely verbose error output in the event that the kernel > doesn't have device-mapper support (perhaps because the module isn't > loaded; it's modular in the stock Debian k

Re: [PATCH] (pxe:server_ip:gateway_ip) parsing wrong?

2010-06-07 Thread Seth Goldberg
Quoting Colin Watson, who wrote the following on Mon, 7 Jun 2010: Oh, and speaking of PXE support, what's the point of the pxe_unload command? As near as I can tell, it seems to be the equivalent of 'rmmod pxe'. Is there some reason one might want to run this if pxe were built into the core

Web site redesign

2010-06-07 Thread Colin Watson
Hi Bruce, I just committed a couple of typo fixes to the website, and suddenly the redesign you committed back in February went live, rather to my surprise when the page I was editing vanished. I guess the GNU admins fixed the commit trigger! Now people are reporting some dead links. I fixed up

Re: [PATCH] File name transformations (split into 7 diffs)

2010-06-07 Thread Vladimir 'φ-coder/phcoder' Serbinenko
Regarding file-name-transformations_1.diff: phco...@debian.bg45.phnet:~$ /usr/local/bin/grub-mkfont --help Usage: /usr/local/bin/grub-mkfont [OPTIONS] FONT_FILES So I would prefer not to use basename for uniformity. Otherwise patch 1 is ok. On 06/04/2010 01:42 AM, Grégoire Sutre wrote: > Hi, > >

Re: [Patch] Unifont search in configure.ac

2010-06-07 Thread Vladimir 'φ-coder/phcoder' Serbinenko
Go ahead On 05/22/2010 03:17 PM, Grégoire Sutre wrote: > Hi, > > With the attached patch, unifont is searched by extension first (pcf > being the preferred one), and then by directory. > > The first directory being searched is `.', which allows non-root users > to install ascii.pf2 and unifont.pf2

Re: [PATCH] Fix verbose error output when device-mapper not supported by kernel

2010-06-07 Thread Colin Watson
On Mon, Jun 07, 2010 at 10:57:27PM +0200, Vladimir 'φ-coder/phcoder' Serbinenko wrote: > On 06/03/2010 06:23 PM, Colin Watson wrote: > > Following the merge of my dmraid-probe branch, several people have > > reported extremely verbose error output in the event that the kernel > > doesn't have devi

Re: [PATCH] grub-emu-lite tool instead of --enable-grub-emu-modules

2010-06-07 Thread Vladimir 'φ-coder/phcoder' Serbinenko
On 06/02/2010 11:29 AM, BVK Chaitanya wrote: > Updated patch with review comments (on IRC) is attached. > > go ahead > > > > > > ___ > Grub-devel mailing list > Grub-devel@gnu.org > http://lists.gnu.org/mailman/listinfo/grub-devel > -- Regards

Re: [PATCH] File name transformations (split into 7 diffs)

2010-06-07 Thread Grégoire Sutre
On 06/07/2010 11:16 PM, Vladimir 'φ-coder/phcoder' Serbinenko wrote: Regarding file-name-transformations_1.diff: phco...@debian.bg45.phnet:~$ /usr/local/bin/grub-mkfont --help Usage: /usr/local/bin/grub-mkfont [OPTIONS] FONT_FILES So I would prefer not to use basename for uniformity. Otherwise p

Re: New build system for GRUB

2010-06-07 Thread BVK Chaitanya
It seems my mail couldn't be delivered due to patch size. Please look at branches/bvk/automake for this patch. On Tue, Jun 8, 2010 at 10:28 AM, BVK Chaitanya wrote: > Hi > > > Attached is the patch for autogen+automake based build system for > GRUB.  It is building successfully on freebsd, cygw