setup-storage: Cannot determine size of /dev/cciss/c1d0 - scheme unknown

[Michael Tautschnig]

> On Thu, May 06, 2010 at 01:45:48PM +0200, Mathieu Alorent wrote:
> > Le jeudi 06 mai 2010 à 11:35 +0200, Mathieu Alorent a écrit :
> > > "Cannot determine size of /dev/cciss/c1d0 - scheme unknown"
> > 
> > Found it !
> > 
> > here is the patch... Hope there is no side effects.
> > 
> > --- usr/share/fai/setup-storage/Init.pm.orig	2010-05-06 13:37:32.000000000 +0200
> > +++ usr/share/fai/setup-storage/Init.pm	2010-05-06 13:36:19.000000000 +0200
> > @@ -182,7 +182,7 @@
> >      return (1, "/dev/$1", $2);
> >    }
> >    elsif ($dev =~
> > -    m{^/dev/(cciss/c\dd\d|ida/c\dd\d|rd/c\dd\d|ataraid/d\d|etherd/e\d+\.\d+)p(\d+)?$})
> > +    m{^/dev/(cciss/c\dd\d|ida/c\dd\d|rd/c\dd\d|ataraid/d\d|etherd/e\d+\.\d+)p?(\d+)?$})
> >    {
> >      defined($2) or return (1, "/dev/$1", -1);
> >      return (1, "/dev/$1", $2);
> 
> I suppose the line should read
> 
> > +    m{^/dev/(cciss/c\dd\d|ida/c\dd\d|rd/c\dd\d|ataraid/d\d|etherd/e\d+\.\d+)(p\d+)?$})
> 
> Seems to make more sense?
> 

Sorry for the late and only very brief reply; I should already be on vacation...
I'm replying on this one as it we need the patched line for further discussion.
I guess that the line should read

m{^/dev/(cciss/c\dd\d|ida/c\dd\d|rd/c\dd\d|ataraid/d\d|etherd/e\d+\.\d+)(p(\d+))?$})

and $2 be replaced by $3 in return (1, "/dev/$1", $2);, i.e., return (1, "/dev/$1", $3);

Best,
Michael

PS.: Will be on VAC for about two weeks, so don't expect any further action from
my side until then.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
Url : http://lists.uni-koeln.de/pipermail/linux-fai/attachments/20100508/0368db13/attachment.bin