Re: genini: comments

2010-06-22 Thread Christopher Faylor
On Wed, Jun 09, 2010 at 09:51:55AM -0400, Christopher Faylor wrote:
>On Tue, Jun 08, 2010 at 11:59:46PM -0500, Yaakov (Cygwin/X) wrote:
>>On Wed, 2010-06-09 at 00:38 -0400, Christopher Faylor wrote:
>>> That's not right.  The intent is to provide comments like:
>>> 
>>> foo: bar #comment
>>
>>Ah, well according to the website, mid-line comments are not allowed.
>>But if the website is wrong and they are supposed to be allowed, then
>>those within quotes (for sdesc/ldesc fields) should be ignored.
>
>I didn't write the website but if it says that quotes have to be at
comments
>the beginning of a line, it's wrong.


Re: genini: comments

2010-06-09 Thread Christopher Faylor
On Tue, Jun 08, 2010 at 11:59:46PM -0500, Yaakov (Cygwin/X) wrote:
>On Wed, 2010-06-09 at 00:38 -0400, Christopher Faylor wrote:
>> That's not right.  The intent is to provide comments like:
>> 
>> foo: bar #comment
>
>Ah, well according to the website, mid-line comments are not allowed.
>But if the website is wrong and they are supposed to be allowed, then
>those within quotes (for sdesc/ldesc fields) should be ignored.

I didn't write the website but if it says that quotes have to be at
the beginning of a line, it's wrong.

cgf


Re: genini: comments

2010-06-08 Thread Yaakov (Cygwin/X)
On Wed, 2010-06-09 at 00:38 -0400, Christopher Faylor wrote:
> That's not right.  The intent is to provide comments like:
> 
> foo: bar #comment

Ah, well according to the website, mid-line comments are not allowed.
But if the website is wrong and they are supposed to be allowed, then
those within quotes (for sdesc/ldesc fields) should be ignored.


Yaakov




Re: genini: comments

2010-06-08 Thread Christopher Faylor
On Tue, Jun 08, 2010 at 11:32:39PM -0500, Yaakov (Cygwin/X) wrote:
>setup.html says (emphasis mine):
>
>> Lines that *begin* with '#' are considered to be comments and are 
>> ignored by the setup.ini generator.
>
>Some of my sdesc/ldesc include a '#' not in the context of a comment
>(e.g. C#), and this was causing the rest of the field to be chomp()ed,
>mangling the resulting setup.ini.
>
>Patch attached.

That's not right.  The intent is to provide comments like:

foo: bar #comment

cgf