[Bug 592672] Review Request: hct - A HDL complexity tool

[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=592672

Petr Pisar <ppisar@xxxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED

--- Comment #2 from Petr Pisar <ppisar@xxxxxxxxxx> 2010-06-15 10:57:38 EDT ---
$ rpmlint -i hct.spec ../RPMS/noarch/hct-0.7.60-1.fc13.noarch.rpm 
hct.noarch: E: useless-provides perl(HCT::Std::IO)
This package provides 2 times the same capacity. It should only provide it
once.

hct.noarch: W: no-manual-page-for-binary hct.pl
Each executable in standard binary directories should have a man page.

1 packages and 1 specfiles checked; 1 errors, 1 warnings.

The error is because of double package HCT definition:

$ grep -Hnr 'package HCT::Std::IO' lib/
lib/HCT/Std.pm:45:package HCT::Std::IO;
lib/HCT/Std/IO.pm:19:package HCT::Std::IO;
lib/HCT/Std/IO.pm:60:package HCT::Std::IO::Handle;

This is upstream bug. Not fatal for Fedora.

Things that I'd like to see corrected:

* The homepage URL should end with slash.
* Description: Please expand the IC abbreviation to full words `integrated
circuit'. The description should be understandable even for guys who do not
want to install the package.
* The big clean-up with %{__rm} -rf `find . -name 'config*' is dangerous.
(Imagine a file name contained a white space). Use "find -name 'config*' -depth
-exec rm -rf -- '{}' \+" or something like that.
* Why the hct.pl has '.pl' extension? Is it necessary? Original build system
delivers "hct" wrapper (if it worked). What about just "hct" name or symlink to
hct.pl?

The spec file looks good otherwise. Please, show me updated spec file.

-- 
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.
--
Fedora Extras Perl SIG
http://www.fedoraproject.org/wiki/Extras/SIGs/Perl
perl-devel mailing list
perl-devel@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/perl-devel


[Index of Archives]     [Fedora Announce]     [Fedora Kernel]     [Fedora Testing]     [Fedora Legacy Announce]     [Fedora PHP Devel]     [Kernel Devel]     [Fedora Legacy]     [Fedora Maintainers]     [Fedora Desktop]     [PAM]     [Red Hat Development]     [Big List of Linux Books]     [Gimp]     [Yosemite Information]
  Powered by Linux