[Merge] lp:~stgraber/indicator-power/revert-20130913 into lp:indicator-power

[Steve Langasek]

On Sat, Sep 14, 2013 at 01:43:00PM -0600, Adam Conrad wrote:
> On Sat, Sep 14, 2013 at 08:58:49AM -0500, Ted Gould wrote:

> > I understand your concern about bringing in additional packages onto the
> > desktop image.  What I don't agree with is removing the feature from
> > indicator-power instead of figuring out the issues with all of those
> > packages being brought onto the image.  The issue is that
> > liburl-dispatcher1 recommends url-dispatcher, as is customary on
> > libraries that implement the interface of a service.

> Is this actually "customary"?  In what circles?  I've actually fought
> pretty hard in the past to *not* have libraries depend on external
> services, daemons, and random binaries, because linking to a library
> doesn't necessarily mean every user of your application wants to use
> every last plugin/service/etc available to it.

> I'd think a Suggests is perfectly reasonable, and explicitly seeding
> in tasks (touch, perhaps eventually desktop) where you want that bit
> to do something other than be a dormant library dependency.

I agree.  I don't think there's anything "customary" about libraries
recommending the service they interface with; I think that gives buggy
semantics.  If the service is not available, the library should fail
gracefully anyway, so if you want to ensure the service is present that
should be handled with a higher-level dependency, *not* via a dependency
from the library.

-- 
Steve Langasek                   Give me a lever long enough and a Free OS
Debian Developer                   to set it on, and I can move the world.
Ubuntu Developer                                    http://www.debian.org/
slangasek at ubuntu.com                                     vorlon at debian.org
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <https://lists.ubuntu.com/archives/ubuntu-release/attachments/20130914/f77d218d/attachment.pgp>