[PATCH] clean(er) squid.conf

2008-08-20 Thread Amos Jeffries
Here is a quick patch I think builds a clean squid.conf file for use The gist of it is to build to shift the existing squid.conf.default to the name squid.conf.documented (anyone got a better one?) Then to generate a new squid.conf.default from that by pruning the comments and empty lines.

Re: [PATCH] clean(er) squid.conf

2008-08-20 Thread Robert Collins
On Wed, 2008-08-20 at 19:03 +1200, Amos Jeffries wrote: Here is a quick patch I think builds a clean squid.conf file for use The gist of it is to build to shift the existing squid.conf.default to the name squid.conf.documented (anyone got a better one?) Then to generate a new

Re: [PATCH] clean(er) squid.conf

2008-08-20 Thread Amos Jeffries
Robert Collins wrote: On Wed, 2008-08-20 at 19:03 +1200, Amos Jeffries wrote: Here is a quick patch I think builds a clean squid.conf file for use The gist of it is to build to shift the existing squid.conf.default to the name squid.conf.documented (anyone got a better one?) Then to generate

Re: [PATCH] clean(er) squid.conf

2008-08-20 Thread Alex Rousskov
On Wed, 2008-08-20 at 19:03 +1200, Amos Jeffries wrote: Here is a quick patch I think builds a clean squid.conf file for use The gist of it is to build to shift the existing squid.conf.default to the name squid.conf.documented (anyone got a better one?) Then to generate a new

Re: [PATCH] clean(er) squid.conf

2008-08-20 Thread Henrik Nordstrom
On ons, 2008-08-20 at 19:03 +1200, Amos Jeffries wrote: Here is a quick patch I think builds a clean squid.conf file for use The gist of it is to build to shift the existing squid.conf.default to the name squid.conf.documented (anyone got a better one?) Or integrade the cfgman script and

Re: [PATCH] clean(er) squid.conf

2008-08-20 Thread Henrik Nordstrom
On ons, 2008-08-20 at 20:18 +1200, Amos Jeffries wrote: It also force-installs squid.conf.documented and squid.conf.default. Assuming that squid.conf is the one user edits to be left untouched. .default is supposed to be force installed already. If it's not thats a bug. Regards Henrik

Re: [PATCH] clean(er) squid.conf

2008-08-20 Thread Henrik Nordstrom
On ons, 2008-08-20 at 11:43 -0600, Alex Rousskov wrote: Please note that this will cause headaches for distributions that include squid.conf patches (e.g., CentOS Squid RPMs). Do they really patch squid.conf and not cf.data.pre? The RPMs I have seen patches cf.data.pre as they should. Regards

Re: [PATCH] clean(er) squid.conf

2008-08-20 Thread Alex Rousskov
On Thu, 2008-08-21 at 00:08 +0200, Henrik Nordstrom wrote: On ons, 2008-08-20 at 11:43 -0600, Alex Rousskov wrote: Please note that this will cause headaches for distributions that include squid.conf patches (e.g., CentOS Squid RPMs). Do they really patch squid.conf and not cf.data.pre?