[Bug 177134] Review Request: mkvtoolnix - Matroska container utilites

[bugzilla at redhat.com]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: mkvtoolnix - Matroska container utilites


https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=177134


ville.skytta at iki.fi changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|gdk at redhat.com              |ville.skytta at iki.fi
OtherBugsDependingO|163776                      |163778
              nThis|                            |




------- Additional Comments From ville.skytta at iki.fi  2006-04-01 02:23 EST -------
Partial review while waiting for legal comments:

- Change BR wxGTK2-devel to wxGTK-devel; the former is ambiguous with the 
  wxGTK-devel and compat-wxGTK2-devel both providing it.

- Unowned %{_datadir}/%{name} directory

- (rpmlint:) Group tag for -gui should be changed to Applications/Multimedia

- (rpmlint:) too long lines in %description, word-wrap at < 80 chars

- FLAC support missing, adding BR: flac-devel >= 1.1.1 fixes it

- Why disable bzip2 by default?  Even rpm itself requires bzip2-libs so it won't
  add any dependency bloat.  The same goes kind of with lzo in the sense that
  some popular multimedia packages which are somewhat likely to be installed on
  systems where this is require it, and it doesn't add any other deps, so I
  think it wouldn't hurt.  If there's a good use case for having these disabled
  by default (or in the first place), please add a comment explaining it in
  the specfile.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.