You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sling.apache.org by Konrad Windszus <ko...@gmx.de> on 2015/08/11 17:49:50 UTC

Quoted string format in https://sling.apache.org/documentation/bundles/configuration-installer-factory.html

Currently in https://sling.apache.org/documentation/bundles/configuration-installer-factory.html the format for Configuration Files ending with .config is pretty well described. One point missing there is what exactly must be quoted within stringsimple. Actually just using quoted-string is not specific enough as also every “=“ need to be preceded by a “/“. What about extending the grammar like following
stringsimple ::= <quoted string representation of the value, where both “ and = need to be quoted with a leading “/“>
Do I need to quote any other characters in string simple?
Thanks,
Konrad


Re: Quoted string format in https://sling.apache.org/documentation/bundles/configuration-installer-factory.html

Posted by Carsten Ziegeler <cz...@apache.org>.
Am 11.08.15 um 18:29 schrieb Konrad Windszus:
> Thank you Carsten. Can you quickly review my proposed update at http://sling.staging.apache.org/documentation/bundles/configuration-installer-factory.html? <http://sling.staging.apache.org/documentation/bundles/configuration-installer-factory.html?>
> In case that fits, I will publish it.

Lgtm,

thanks
Carsten


-- 
Carsten Ziegeler
Adobe Research Switzerland
cziegeler@apache.org

Re: Quoted string format in https://sling.apache.org/documentation/bundles/configuration-installer-factory.html

Posted by Konrad Windszus <ko...@gmx.de>.
Thank you Carsten. Can you quickly review my proposed update at http://sling.staging.apache.org/documentation/bundles/configuration-installer-factory.html? <http://sling.staging.apache.org/documentation/bundles/configuration-installer-factory.html?>
In case that fits, I will publish it.
Konrad
> On 11 Aug 2015, at 17:55, Carsten Ziegeler <cz...@apache.org> wrote:
> 
> Am 11.08.15 um 17:49 schrieb Konrad Windszus:
>> Currently in https://sling.apache.org/documentation/bundles/configuration-installer-factory.html the format for Configuration Files ending with .config is pretty well described. One point missing there is what exactly must be quoted within stringsimple. Actually just using quoted-string is not specific enough as also every “=“ need to be preceded by a “/“. What about extending the grammar like following
>> stringsimple ::= <quoted string representation of the value, where both “ and = need to be quoted with a leading “/“>
>> Do I need to quote any other characters in string simple?
> 
> I think those are the only two, however I would call this escaping :)
> In fact I have the feeling that the special handling of "=" within a
> quoted string is wrong, but that's in there for ages, so it probably
> good not to change it.
> 
> Regards
> Carsten
> -- 
> Carsten Ziegeler
> Adobe Research Switzerland
> cziegeler@apache.org


Re: Quoted string format in https://sling.apache.org/documentation/bundles/configuration-installer-factory.html

Posted by Carsten Ziegeler <cz...@apache.org>.
Am 11.08.15 um 17:49 schrieb Konrad Windszus:
> Currently in https://sling.apache.org/documentation/bundles/configuration-installer-factory.html the format for Configuration Files ending with .config is pretty well described. One point missing there is what exactly must be quoted within stringsimple. Actually just using quoted-string is not specific enough as also every “=“ need to be preceded by a “/“. What about extending the grammar like following
> stringsimple ::= <quoted string representation of the value, where both “ and = need to be quoted with a leading “/“>
> Do I need to quote any other characters in string simple?

I think those are the only two, however I would call this escaping :)
In fact I have the feeling that the special handling of "=" within a
quoted string is wrong, but that's in there for ages, so it probably
good not to change it.

Regards
Carsten
-- 
Carsten Ziegeler
Adobe Research Switzerland
cziegeler@apache.org