1
edit
Changes
no edit summary
GST-OPENMAX 0.2
My package was reviewed by Ryan Lawrance at https://bugzilla.redhat.com/show_bug.cgi?id=879881
and based on the errors found I had to make certain changes.
SRC RPM: https://docs.google.com/open?id=0B0THh1GbRWnAV3RJN3hKbEJLTDQ
BAD: The package must be named according to the Package Naming Guideline.
FIXED: changed the name to gst-openmax-0.10.1
BAD: The package must meet the Packaging Guidelines .
1. summary does not provide a concise description of the package
2. changelog tag specified twice
3. source tag not pointing to upstream source
4. upstream documentation missing
5. files tag points to a specific libdir
6. doc tag does not include all documentation in build
FIXED:
1. Chaged the summary
2. Only one changelog tag is being used
3. Source tag points to the proper source
4. Added documentation in the %doc
5. changed to the macro
6. Added all the documentation to the build
BAD: If (and only if) the source package includes the text of the license(s) in its own file, then that file, containing the text of the license(s) for the package must be included in %doc.
• Not included in doc tag
FIXED: added all the files
BAD: The sources used to build the package must match the upstream source, as provided in the spec URL. Reviewers should use sha256sum for this task as it is used by the sources file once imported into git. If no upstream URL can be specified for this package, please see the Source URL Guidelines for how to deal with this.
• Source does not use an upstream URL
FIXED: Changed to upstream URL
BAD: Static libraries must be in a -static package.
• Package only provides libraries
COMMENT: it is not a static library, but a gstreamer plugin
BAD: If the source package does not include license text(s) as a separate file from upstream, the packager SHOULD query upstream to include it
? Not included in spec file
FIXED: Added COPYING file to the %doc
BAD: The description and summary sections in the package spec file should contain translations for supported Non-English languages, if available.
• summary does not provide a concise description of the package
FIXED: Changed the summary
BAD: The reviewer should test that the package functions as described. A package should not segfault instead of running, for example.
• Unable to list codecs per documentation instructions
FIXED: Added aditional file called README.fedora with proper instructions
GST-OPENMAX 0.1
