Red Hat Bugzilla – Bug 480483
Review Request: gaupol - Subtitle editor
Last modified: 2009-02-07 17:23:43 EST
Spec URL: http://lucilanga.fedorapeople.org/gaupol.spec
SRPM URL: http://lucilanga.fedorapeople.org/gaupol-0.13.1-1.fc10.src.rpm
Description: Editor for text-based subtitle files. It supports multiple subtitle file formats and provides means of correcting texts and timing subtitles to match
video. The user interface is designed with attention to batch processing of
multiple documents and convenience of translating.
OK source files match upstream: 11c62f478bcd9cd10a88752e4545e1d2
OK package meets naming and versioning guidelines.
OK specfile is properly named, is cleanly written and uses macros consistently.
OK dist tag is present.
OK build root is correct.
OK license field matches the actual license.
OK license is open source-compatible. License text not included upstream.
OK latest version is being packaged.
OK BuildRequires are proper.
OK %clean is present.
OK package builds in mock (Rawhide/x86_64).
OK debuginfo package isn't need.
OK rpmlint is silent.
OK final provides and requires look sane.
OK no shared libraries are added to the regular linker search paths.
OK owns the directories it creates.
OK no duplicates in %files.
OK file permissions are appropriate.
OK no scriptlets present.
OK code, not content.
OK documentation is small, so no -docs subpackage is necessary.
OK %docs are not necessary for the proper functioning of the package.
OK no headers.
OK no pkgconfig files.
OK no libtool .la droppings.
Thank you for the review.
New Package CVS Request
Package Name: gaupol
Short Description: Subtitle editor
Branches: F-9 F-10 EL-5
gaupol-0.13.1-1.fc9 has been submitted as an update for Fedora 9.
gaupol-0.13.1-1.fc10 has been submitted as an update for Fedora 10.
gaupol-0.13.1-1.fc10 has been pushed to the Fedora 10 stable repository. If problems still persist, please make note of it in this bug report.
gaupol-0.13.1-1.fc9 has been pushed to the Fedora 9 stable repository. If problems still persist, please make note of it in this bug report.