Hi, I must object to this change, as I use varnishncsa -b extensively to pull out reports on missed objects being fetch from the backends too often. Please revert this change, as varnishncsa -b has its use.
Anders. On Mon, Jan 25, 2010 at 01:19:22PM +0100, [email protected] wrote: > Author: kristian > Date: 2010-01-25 13:19:22 +0100 (Mon, 25 Jan 2010) > New Revision: 4480 > > Modified: > trunk/varnish-cache/bin/varnishncsa/varnishncsa.c > Log: > Only handle client requests (-c) for varnishncsa > > Fixes #617 > > > Modified: trunk/varnish-cache/bin/varnishncsa/varnishncsa.c > =================================================================== > --- trunk/varnish-cache/bin/varnishncsa/varnishncsa.c 2010-01-25 11:36:43 UTC > (rev 4479) > +++ trunk/varnish-cache/bin/varnishncsa/varnishncsa.c 2010-01-25 12:19:22 UTC > (rev 4480) > @@ -553,6 +553,13 @@ > case 'w': > w_arg = optarg; > break; > + case 'b': > + fprintf(stderr, "-b is not valid for varnishncsa\n"); > + exit(1); > + break; > + case 'c': > + /* XXX: Silently ignored: it's required anyway */ > + break; > default: > if (VSL_Arg(vd, c, optarg) > 0) > break; > @@ -560,6 +567,8 @@ > } > } > > + VSL_Arg(vd, 'c', optarg); > + > if (VSL_OpenLog(vd, n_arg)) > exit(1); > > > _______________________________________________ > varnish-commit mailing list > [email protected] > http://projects.linpro.no/mailman/listinfo/varnish-commit -- Anders. _______________________________________________ varnish-dev mailing list [email protected] http://projects.linpro.no/mailman/listinfo/varnish-dev
