Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 28 Nov 2005 13:09:54 -0800
From:      Charlie Schluting <charlie@schluting.com>
To:        Edwin Groothuis <edwin@FreeBSD.org>
Cc:        freebsd-ports-bugs@FreeBSD.org
Subject:   Re: ports/84530: [New Port] net-mgmt/ourmon: Network Monitoring and Anomaly Detection System
Message-ID:  <438B7222.60002@schluting.com>
In-Reply-To: <200511242353.jAONrpmx045000@freefall.freebsd.org>
References:  <200511242353.jAONrpmx045000@freefall.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
Edwin Groothuis wrote:
> Synopsis: [New Port] net-mgmt/ourmon: Network Monitoring and Anomaly Detection System
> 
> State-Changed-From-To: open->closed
> State-Changed-By: edwin
> State-Changed-When: Thu Nov 24 23:49:17 GMT 2005
> State-Changed-Why: 

First, I found a bug in the Makefile. (used spaces instead of a tab)
Can someone change the spaces on line 51 to a tab? (is a patch necessary 
for this?)

> Please let me nag you for a second:
> 
> - missing pkg-plist and/or incorrect PLIST_DIR/PLIST_FILES

I was under the impression that PLIST_DIRS= would use $PREFIX/name. I 
don't know where I got that idea. Thanks for generating the pkg-plist file.

> - lots of confusing and out-of-order targets in the Makefile

I know :(  See below..

> - why NO_PACKAGE?

This program has multiple parts, a web front-end and a C/perl back-end. 
The install process requires user interaction.. users can install either 
or both parts.

> - please create a pkg-install script which takes care of the
>   pre-install
> - see if you can extract the building phase from the installation
>   phase.

Well. It's calling a large perl script that's doing the install. In the 
script, it builds various C programs, talks to the user, and installs 
more files. There's really no way to make it ports-friendly in this 
regard, without rewriting the perl installer completely. That's a huge 
task..
It might be possible to provide default answers to the perl install 
script via a pkg-install script, if I really have to make it package-able.

> Committed, thanks!
> 

Thank you!

-Charlie



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?438B7222.60002>