Opened 9 years ago

Closed 9 years ago

Last modified 9 years ago

#46443 closed enhancement (fixed)

p5-fusioninventory-agent: upgrade and add 5.18 5.20

Reported by: mojca (Mojca Miklavec) Owned by: dbevans (David B. Evans)
Priority: Normal Milestone:
Component: ports Version:
Keywords: Cc:
Port: p5-fusioninventory-agent

Description

I'm attaching a patch for changes for p5-fusioninventory-agent. The dependencies have been generated with cpan2port.

I added all dependencies:

  • r131169 p5-test-mockmodule: new port
  • r131168 p5-test-http-server-simple: new port
  • r131167 p5-test-compile: new port
  • r131166 p5-poe-component-client-ping: new port
  • r131165 p5-poe: new port (tests fail, #46435)
  • r131164 p5-io-pipely: new port
  • r131163 p5-poe-test-loops: new port
  • r131162 p5-xml-treepp: new port
  • r131161 p5-io-capture: new port
  • r131160 p5-http-server-simple-authen: new port
  • r131159 p5-authen-simple: new port
  • r131156 p5-http-proxy: new port

as well as (unauthorized) r131158 because I needed p5.20-crypt-passwdmd5, but the tests still keep failing, partially due to missing dependencies (for example Can't locate File/Copy/Recursive.pm).

If anyone feels like finishing the job, feel free to do so.

(It also seems that the tests try to read the contents of the complete hard drive.)

Attachments (3)

p5-fusioninventory-agent.diff (2.4 KB) - added by mojca (Mojca Miklavec) 9 years ago.
Upgrade to version 2.3.14, add 5.18 5.20, add dependencies
p5-fusioninventory-agent-2.3.15.diff (2.7 KB) - added by dbevans (David B. Evans) 9 years ago.
Proposed patch revised, update to version 2.3.15, dependencies.
test.log (176.3 KB) - added by dbevans (David B. Evans) 9 years ago.
Successful test log for fusioninventory-agent

Download all attachments as: .zip

Change History (10)

Changed 9 years ago by mojca (Mojca Miklavec)

Upgrade to version 2.3.14, add 5.18 5.20, add dependencies

comment:1 Changed 9 years ago by dbevans (David B. Evans)

Cc: devans@… added
Owner: devans@… deleted

comment:2 Changed 9 years ago by mojca (Mojca Miklavec)

Sorry, wrong field. And thanks for fixing.

Last edited 9 years ago by mojca (Mojca Miklavec) (previous) (diff)

Changed 9 years ago by dbevans (David B. Evans)

Proposed patch revised, update to version 2.3.15, dependencies.

comment:3 Changed 9 years ago by dbevans (David B. Evans)

Updated patch attached which

  • updates the port to recently released 2.3.15
  • fixes homepage
  • adds the dependency on p5-file-copy-recursive listed as recommended in META.yml and mandatory for deploy task in README.

This cleans up the tests a bit but they eventually fail after a lot of disk activity as noted above.

comment:4 Changed 9 years ago by dbevans (David B. Evans)

Owner: set to devans@…
Status: newassigned

comment:5 Changed 9 years ago by dbevans (David B. Evans)

Cc: devans@… removed
Resolution: fixed
Status: assignedclosed

" ... and now for something completely different." M. Python

After working on several destroot errors, it became apparent that this port is intended to be a perl application that explicitly installs outside the normal perl hierarchy. The perl modules that it provides are for its use only and are not intended to be made part of the public perl extensions.

This being the case, it's more appropriate to rename this port to fusioninventory-agent and use perl variants rather than subports to resolve conflicts between perl versions.

fusioninventory-agent updated to version 2.3.15 with variants +perl5_16 +perl5_18 +perl5_20 committed in r131617.
p5-fusioninventory-agent declared obsolete, replaced by fusioninventory-agent and p5-graveyard updated in r131622.

Test now is successful but takes a long time to run (apparently searching local disk several times trying to construct new inventories for several different file categories). Test log attached.

Changed 9 years ago by dbevans (David B. Evans)

Attachment: test.log added

Successful test log for fusioninventory-agent

comment:6 in reply to:  5 Changed 9 years ago by mojca (Mojca Miklavec)

Replying to devans@…:

p5-fusioninventory-agent declared obsolete, replaced by fusioninventory-agent and p5-graveyard updated in r131622.

You accidentally deleted the comments above p5-acme-lolcat.

But off-topic (and not to relevant) question: what's the point in automatically updating all ports in p5-graveyard? The list now contains a bunch of ports that previously never existed (say, p5-crypt-openssl-dsa) and I don't see any need to keep updating the version numbers for the buried ports until we'll need a replacement for p5.16-foo.

comment:7 Changed 9 years ago by mojca (Mojca Miklavec)

And of course thanks a lot for fixing the port.

Note: See TracTickets for help on using tickets.