Bug#581511: also needed for cups-pk-helper support

[Guido Günther]

On Wed, Jul 28, 2010 at 01:34:18PM +0200, Josselin Mouette wrote:
> Le mercredi 28 juillet 2010 à 12:32 +0200, Guido Günther a écrit :
> > I've updated the package at:
> > 
> >     http://git.debian.org/?p=users/agx/system-config-printer.git;a=summary
> > 
> > and it works here. These are the patches I'm unsure what to do with:
> > 
> > - 04_jobsheets_traceback.patch 
> >   I'm not seeing any pronlems with jobsheets here so I'd rather drop
> >   that one for now.
> 
> It was introduced to fix a crash. Most likely upstream fixed it in a
> different way.
O.k. Let's hope so for now. We do have the patch in the VCS should this
crop up again.

> 
> > - 08_stock_icons.patch 
> >   I'm unclear what's the intention of this one
> 
> It’s an UI fix; the original software uses the wrong icon. I don’t know
> whether this is fixed, but it probably needs to be done in the .ui now.
Fixed differently upstream.

> 
> > - 17_new_printer_notification.patch 
> >   Same for this one
> 
> It’s a fix for the D-Bus permissions, allowing the user’s
> system-config-printer-applet to listen to the signal.
The applet gets all the status information needed (i.e. reports when the
printer got installed, etc.) so I assume this if not needed anymore.

> 
> > - 18_debian_packages.patch 
> >   Partially merged into 0003-Adjust-package-names.patch, needs more work
> >   to get package installations working again.
> 
> If upstream uses packagekit now, it should be possible (after fixing the
> package names of course) by depending on sessioninstaller.
Great. Didn't know about sessioninstaller, with that we only need a
gpk-install-package-name wrapper which is simple.

> 
> > Could you comment on them?  I'd be happy to do an upload to
> > experimental
> > so more people can test this easily.
> 
> Looks like the way to go.
O.k. I can do an upload after fixing up some more minor things if there
are no objections.
 -- Guido