[Bug 462818] Review Request: perl-Net-SMTP-SSL - SSL support for Net::SMTP

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


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


Paul Howarth <paul at city-fan.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
                 CC|                            |paul at city-fan.org
         AssignedTo|nobody at fedoraproject.org    |paul at city-fan.org
               Flag|                            |fedora-review?




--- Comment #1 from Paul Howarth <paul at city-fan.org>  2008-10-13 18:59:40 EDT ---
Looks good. From a first glance, the only thing that sticks out as needing
changing is that your %files list needs to have:

%{perl_vendorlib}/Net/

rather than:

%{perl_vendorlib}/Net/SMTP/SSL.pm

so that directories %{perl_vendorlib}/Net/ and %{perl_vendorlib}/Net/SMTP/ are
owned by the package.

Formal review to follow.

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