Danek Duvall wrote:

> Other suggestions are welcome.

Bart is Smart:

    --- a/src/modules/client/actuator.py
    +++ b/src/modules/client/actuator.py
    @@ -115,11 +115,17 @@ class Actuator(GenericActuator):
                     self.cmd_path = ""
     
             def __str__(self):
    +                def check_val(dfmri):
    +                        if callable(dfmri):
    +                                return [ "true" ]
    +                        else:
    +                                return dfmri
    +
                     return "\n".join([
                         "  %16s: %s" % (fmri, smf)
                         for d in [self.removal, self.update, self.install]
                         for fmri in d
    -                    for smf in d[fmri]
    +                    for smf in check_val(d[fmri])
                         ])
     
             def reboot_needed(self):

without the rest of the junk.  Given that insight, I'm also changing the
specific search for the "activate-drivers" actuator to a search for all the
actuators which have callables as their values, and executing those.

The webrev is updated in place:

    http://cr.opensolaris.org/~dduvall/pkg-add_drv-u/

Thanks,
Danek
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss

Reply via email to