Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 14 Oct 2009 01:50:13 +0200
From:      =?utf-8?Q?Dag-Erling_Sm=C3=B8rgrav?= <des@des.no>
To:        Doug Barton <dougb@FreeBSD.org>
Cc:        cvs-ports@FreeBSD.org, cvs-all@FreeBSD.org, ports-committers@FreeBSD.org
Subject:   Re: cvs commit: ports/www/varnish Makefile ports/www/varnish/files pkg-message.in varnishd.in varnishlog.in varnishncsa.in
Message-ID:  <86iqeietvu.fsf@ds4.des.no>
In-Reply-To: <4AD4DE92.3090808@FreeBSD.org> (Doug Barton's message of "Tue, 13 Oct 2009 13:09:54 -0700")
References:  <200910131851.n9DIpB7P057665@repoman.freebsd.org> <4AD4DE92.3090808@FreeBSD.org>

next in thread | previous in thread | raw e-mail | index | archive | help
Doug Barton <dougb@FreeBSD.org> writes:
> Also in varnishd, the test for the existence of $varnishd_config
> should really be encased in a pre_start method since that's the only
> time it's relevant (unless I'm missing something).

When does pre_start run?  Does your patch allow the user to override
varnishd_flags?  If not, it should remain as it is.

DES
--=20
Dag-Erling Sm=C3=B8rgrav - des@des.no



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?86iqeietvu.fsf>