[Bug 237330] Review Request: perl-Crypt-OpenSSL-AES -- Perl OpenSSL bindings for AES support -- SPONSOR NEEDED

[bugzilla at redhat.com]

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

Summary: Review Request:  perl-Crypt-OpenSSL-AES -- Perl OpenSSL bindings for AES support -- SPONSOR NEEDED


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





------- Additional Comments From tibbs at math.uh.edu  2007-05-07 00:39 EST -------
Not a very good start, I'm afraid.  This one fails to build for me, and in a
manner that makes me wonder if you tried to build the package yourself.

First off, it should have BuildRequires: openssl-devel in order to bring the
openssl headers necessary for compiling.

Once that's done, the %files section of the spec is that from  a noarch package,
but this one is arch-specific.  You need to delete the perl_vendorlib bit and
replace it with:
  %{perl_vendorarch}/auto/*
  %{perl_vendorarch}/Crypt/

The explicit dependency on openssl is unnecessary; rpm will automatically find a
dependency on libcrypto.so.6, which happens to be satisfied by openssl.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.