[PATCH 3/9] Use parents consistently in Device constructors.

Brian C. Lane bcl at redhat.com
Wed Jul 25 23:46:43 UTC 2012


On Wed, Jul 25, 2012 at 04:48:28PM -0500, David Lehman wrote:
>      def newLV(self, *args, **kwargs):
>          """ Return a new LVMLogicalVolumeDevice instance. """
> -        if kwargs.has_key("vg"):
> -            vg = kwargs.pop("vg")
> -
> +        vg = kwargs.get("parents")[0]

Should probably protect against parents not being set.

-- 
Brian C. Lane | Anaconda Team | IRC: bcl #anaconda | Port Orchard, WA (PST8PDT)
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 482 bytes
Desc: not available
URL: <https://fedorahosted.org/pipermail/anaconda-patches/attachments/20120725/c8d68772/attachment.sig>


More information about the anaconda-patches mailing list