Opened 13 years ago

Closed 12 years ago

#31330 closed update (fixed)

ntop: update to 4.1.0

Reported by: ryandesign (Ryan Carsten Schmidt) Owned by: markd@…
Priority: Normal Milestone:
Component: ports Version:
Keywords: haspatch Cc:
Port: ntop

Description

ntop should be updated to the latest version, currently 4.1.0. This should resolve a few of the outstanding ntop tickets including #28457. I've been working on this update but don't use ntop so I don't know if it works properly. I'll attach my patch later.

Attachments (1)

ntop-4.1.0.diff (10.8 KB) - added by ryandesign (Ryan Carsten Schmidt) 12 years ago.

Download all attachments as: .zip

Change History (3)

comment:1 Changed 12 years ago by ryandesign (Ryan Carsten Schmidt)

Keywords: haspatch added

Here's my patch finally. It does many things:

  • update port to 4.1.0
  • rewrite master_sites to avoid redirects
  • add automake, autoconf, libtool build dependencies so we're not using possibly obsolete OS versions
  • add python27 dependency and add env vars to use it
  • remove unnecessary configure cflags and args
  • rewrite startupitem to use startupitem.executable and no longer need ntop.sh wrapper
  • install sample config file and databases; don't register them to the port so they're not clobbered at port upgrades
  • enable universal variant
  • patch autogen.sh to not require wget
  • patch autogen.sh to look for libtool.m4 in ${prefix} instead of /usr/local
  • patch configure to not look in /usr/local or hardcoded /opt/local
  • patch configure to find python
  • patch configure to fix syntax errors in removal of arch flags from python config output
  • patch configure to no longer use obsolete 2002-era Mac OS X versions of ltconfig and ltmain.sh; this fixes the port so it's UsingTheRightCompiler which addresses comment:ticket:28457:6
  • patch configure to use MacPorts libtool
  • remove patch-external-geoip.diff; I don't know what to do with that right now

I don't use ntop so I don't know if it still works correctly.

The universal variant almost works; the reason why it was originally disabled no longer applies, but though the ntop program and its dylibs end up being universal, none of the plugins in ${prefix}/lib/ntop/plugins/ do. So that still needs to be fixed.

Changed 12 years ago by ryandesign (Ryan Carsten Schmidt)

Attachment: ntop-4.1.0.diff added

comment:2 Changed 12 years ago by ryandesign (Ryan Carsten Schmidt)

Resolution: fixed
Status: newclosed

Committed in r94641. Did not re-enable universal variant since I could not immediately figure out how to fix its problems.

Note: See TracTickets for help on using tickets.