Bug 479515 - Review Request: Music Organizer - Music Organizer , organize your mp3, ogg, flac and mp4
Summary: Review Request: Music Organizer - Music Organizer , organize your mp3, ogg, f...
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
low
medium
Target Milestone: ---
Assignee: Nobody's working on this, feel free to take it
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: FE-DEADREVIEW
TreeView+ depends on / blocked
 
Reported: 2009-01-10 12:45 UTC by Camille GALLET
Modified: 2009-09-29 08:44 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2009-09-29 08:44:36 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Camille GALLET 2009-01-10 12:45:01 UTC
Spec URL: ftp://downloads.tuxfamily.org/morg/files/fedora/Music_Organizer-2.5.1-1.spec
SRPM URL: ftp://downloads.tuxfamily.org/morg/files/fedora/Music_Organizer-2.5.1-1-2.5.1-1.fc9.src.rpm
Description: Music Organizer is a multi-platform program that manage mp3, ogg, flac and mp4 files. For mp3, it recogonize both id3V2 and id3V1 tags and edit them separatly. It can also organize your files i.e. create a tree from tags and puts them smartly into those. musicorganizer generate tags from files name, generate files name from the tags and let you listen your music whith your favourite player.Music Organizer use taglib to read tags and wxWidgets for the GUI. Website: http://www.musicorganizer.org

Comment 1 Fabian Affolter 2009-01-10 17:47:56 UTC
Just some comment on your spec file

- The name of the spec file don't match guidelines
  https://fedoraproject.org/wiki/Packaging/NamingGuidelines#Spec_file_name
- The summary needs to be improved.  It's not necessary to repeat the package name.
- The license is wrong, check source header -> GPLv2+
- Your description looks very ugly. 
- Please preserve the time stamps in the install section if it's possible
  make DESTDIR=%{buildroot} INSTALL="install -p" install
- There are translations...
  https://fedoraproject.org/wiki/Packaging/Guidelines#Handling_Locale_Files
- AUTHORS, COPYING, and so on must be include as %doc.
  https://fedoraproject.org/wiki/PackageMaintainers/CreatingPackageHowTo#.25files_prefixes
- You must install /verify the .desktop file
  https://fedoraproject.org/wiki/Packaging/Guidelines#Desktop_files
- You must use macros in the %files section
  https://fedoraproject.org/wiki/PackageMaintainers/CreatingPackageHowTo#Macros
- Stuff from the doc directory in the source should marked as %doc in the package.
- You should add a proper changelog entry.
  https://fedoraproject.org/wiki/Packaging/Guidelines#Changelogs

From my point of view this package should be named as 'musicorganizer'. The Mac version is called 'MusicOrganizer', the Debian version 'musicorganizer', and the Windows package 'Music.Organizer'.

Comment 2 Fabian Affolter 2009-01-27 12:18:23 UTC
Any progress on this?

Comment 3 Fabian Affolter 2009-04-08 05:58:46 UTC
Is this review dead?

Comment 4 Camille GALLET 2009-04-14 16:43:28 UTC
It's not dead I just didn't have any time to deal with this the last few month, I will try to find time do the modifications in the next weeks ( April or May more exactly).

Comment 5 Fabian Affolter 2009-06-16 10:15:04 UTC
and?

Comment 6 Fabian Affolter 2009-07-27 11:25:46 UTC
Camille, are you still interested to maintain this package in Fedora?

Comment 7 Fabian Affolter 2009-09-18 07:23:22 UTC
I will mark this as dead in some days.


Note You need to log in before you can comment on or make changes to this bug.