Sebastien Marie writes:

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

We may need to create a port for AWS::Signature4 which may be used by an
AWS module of Rex.
Maybe I forgot some dependencies, I don't really know how to catch all
deps needed.

> And following packages seems to be unneed:
>   - devel/p5-Term-ReadKey
>   - net/p5-Net-SSH2
>

Rex doesn't start if the Term::ReadKey isn't installed and can't use ssh
without Net::SSH2

> 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

I didn't know that, handy tool !

Reply via email to