On Sun, Jul 03, 2016 at 02:21:30PM +0200, Sebastien Marie wrote:
> On Sun, Jul 03, 2016 at 01:01:54PM +0200, Solène RAPENNE wrote:
> >
> > New tarball after fixed things reported by portcheck
>
> If someone else want to import it, he has my OK.
>
I might have been to quick to told OK. sorry.
Additionnal questions for people really aware in perl port stuff (or
more aware than me at least):
The package is pure perl code, I think `PKG_ARCH = *' has to be set.
When building, the following warnings are shown:
Warning: prerequisite AWS::Signature4 0 not found.
Warning: prerequisite Devel::Caller 0 not found.
Warning: prerequisite HTTP::Request 0 not found.
Warning: prerequisite HTTP::Request::Common 0 not found.
Warning: prerequisite Hash::Merge 0 not found.
Warning: prerequisite IO::Pty 0 not found.
Warning: prerequisite IO::String 0 not found.
Warning: prerequisite JSON::XS 0 not found.
Warning: prerequisite LWP::UserAgent 0 not found.
Warning: prerequisite List::MoreUtils 0 not found.
Warning: prerequisite Net::OpenSSH 0 not found.
Warning: prerequisite Net::SFTP::Foreign 0 not found.
Warning: prerequisite Sort::Naturally 0 not found.
Warning: prerequisite Test::Pod 0 not found.
Warning: prerequisite Text::Glob 0 not found.
Warning: prerequisite URI 0 not found.
Warning: prerequisite URI::QueryParam 0 not found.
Warning: prerequisite XML::Simple 0 not found.
As the build stuff is to copying, I think it is ok.
I am also unsure about RUN_DEPENDS regarding the wanted prerequisites.
The following perl packages will not be available at runtime:
- AWS::Signature4 (and no package for it)
- HTTP::Request p5-HTTP-Message
- HTTP::Request::Common p5-HTTP-Message
- Hash::Merge p5-Hash-Merge
- IO::Pty p5-IO-Tty
- IO::String p5-IO-String
- Net::SFTP::Foreign p5-Net-SFTP-Foreign
- Test::Pod p5-Test-Pod
- URI p5-URI
- URI::QueryParam p5-URI
And following packages seems to be unneed:
- devel/p5-Term-ReadKey
- net/p5-Net-SSH2
It seems also the test target isn't covered.
Please note that portgen(1) could be used to generate a complete port (+
missing ports):
$ portgen p5 Rex
[...]
/usr/ports/mystuff/cpan/p5-AWS-Signature4
/usr/ports/mystuff/cpan/p5-String-Escape
/usr/ports/mystuff/cpan/p5-Test-UseAllModules
/usr/ports/mystuff/cpan/p5-Rex
--
Sebastien Marie
