New Ticket     Tickets     Wiki     Browse Source     Timeline     Roadmap     Ticket Reports     Search

Ticket #26325 (closed submission: fixed)

Opened 3 years ago

Last modified 2 years ago

asused

Reported by: fclaire@… Owned by: macports-tickets@…
Priority: Normal Milestone:
Component: ports Version: 1.9.1
Keywords: Cc: raimue@…
Port: asused

Description

Here's a port of the RIPE NCC's asused tool

Known problems:

  • I was unable to find the source package on RIPE's FTP server ftp://ftp.ripe.net/tools/ so I use a FreeBSD server to fetch the source code
  • It seems incompatible with rrdtool port. If rrdtool is installed you'll get the following message:
--->  Activating asused @3.72_whois_new_query_fix_0
Error: Target org.macports.activate returned: Image error: /opt/local/lib/perl5/5.8.9/darwin-2level/perllocal.pod is being used by the active rrdtool port.  Please deactivate this port first, or use 'port -f activate asused' to force the activation.
Error: Status 1 encountered during processing.

Any suggestion to fix them is welcomed.

Attachments

Portfile (2.7 KB) - added by fclaire@… 2 years ago.
deletes perllocal.pod
patch-ipv4pack-ipv4pack.pm.diff (518 bytes) - added by fclaire@… 2 years ago.

Change History

comment:1 Changed 3 years ago by jmr@…

No port should install perllocal.pod; delete it in post-destroot if you can't prevent the build system from installing it in the first place.

comment:2 Changed 3 years ago by fclaire@…

Hi,

I've updated the Portfile to delete perllocal.pod in post-destroot.

The Makefile is appending text to perllocal.pod and this Makefile is generated by a perl script. As I'm not familiar with perl I didn't try to patch this script to prevent the build system to generate perllocal.pod.

comment:3 Changed 3 years ago by raimue@…

  • Cc raimue@… added

For a better version number:

version 3.72
...
distname ${name}-${version}_whois_new_query_fix

Using perl is too general, especially without additional dependencies. With a fallback to /usr/bin/perl this would end up in system directories.

PortGroup perl5 1.0

...

depends_lib         path:[string range ${perl5.bin} [string length ${prefix}/] end]:perl5

configure.cmd       ${perl5.bin}
configure.env       PERL_AUTOINSTALL=--skipdeps
configure.pre_args  Makefile.PL
configure.args      INSTALLDIRS=vendor

To make this more portable even when the version of perl changes (or perl5.10 are perl5.12 used):

post-destroot {
    delete [glob ${destroot}${prefix}/lib/perl5/*/*/perllocal.pod]
}

According to FreeBSD, this also needs patching for the config file:

post-patch {
    reinplace "s|asused\\.conf|${prefix}/etc/asused\\.conf|g" ${worksrcpath}/asused.PL
}

In general, the config file should be handled as described in PortfileRecipes.

Debian has a few more patches, maybe one of them is worth porting?

All the comments above are untested, but I hope that will be enough :-)

comment:4 Changed 3 years ago by fclaire@…

Thanks for your remarks and the time spent on this. I'll work on it in the coming days.

Changed 2 years ago by fclaire@…

deletes perllocal.pod

Changed 2 years ago by fclaire@…

comment:5 Changed 2 years ago by fclaire@…

Implemented your proposals in the Portfile, example config file should be properly handled now.

For the Debian patchs:

  • 01_whois_new_query_fix.dpatch, 02_typo_fix.dpatch and 03_interpreter_fix.dpatch are already implemented in asused-3.72_whois_new_query_fix.tar.gz
  • 04_drop_network_tests.dpatch's goal is to avoid execution of tests requiring a network connection. This is useful for distribs on CDs/DVDs but in the case of macports the install is online anyhow. I didn't applied this patch.
  • 05_fix_priv_range.dpatch is applied

I also fixed the livecheck.

Thanks for reviewing and comitting.

comment:6 Changed 2 years ago by raimue@…

  • Status changed from new to closed
  • Resolution set to fixed

Committed in r74303.

Note: See TracTickets for help on using tickets.