On 5/17/06, Enrico Forestieri <[EMAIL PROTECTED]> wrote:
On Wed, May 17, 2006 at 12:40:51PM -0500, Bo Peng wrote:
> >I think you mean something like the attached.
>
> No, the original one tries to strip ' plus \n, and then strip another
> ', from a line like
> key = 'value'\n
Exact. This is what my patch does, too.
Yes. You are right, strip("'") strips both leading and trailing '.
Then, my replace function just complicates the issue unnecessarily.
> or, because this ugly part is translated from configure.m4, and the
> whole key value business is used *only* for the next chk_fontenc, you
> can remove it all, and replace it with things like:
>
> allvars = open('chkconfig.vars').read()
> if allvars.match('chk_fontenc = "yes"'):
> addToRC(....)
> else:
> addToRC(....)
I can only propose, I cannot do anything ;-)
OK. If you have tested your patch on cygwin, I will apply this to the
trunk, and leave this bigger idea later. Actually, JMarc has suggest
that I read configure.py and remove such things (and possibly use some
python-oriented file formats).
Bo
