Don't check on category
Jon Nettleton
jon.nettleton at gmail.com
Tue Nov 8 13:00:07 PST 2005
On Tue, 2005-11-08 at 15:21 -0500, John (J5) Palmieri wrote:
> On Tue, 2005-11-08 at 15:01 -0500, Joe Shaw wrote:
> > Hi,
> >
> > On Tue, 2005-11-08 at 14:48 -0500, John (J5) Palmieri wrote:
> > > A patch was added awhile ago which was a quick fix but not the correct
> > > one:
> > >
> > > 2005-08-31 Danny Kukawka <danny.kukawka at web.de>
> > >
> > > * fdi/information/10freedesktop/10-usb-music-players.fdi:
> > > Removed merge of not useful key info.category=portable_audio_player.
> > > This merge breakes the storage policy and prevent mount of USB storage
> > > players. See bug: https://bugzilla.novell.com/show_bug.cgi?id=113966
> > >
> > > The real fix was to change the storage policy to match on capabilities
> > > and not category. Attached is a patch which does this. As for the
> > > merge of info.category I propose that we nuke this for every device and
> > > only rely on capabilities. It gets confusing and I can see apps in the
> > > future running into a bug like this.
> >
> > Yeah, your patch is more correct.
> >
> > I'm not so sure about removing the merging of info.category, though. I
> > actually think that Danny's patch should be reverted. The idea being
> > that the classic definition of category vs. capability was what the
> > device is vs. what the device does. An audio player should appear as an
> > audio player; apps should deal with them as such and present UIs
> > appropriate to that. The fact that they're also a storage device is a
> > technical detail... That's what it does, not what it is.
>
> If people think it is important I really don't care if we keep the key.
> I'm only pointing out it will be a cause of bugs simply because it is a
> lot easier to check a string than a list and programmers are lazy ;-)
> Besides, isn't what a device is a product of its capabilities? What
> would the iPod Video be listed as? A portable_audio_player;
> video_player; a whole other category? I would suspect UI would be
> geared towards the capabilities of a device and not the category.
>
For classification's sake, my two cents would be they are all portable
media players, with capabilities of audio, video, and usb storage. I
think that pretty much covers everything that is out there. Another
would be a phone category with possibilities of the same capabilities.
Jon
More information about the hal
mailing list