[Bug 244623] Review Request: OggConvert - OggConvert is a small GNOME utility that uses GStreamer to convert media files to the licence-free Theora and Vorbis formats.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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: OggConvert - OggConvert is a small GNOME utility that uses GStreamer to convert media files to the licence-free Theora and Vorbis formats.


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





------- Additional Comments From bdpepple@xxxxxxxxxxxxx  2007-06-27 15:32 EST -------
MD5Sums:
a825bf00977e3410f9009198dbcc1fbb  oggconvert-0.2.0.tar.gz

Good:
* Source URL is canonical
* Upstream source tarball verified
* Package name conforms to the Fedora Naming Guidelines
* Group Tag is from the official list
* Buildroot has all required elements
* All paths begin with macros
* All necessary BuildRequires listed.
* All desired features are enabled

Bad
* Unnecessary Requires on python. In general you don't need to require python
since python packages require the python(abi).

* Directory ownership is incorrect for '%{python_sitelib}/OggConvert'.  you
should use '%{python_sitelib}/OggConvert/', instead of
'%{python_sitelib}/OggConvert/*' to correct this.

* Desktop file does not have the vendor set to 'fedora'.  Also the desktop file
contains the depreciated category 'Application'.  You need to add
'--remove-category=Application' to your desktop-file-install call to fix this.

* rpmlint produces the following errors:
E: oggconvert non-executable-script
/usr/lib/python2.5/site-packages/OggConvert/ocv_info.py 0644
E: oggconvert non-executable-script
/usr/lib/python2.5/site-packages/OggConvert/__init__.py 0644
E: oggconvert non-executable-script
/usr/lib/python2.5/site-packages/OggConvert/ocv_util.py 0644
E: oggconvert non-executable-script
/usr/lib/python2.5/site-packages/OggConvert/ocv_gtk.py 0644
E: oggconvert non-executable-script
/usr/lib/python2.5/site-packages/OggConvert/ocv_gst.py 0644
E: oggconvert explicit-lib-dependency libglade2
E: oggconvert description-line-too-long OggConvert is a small GNOME utility that
uses GStreamer to convert media files to the patent-free Theora and Vorbis
formats. It's painfully easy to use: drag a file onto the source bar (or use the
file chooser) and hit convert. Of course, you can also change the quality
settings and the output filename if you like. It uses GStreamer, so it can
convert (almost) any file which Totem can play. It can deal with audio-only
files, video-only files, and files with many audio tracks (such as DVD rips with
a commentary track). Thanks to the magic of GStreamer, metadata (for example,
title and artist info on an MP3) is preserved. Finally, it adheres to the GNOME
HIG as much as possible.

* It builds in Mock, but there are some compiling errors that should be looked
at for example:
<snip>
***MEMORY-WARNING***: [2744]: GSlice: g_thread_init() must be called before all
other GLib functions; memory corruption due to late invocation of
g_thread_init() has been detected; this program is likely to crash, leak or
unexpectedly abort soon...
</snip>
I'll attach my build log for you to go through, and look at.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.

_______________________________________________
Fedora-package-review mailing list
Fedora-package-review@xxxxxxxxxx
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]