[Bug 705363] Review Request: spacewalk-web - Spacewalk Web site packages

[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.


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

--- Comment #3 from Miroslav Suchý <msuchy@xxxxxxxxxx> 2011-07-29 07:26:12 EDT ---
>TODO: I think `Spacewalk web site' or `Spacewalk web interface' would be
>enough. The word (Spacewalk) `packages' interfere with RPM packages.

fixed

>TODO: spacewalk-html subpackage contains MIT licensed code

fixed

> TODO: spacewalk-base-minimal summary is too cryptic. Replace ".pm's" with `Perl
modules' or make it more human-friendly in other way.

fixed

> TODO: spacewalk-web package summary contains lower-case `rpm' abbreviation. Use
upper case.

fixed

> TODO: Some package descriptions do not end with full stop.

fixed

> TODO: Remove BuildRoot definition and it's cleaning as it's default behavior of
rpmbuild.

Negative sir. We still care about EPEL4, where we need it.

>FIX: Add Requires: perl(:MODULE_COMPAT_%(eval "`%{__perl} -V:version`"; echo
$version)) too all subpackages with Perl modules

fixed

> TODO: Add %{?_smp_mflags} to make arguments to utilize all CPU's while
`compiling'.

fixed

> TODO: Remove %defattr macro from %files sections as this is implicit.

fixed

>FIX: Escape percentage symbol in changelog, otherwise it's subject of
SPEC-macro expansion.

fixed 

> Notice: Is /var/www/html/network/software/channels/keys/BETA-RPM-GPG-KEY a GPG
key stored along web pages? Should it be a %config file? I don't like it.

Dead files. Removed.

>  Also your Perl modules contain tests and other Perl infrastructure that is not utilized by your spec file.

That is because that those tests are not maintained for ages and very probably
do not work.

-- 
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.
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review



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