[PATCH 08/10] DNFPayload: initial version.

Chris Lumens clumens at redhat.com
Tue Aug 20 15:33:26 UTC 2013


> +try:
> +    import dnf
> +    import dnf.exceptions
> +    import dnf.output
> +    import rpm
> +except ImportError as e:
> +    log.error("dnfpayload: component import failed: %s" % e)
> +    dnf = rpm = None

Just a stylistic thing - I don't think we do anything that looks like
this anywhere else in anaconda.

> +    def _bump_tx_id(self):
> +        cur = self.txID
> +        self.txID = 1 if cur is None else cur + 1
> +        return self.txID

Same here.  I'm surprised Brian hasn't said something yet.

- Chris


More information about the anaconda-patches mailing list