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

Ralf Corsepius <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[email protected]



--- Comment #5 from Ralf Corsepius <[email protected]> ---
Some remarks:

I personally hate reviews, which are rushed through in such a haste, they don't
give anybody a chance to look into a packages.

In this case, it was LESS THAN AN HOUR, which, to me, excludes any possibility
of a careful review. Simply running rpmlint or fedora-review also is not a
careful review!


Back to this  package:

* This section from the spec is bogus and should be needs to be removed:
# compress man-pages
pushd man3
gzip -9 *
popd

* Any proof for this package being BSD-licensed? The license file contained in
this package is added by Björn, but I can't find any upstream evidence linpack
is actually licensed BSD. Googling also did not provide further information.

* I prefer packages to install their docs into /usr/share/doc/<package>.
Installing docs into <package>-doc seems quite nonsensical to me.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=pPNVHHNik4&a=cc_unsubscribe
_______________________________________________
package-review mailing list
[email protected]
https://admin.fedoraproject.org/mailman/listinfo/package-review

Reply via email to