FAQ Search Today's Posts Mark Forums Read
» Video Reviews

» Linux Archive

Linux-archive is a website aiming to archive linux email lists and to make them easily accessible for linux users/developers.


» Sponsor

» Partners

» Sponsor

Go Back   Linux Archive > Redhat > Fedora User

 
 
LinkBack Thread Tools
 
Old 02-09-2010, 11:13 AM
Martin Sivak
 
Default Fix backtrace when trying to use LV for /boot (#562325)

Looks OK I think.

Martin

----- "Hans de Goede" <hdegoede@redhat.com> wrote:

> ---
> platform.py | 21 ++++++++++++++-------
> 1 files changed, 14 insertions(+), 7 deletions(-)
>
> diff --git a/platform.py b/platform.py
> index f3154d4..94b85d9 100644
> --- a/platform.py
> +++ b/platform.py
> @@ -229,13 +229,20 @@ class EFI(Platform):
> if req.format.type != "efi":
> errors.append(_("/boot/efi is not EFI."))
>
> - disk = req.disk.format.partedDisk
> + # Don't try to check the disklabel on lv's etc, using lv for
> /boot
> + # is already checked in the generic
> Platform.checkBootRequest()
> + if req.type == "partition":
> + partitions = [ req ]
> + elif req.type == "mdarray":
> + partitions = filter(lambda d: d.type == "partition",
> req.parents)
>
> # Check that we've got a correct disk label.
> - labelType = self.diskLabelType(disk.device.type)
> - if disk.type != labelType:
> - errors.append(_("%s must have a %s disk label.")
> - % (req.disk.name, labelType.upper()))
> + for p in partitions:
> + partedDisk = p.disk.format.partedDisk
> + labelType = self.diskLabelType(partedDisk.device.type)
> + if partedDisk.type != labelType:
> + errors.append(_("%s must have a %s disk label.")
> + % (p.disk.name, labelType.upper()))
>
> return errors
>
> @@ -265,7 +272,7 @@ class Alpha(Platform):
> def checkBootRequest(self, req):
> errors = Platform.checkBootRequest(self, req)
>
> - if not req or not req.disk:
> + if not req or req.type != "partition" or not req.disk:
> return errors
>
> disk = req.disk.format.partedDisk
> @@ -408,7 +415,7 @@ class NewWorldPPC(PPC):
> def checkBootRequest(self, req):
> errors = PPC.checkBootRequest(self, req)
>
> - if not req or not req.disk:
> + if not req or req.type != "partition" or not req.disk:
> return errors
>
> disk = req.disk.format.partedDisk
> --
> 1.6.6
>
> _______________________________________________
> Anaconda-devel-list mailing list
> Anaconda-devel-list@redhat.com
> https://www.redhat.com/mailman/listinfo/anaconda-devel-list

_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@redhat.com
https://www.redhat.com/mailman/listinfo/anaconda-devel-list
 

Thread Tools




All times are GMT. The time now is 08:53 PM.

VBulletin, Copyright ©2000 - 2014, Jelsoft Enterprises Ltd.
Content Relevant URLs by vBSEO ©2007, Crawlability, Inc.
Copyright 2007 - 2008, www.linux-archive.org