Re: [PATCΗ] Quoted values in squid.conf

On 28/05/2013 4:12 a.m., Alex Rousskov wrote:
> On 05/27/2013 04:54 AM, Tsantilas Christos wrote:
>> On 05/26/2013 10:05 PM, Kinkie wrote:
>>>>> The only thing I would like to see differently implemented is the
>>>>> syntax used to include files:
>>>>> file(path) would be IMO easier to understand and less prone to
>>>>> confusion than the proposed syntax.
>>>> OK.
>>>> But imagine in the future also the following syntax:
>>>> file:/path/file
>>>> system:/usr/local/squid/bin/my-squid-conf (to read from an executable
>>>> stdout configuration options)
>>>> http://hostname/cfgfile (to get from web page configuration)
>>>>
>>>> All the above can be implemented in the future...
>>> Sure, I agree.
>>>
>>> file(/path/file)
>>> system(/some/executable)
>>> http_get(http://hostname/file)
>> Well, this is not a bad scheme :-)
>>
>> Just the file:/path/to/file a little easier to implement. But not
>> something important...
>
> I agree that file() is a good alternative. Amos (and others), do you
> have a preference between
>
> file:"/path/file"
>
> and
>
> file("/path/file")
>
>
> syntax?

I slightly prefer file: without the quotes since we can use the URL
parser to interpret it, and like Christo said extend it to full URI
loaded remote content someday.

If we retain the quotes, file= would seem clearer. () is a form of
quoting and you arleady have my opinion of adding more than one form of
quoting. (dont know how I missed out ()-quoting in my rant earlier).

Amos
Received on Tue May 28 2013 - 13:14:52 MDT