On Mon, Sep 03, 2007 at 03:55:02PM +0200, steven mestdagh wrote: > It seems to basically work on amd64 here.
Thanks!
> However it would be nice to install some documentation or at least a pointer
> to it.
I've included linkage in pkg/MESSAGE now. In the future I may make a man
page and submit upstream.
> Some tweaks:
> * Please check the CONFIGURE_ARGS again, --prefix=${SYSCONFDIR}/nginx
> looks confusing. Isn't it possible to just use ${PREFIX} there?
I have not done this. The location of some things are *only* done
relative to --prefix, and I want to keep all config in the same place
(/etc/nginx/). If there's something I'm not seeing here, please use the
cluestick.
> And do you still need any of that if you are using your own do-install?
> * Replace the post-patch target with a real patch file.
> * Don't rewrite the do-build target, but set ALL_TARGET empty.
Done, and done. Thanks!
In addition, I have moved the pid file and logs to /var/nginx/logs/
which matches how similar ports do it, at merdely@'s suggestion.
New tarball with above changes attached.
--
Darrin Chandler | Phoenix BSD User Group | MetaBUG
[EMAIL PROTECTED] | http://phxbug.org/ | http://metabug.org/
http://www.stilyagin.com/ | Daemons in the Desert | Global BUG Federation
nginx-0.5.31.tar.gz
Description: application/tar-gz
pgpzXxrkU5lAy.pgp
Description: PGP signature
