[packagekit] Yum backend RequireReboot

Richard Hughes hughsient at gmail.com
Wed Oct 3 12:30:44 PDT 2007


On Wed, 2007-10-03 at 13:44 -0400, David Zeuthen wrote:
> On Wed, 2007-10-03 at 18:14 +0100, Richard Hughes wrote:
> > Yes, we need this for GetUpdateDetail:
> > 
> >     <signal name="UpdateDetail">
> >       <arg type="s" name="tid" direction="out"/>
> >       <arg type="s" name="package_id" direction="out"/>
> >       <arg type="s" name="updates" direction="out"/>
> >       <arg type="s" name="obsoletes" direction="out"/>
> >       <arg type="s" name="url" direction="out"/>
> >       <arg type="s" name="restart" direction="out"/>
>                   ^^
> Should probably s/s/b/ and s/restart/requires_restart/ here.

Nope, this is an enum that can be "session" for a logout-login and
"system" for a shutdown-restart.

> >       <arg type="s" name="update_text" direction="out"/>
> >     </signal>
> > 
> > We can extend this as required. Yell if you want to add anything.
> 
> This is hard to extend as it would require changing the signature for
> the UpdateDetail() signal and thus break the ABI. 

Sure, ABI isn't a big concern right now, in a few weeks/months it will
be.

> You probably want to use key/value pairs (basically a dict: string ->
> any) for optional stuff (e.g. no need to stuff 'tid', 'updates',
> 'obsoletes' etc. into the dict).

Yes, a very good plan. I'll look into this tomorrow.

Richard.





More information about the PackageKit mailing list