Bug 307821
Summary: | Review Request: sooperlooper - Realtime software looping sampler | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Fernando Lopez-Lezcano <nando> |
Component: | Package Review | Assignee: | Hans de Goede <hdegoede> |
Status: | CLOSED RAWHIDE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | fedora-package-review, hdegoede, kms, kwizart, notting, opensource |
Target Milestone: | --- | Flags: | hdegoede:
fedora-review+
kevin: fedora-cvs+ |
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2008-04-15 18:49:55 UTC | Type: | --- |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: |
Description
Fernando Lopez-Lezcano
2007-09-26 19:14:50 UTC
GPL is not a valid value for the License-Tag, see: http://fedoraproject.org/wiki/Packaging/LicensingGuidelines#head-f21ae23bf2f278444e2c385463cfa74a502396b8 Afaik, these categories in the desktop file should not be used: X-Fedora Application And I guess these, too: X-PlanetCCRMA X-Digital_Processing X-Jack Maybe you want to add this category: Audio (In reply to comment #1) > GPL is not a valid value for the License-Tag, see: > http://fedoraproject.org/wiki/Packaging/LicensingGuidelines#head-f21ae23bf2f278444e2c385463cfa74a502396b8 Thanks, changed to GPLv2+ (updated spec file linked above) > Afaik, these categories in the desktop file should not be used: > X-Fedora Application Do you have a reference by any chance on categories that should / should not be included in desktop files? In my fc6 machine I have 335 application desktop files. Of those that are not from Planet CCRMA origin: 185 have "Application;" in their category 84 have "X-Fedora;" in their category list A similar result happens in fc7. > And I guess these, too: > X-PlanetCCRMA > X-Digital_Processing > X-Jack I'd rather preserve X-Digital_Processing and X-Jack. With a full Planet CCRMA install the "Applications" -> "Sound & Video" is very very long. Planet CCRMA currently includes a separate menu tree (currently "Applications" -> "Planet CCRMA" but could be renamed if it ever becomes part of Fedora, of course) that further categorizes apps with the end result that the menu is actually usable to find and use applications. Leaving those categories in does not affect systems without the extra menu structure but makes the system much more usable in those that have it. > Maybe you want to add this category: > Audio This one does not seem to be very popular either, only 7 entries in my fc6 system (I'm not saying this is not the proper entry, only that it does not seem to be used). Thanks for the feedback. > I'd rather preserve X-Digital_Processing and X-Jack. With a full Planet CCRMA
> install the "Applications" -> "Sound & Video" is very very long. Planet CCRMA
> currently includes a separate menu tree (currently "Applications" -> "Planet
> CCRMA" but could be renamed if it ever becomes part of Fedora, of course) that
> further categorizes apps with the end result that the menu is actually usable to
> find and use applications. Leaving those categories in does not affect systems
> without the extra menu structure but makes the system much more usable in those
> that have it.
I think that we need to consider how the "Sound & Video" menu will be structured
as more Planet CCRMA applications are migrated to Fedora. I have a large number
of applications from Planet CCRMA installed on F7 and the "Sound & Video" menu
is unusable - the Planet CCRMA menu provides a much better structure.
I guess a new bug should be opened for the menu issue - against which component?
The place to look when deciding what to do with the Categories field in a .desktop file is: http://standards.freedesktop.org/menu-spec/latest/apa.html Notice that there are main categories and sub categories. An application must always have atleast one main category, and may have as many subcategories as one one wants. In the case of audio applications, there also is this note in the table: Audio An audio application Desktop entry must include AudioVideo as well So the minimal Categories field for an audio app is: Categories=AudioVideo;Audio; Notice no Application!, Application is not a valid Category. If you want to further give hints for the menus what kind of application this is , the following sub categories are candidates: Midi An app related to MIDI Mixer Just a mixer Sequencer A sequencer Tuner A tuner TV A TV application AudioVideoEditing Application to edit audio/video files Player Application to play audio/video files Recorder Application to record audio/video files If that is not fine grained enough using X-foo Categories is a good solution. As for adding some kinda finer grained menu for Audio stuff, take a look at the games-menus package, that already does that for the Games menu. (In reply to comment #4) > The place to look when deciding what to do with the Categories field in a > .desktop file is: http://standards.freedesktop.org/menu-spec/latest/apa.html Thanks, perfect... In fact, the latest version of one of the Fedora packages in my build environment is actually complaining about "Application" and stopping the build with an error :-) > Notice that there are main categories and sub categories. An application must > always have atleast one main category, and may have as many subcategories as one > one wants. > > In the case of audio applications, there also is this note in the table: > Audio An audio application Desktop entry must include AudioVideo as well > > So the minimal Categories field for an audio app is: > Categories=AudioVideo;Audio; > > Notice no Application!, Application is not a valid Category. > > If you want to further give hints for the menus what kind of application this is > , the following sub categories are candidates: > > Midi An app related to MIDI > Mixer Just a mixer > Sequencer A sequencer > Tuner A tuner > TV A TV application > AudioVideoEditing Application to edit audio/video files > Player Application to play audio/video files > Recorder Application to record audio/video files > > If that is not fine grained enough using X-foo Categories is a good solution. This is what I'm currently using (prepending "X-" to each, "_" is automatically changed to " " for the menu titles). Overlaps with the existing Freedesktop categories noted between square brackets: Audio_Tools Mixers [Mixer] Digital_Processing Drumming DSSI_Plugins Graphics_Apps <I used to use this, not any longer, I think> Jack LADSPA_Plugins Lash MIDI [Midi] Miscellanea Multitrack Notation Players [Player] Programming Recorders [Recorder] Sequencers [Sequencer] Soundcard_Tools Editors Synthesis Trackers Video > As for adding some kinda finer grained menu for Audio stuff, take a look at the > games-menus package, that already does that for the Games menu. That's very good, I'll take a look and see how we could make that structure work for audio/music/midi apps. Do you think something like this would work for sooperlooper?: ---- # desktop file categories BASE="AudioVideo Audio" XTRA="X-Digital_Processing X-Jack X-MIDI Midi" ---- I erased Application, X-Fedora, etc and added Audio and Midi, but kept the X- tags I'm currently using so that the application stays in the proper place for users that have planetccrma-menus already installed. I could change planetccrma-menus to account for the overlap so that eventually something like "X-MIDI" can be dropped from the apps (and from planetccrma-menus or whatever it is called), and "Midi" used instead exclusively. (In reply to comment #5) > Do you think something like this would work for sooperlooper?: > > ---- > # desktop file categories > BASE="AudioVideo Audio" > XTRA="X-Digital_Processing X-Jack X-MIDI Midi" > ---- > > I erased Application, X-Fedora, etc and added Audio and Midi, but kept the X- > tags I'm currently using so that the application stays in the proper place for > users that have planetccrma-menus already installed. I could change > planetccrma-menus to account for the overlap so that eventually something like > "X-MIDI" can be dropped from the apps (and from planetccrma-menus or whatever it > is called), and "Midi" used instead exclusively. Sounds good to me. What is the current status of this bug? Someone, ping? I guess that reviewers are waiting for a new version with the .desktop Catogories fixed and that Fernando is busy and waiting for a full review. I'll try todo a full review as time permits. (In reply to comment #9) > I guess that reviewers are waiting for a new version with the .desktop > Catogories fixed and that Fernando is busy and waiting for a full review. > > I'll try todo a full review as time permits. I'm sorry about the delay, I've been very busy (and this last week rebuilding Planet CCRMA on f8 on top of everything, out soon!). I'll see if I can release a new spec/srpm later today for a full review. (In reply to comment #10) > (In reply to comment #9) > > I guess that reviewers are waiting for a new version with the .desktop > > Catogories fixed and that Fernando is busy and waiting for a full review. > > > > I'll try todo a full review as time permits. > > I'm sorry about the delay, I've been very busy (and this last week rebuilding > Planet CCRMA on f8 on top of everything, out soon!). I'll see if I can release a > new spec/srpm later today for a full review. Spec URL: http://ccrma.stanford.edu/planetccrma/extras/sooperlooper.spec SRPM URL: http://ccrma.stanford.edu/planetccrma/extras/sooperlooper-1.2.0-2.src.rpm Full review done, here are the results: Should Fix: ----------- * adding "-n sooperlooper-%{version}" to %setup archs is superficial / not needed No blockers, approved! ping? What is the status of this bug? (In reply to comment #14) > What is the status of this bug? The spec file has been approved but that was just before I went on vacation and I had no time to learn the new ins and outs of actually submitting the package. I just came back and will do it, hopefully within a week... (In reply to comment #15) > The spec file has been approved but that was just before I went on vacation and > I had no time to learn the new ins and outs of actually submitting the package. > I just came back and will do it, hopefully within a week... I hope your vacation was fun. Please update next time this wiki page to let others know that you are away: http://fedoraproject.org/wiki/Vacation Fernando: ping? All you need todo is ask for cvs creation, import and fire a build. Fernando, any chance you could get around to importing this? I'll do the CVS creation request for you to speed things up: New Package CVS Request ======================= Package Name: sooperlooper Short Description: Realtime software looping sampler Owners: nando, jwrdegoede Branches: F-8 InitialCC: Cvsextras Commits: Yes cvs done. Imported and build, closing. |