Re: [PATCH 7/8] Add placeholder for DASD value reduction method.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 10/10/2009 12:27 AM, David Cantrell wrote:
> ---
>  storage/devices.py |   12 +++++++++++-
>  1 files changed, 11 insertions(+), 1 deletions(-)
> 
> diff --git a/storage/devices.py b/storage/devices.py
> index ad936ea..d480dc5 100644
> --- a/storage/devices.py
> +++ b/storage/devices.py
> @@ -3160,7 +3160,10 @@ class DASDDevice(DiskDevice):
>          DiskDevice.__init__(self, *args, **kwargs)
> 
>      def dracutSetupData(self):
> -        arg = {'option': 'dasd', 'combine': True, 'delimiter': ','}
> +        arg = {'option': 'dasd',
> +               'combine': True,
> +               'delimiter': ',',
> +               'filter': self.filterDASDValues}
>          subargs = []
> 
>          if self.readonly:
> @@ -3179,6 +3182,13 @@ class DASDDevice(DiskDevice):
> 
>          return [arg]
> 
> +    def filterDASDValues(self, values=[]):
> +        # XXX: will reduce a list of dasd= specs from:
> +        #     dasd=0.0.0200,0.0.0201,0.0.0202,0.0.0203,0.0.0204
> +        # to:
> +        #     dasd=0.0.0200-0.0.0204
> +        return values
> +

I really hope, we won't need that. My explanation is going to follow in
a reply to your patch intro mail.

> 
>  class NFSDevice(StorageDevice, NetworkStorageDevice):
>      """ An NFS device """

Steffen

Linux on System z Development

IBM Deutschland Research & Development GmbH
Vorsitzender des Aufsichtsrats: Martin Jetter
Geschäftsführung: Erich Baier
Sitz der Gesellschaft: Böblingen
Registergericht: Amtsgericht Stuttgart, HRB 243294


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

[Index of Archives]     [Kickstart]     [Fedora Users]     [Fedora Legacy List]     [Fedora Maintainers]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [Yosemite Photos]     [KDE Users]     [Fedora Tools]
  Powered by Linux