Darrin Chandler [2007-09-02, 11:11:00]:
> After some feedback I've changed this to use _nginx user and group.
> Please test and commit.

It seems to basically work on amd64 here.
However it would be nice to install some documentation or at least a pointer
to it.

Some tweaks:
* Please check the CONFIGURE_ARGS again, --prefix=${SYSCONFDIR}/nginx
looks confusing. Isn't it possible to just use ${PREFIX} there?
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.

Reply via email to