[Bug 481508] Review Request: pipviewer - Visualizer for multiple alignments of genomic sequences

bugzilla at redhat.com bugzilla at redhat.com
Tue Jan 27 08:26:20 UTC 2009


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


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


Marcela Maslanova <mmaslano at redhat.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
         AssignedTo|nobody at fedoraproject.org    |mmaslano at redhat.com




--- Comment #1 from Marcela Maslanova <mmaslano at redhat.com>  2009-01-27 03:26:18 EDT ---
OK source files match upstream: 6da4d203440470d2709f3894e648b4aa
OK package meets naming and versioning guidelines.
FAIL specfile is properly named, is cleanly written and uses macros 
consistently.
- RPM_BUILD_ROOT vs. buildroot - use only one of these macros
OK dist tag is present.
OK build root is correct.
FAIL license field matches the actual license.
- You can find GPLv3+ in COPYING and PKG-INFO
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.

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




More information about the Fedora-package-review mailing list