[Bug 895480] Review Request: perl-ClamAV-Client - Client class for the ClamAV clamd virus scanner daemon

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=895480

--- Comment #2 from Mathieu Bridon <bochecha@xxxxxxxxxxxxxxxxx> ---
(In reply to comment #1)
> TODO: You can replace %{__perl} macros with plain perl command.

Fixed.

> $ rpm -q --requires -p
> ../RPMS/noarch/perl-ClamAV-Client-0.11-1.fc19.noarch.rpm | sort |uniq -c
>       1 perl(base)
>       1 perl(Carp)
>       1 perl(ClamAV::Config)
>       1 perl(constant)
>       1 perl(Error)
>       1 perl(Error::Simple)
>       1 perl(IO::File)
>       1 perl(IO::Socket)
>       1 perl(:MODULE_COMPAT_5.16.2)
>       1 perl(strict)
>       1 perl(warnings)
>       1 rpmlib(CompressedFileNames) <= 3.0.4-1
>       1 rpmlib(FileDigests) <= 4.6.0-1
>       1 rpmlib(PayloadFilesHavePrefix) <= 4.0-1
>       1 rpmlib(PayloadIsXz) <= 5.2-1
> FIX: Run-require `perl(IO::Socket::INET)' (lib/ClamAV/Client.pm:387).
> FIX: Run-require `perl(IO::Socket::UNIX)' (lib/ClamAV/Client.pm:448).

Oh, good catch. Fixed.

> $ resolvedeps rawhide
> ../RPMS/noarch/perl-ClamAV-Client-0.11-1.fc19.noarch.rpm 
> Binary dependencies resolvable. Ok.

[OFF-TOPIC] resolvedeps looks like a cool tool, where can I find it? :)

> Please correct all `FIX' issues, consider fixing `TODO' items, and provide
> new spec file.

Thanks for the comments, here's the new submission.

Spec URL: http://bochecha.fedorapeople.org/packages/perl-ClamAV-Client.spec
SRPM URL:
http://bochecha.fedorapeople.org/packages/perl-ClamAV-Client-0.11-2.fc18.src.rpm

-- 
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=g8rgajY7Xs&a=cc_unsubscribe
_______________________________________________
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]