bugGNU GRUB - Bugs: bug #9628, convert() says (hd0,)

 
 

You are not allowed to post comments on this tracker with your current authentication level.

bug #9628: convert() says (hd0,)

Submitter:  None
Submitted:  Tue 13 Jul 2004 11:45:29 AM UTC
   
 
Category:  Disk & Partition Severity:  Major
Priority:  5 - Normal Item Group:  Software Error
Status:  Fixed Privacy:  Public
Assigned to:  okuji Originator Name: 
Originator Email:  -email is unavailable- Open/Closed:  Closed
Release:  None Release:  0.95
Reproducibility:  Every Time Planned Release:  None

Fri 23 Feb 2007 07:55:56 AM UTC, comment #2: 

I want to reopen this bug for grub-0.97.

I got the same error as in the original submission using this command-line:

grub-install /dev/cciss/c0d0

I've included a patch to fix this problem. The patch was done using grub-install 1.49 from here: http://cvs.savannah.gnu.org/viewcvs/*checkout*/grub/util/grub-install.in?rev=1.49&root=grub

Best regards,

Carsten

(file #12029)

Carsten Dumke <cdumke>
Fri 16 Jul 2004 11:37:38 AM UTC, comment #1: 

I fixed it in the CVS.

Yoshinori K. Okuji <okuji>
Group administrator
Tue 13 Jul 2004 11:45:29 AM UTC, original submission:  

The convert() function inside grub-install, if its argument
is e.g. /dev/hda, prints a warning:
expr: non-numeric argument
and returns the answer "(hd0,)"
In 0.94 it correctly returned "(hd0)" without any warning.

The long sed commands which calculate tmp_disk and tmp_part
inside convert() have introduced a new line with d[0-9]*p...
This one seems to be buggy. Currently
tmp_part=.....
     -e 's%.*d[0-9]*p*%%'...
I'm not sure but I think this should be
tmp_part=.....
     -e 's%.*d[0-9]*p[0-9]*%%'...

Anonymous

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attached Files
file #12029:  grub-install-fix-tmp_part.diff added by cdumke (440B - application/octet-stream - patch for this bug)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by cdumke (Updated the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

     

    Follow 6 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2007-02-23 cdumke Attached File- Added grub-install-fix-tmp_part.diff, #12029
    2004-07-16 okuji StatusNone Fixed
        Assigned toNone okuji
        Open/ClosedOpen Closed
        Planned Release 0.96
    2004-07-13 egmont Carbon-Copy- Added egmont

    Back to the top

    Powered by Savane 3.13-4448.
    Corresponding source code