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-Net-FTPServer - Secure, extensible and configurable Perl FTP server Alias: perl-Net-FTPServer https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=248431 cweyl@xxxxxxxxxxxxxxx changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |cweyl@xxxxxxxxxxxxxxx Alias| |perl-Net-FTPServer ------- Additional Comments From cweyl@xxxxxxxxxxxxxxx 2007-08-09 21:56 EST ------- Excellent. I'm going to need this package on a couple boxes of mine soon :) (In reply to comment #2) > I was debating adding BR /usr/bin/uudecode, but that would make more sense if I > added a runtime dependency also, and I'm not sure I really want to encourage > turning on that functionality in a FTP server. Adding a BR will allow the test suite to fully exercise functional bits of the module and would seem to be in order... > The zip generation is optional functionality. I'm not sure how important the > test error is. The build completed fine for me even with that error. > > I can look into it a little deeper now if you want though. (I planned on > looking at that later anyway...) Filing a bug upstream and noting it in the spec would probably be a good way to address this sort of error in the test itself. (Just from the message here, it sounds like t/350generatorzip.t is probably mis-calling Archive::Zip.) -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug, or are watching someone who is. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review