[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: [RFR] templates://squid/{templates}



I forgot adding soem rationale for the proposed changes:



> @@ -53,14 +65,19 @@
>  Type: boolean
>  Default: true
>  _Description: Upgrade squid.conf automatically?
> - Incompatible settings have been found in the existing squid.conf file;
> - they will prevent squid from starting or working correctly. However, this
> - setup can try to modify them to work again. Do you want to apply the
> - changes?
> + Incompatible settings have been found in the existing squid.conf file.
> + .
> + They will prevent Squid from starting or working correctly. 
> + .
> + These settings can be corrected now. Please choose whether you want
> + to apply the needed changes.

1) Remove the question in long description. Discouraged in Developer's
Reference

2) Split out in paragraphs for clarity

> @@ -78,11 +98,13 @@
>  Type: boolean
>  Default: false
>  _Description: Fix permissions of cache_dir?
> - Values for cache_effective_user and/or cache_effective_group in config file 
> - are incompatible with owner/group of cache directories. Do you want to
> - automatically fix permissions on cache directory?
> - .
> - WARNING: If you specified a cache directory different from /var/spool/squid
> - and selected some other directory used by other programs (e.g. /tmp), this
> - could affect those programs.
> + The values for 'cache_effective_user' and/or 'cache_effective_group'
> + in Squid's configuration file are incompatible with owner/group of cache
> + directories.
> + . 
> + Please choose whether this hould be fixed automatically.
> + .
> + However, please note that if you specified a cache directory
> + different from /var/spool/squid and selected some other directory
> + used by other programs (e.g. /tmp), this could affect those programs.

Add a "the" in the hope this does not make the whole thing sound too
Frenglish.

Specify the "config file" is the configuration fiel for Squid.

Avoid "WARNING". Personal taste. I don't like such big shouts. The
long paragraph should eb enough to get attention.

PS: the "hould" typo is already corrected..:-)


>  
> --- ../squid.old/debian/control	2007-05-14 13:54:33.854125028 +0200
> +++ debian/control	2007-05-20 09:58:51.632618146 +0200
> @@ -14,24 +14,26 @@
>  Conflicts: squid-novm, sarg (<< 1.1.1-2)
>  Replaces: squid-novm
>  Suggests: squidclient, squid-cgi, logcheck-database, resolvconf (>= 0.40), smbclient
> -Description: Internet Object Cache (WWW proxy cache)
> - This is the Squid Internet Object Cache developed by the National
> - Laboratory for Applied Networking Research (NLANR) and Internet
> - volunteers.  This software is freely available for anyone to use.  The
> - Squid home page is http://www.squid-cache.org/
> +Description: Internet object cache (WWW proxy cache)

Avoid Extra Capitalization That Make Titles Look Ugly


> + This package provides the Squid Internet Object Cache developed by
> + the National Laboratory for Applied Networking Research (NLANR) and
> + Internet volunteers.
> + .
> +  Homepage: http://www.squid-cache.org/

Remove the useless "freely available". Everything in Debian is.

Split out and use the current practice to specify the homepage

> +Description: Internet object cache (WWW proxy cache) - common files

There's more than one file, right?

> -Description: Command line URL extractor that talks to (a) squid
> +Description: command-line URL extractor for Squid

hyphen and removal of initial capital letter

mention 'Squid' as a noun


Attachment: signature.asc
Description: Digital signature


Reply to: