This issue tracker has been migrated to GitHub ,
and is currently read-only.
For more information,
see the GitHub FAQs in the Python's Developer Guide.
Created on 2013年10月31日 17:24 by Nacsa.Kristóf, last changed 2022年04月11日 14:57 by admin. This issue is now closed.
| Files | ||||
|---|---|---|---|---|
| File name | Uploaded | Description | Edit | |
| 89484_issue19461.patch | Norman.Denayer, 2014年03月08日 09:51 | |||
| Messages (5) | |||
|---|---|---|---|
| msg201829 - (view) | Author: Nacsa Kristóf (Nacsa.Kristóf) | Date: 2013年10月31日 17:24 | |
http://hg.python.org/cpython/file/8d5df9602a72/Lib/ConfigParser.py#l529 RawConfigParser has a special case of replacing '""' with '' in reader. However the writer does not do the same. This may cause problems, for example if you merge multiple ini files into one and the 3rd party program that should use the output segfaults on an ini that has `key=` instead of `key=value`, as it is in my case. :) I think that the reasoning with the reader maybe was that an empty string must be an empty python string. However there is no option to control how it gets written back to the output. One can of course walkaround it by replacing the '' values with '""', but I think the standard library is ought to be consistent, walkarounds are unpythonic, a raw parser should be transparent either way, and there should preferrably be one obvious way to do it. I think this is not just unintuitive (especially in a parser called "raw"), but goes against the philosophy of "refusing the temptation to guess". As I know, the syntax of .ini files is not exactly well-defined, so the best option in my opinion would be allowing to configure the parser how to treat these values. Whether or not adding this option is viable, I think the problem still remains that the writer now works inconsistently of the reader. |
|||
| msg201830 - (view) | Author: Nacsa Kristóf (Nacsa.Kristóf) | Date: 2013年10月31日 17:25 | |
typo: `key=` instead of `key=value` was meant as `key=` instead of `key=""` |
|||
| msg201834 - (view) | Author: R. David Murray (r.david.murray) * (Python committer) | Date: 2013年10月31日 17:47 | |
A configurable option is probably the only way forward, due to backward compatibility reasons, but Łukasz will know for sure. If so, it can only go in the *next* version of Python, and while we haven't hit beta yet on 3.4 it may be too late for anyone to have time to get it in before then. Also, I was under the impression that 'key=' was the canonical way to indicate an empty value in an ini file. As you say, the format is not well defined (no surprise). |
|||
| msg212924 - (view) | Author: Norman Denayer (Norman.Denayer) * | Date: 2014年03月08日 09:51 | |
I guess it's logical to have the value "" turned to '' in the reader, but I would expect the same transformation in the writer. If the write function would write "" for an empty string, will it solve your issue? |
|||
| msg226396 - (view) | Author: Łukasz Langa (lukasz.langa) * (Python committer) | Date: 2014年09月05日 02:10 | |
This bug has been fixed in Python 3.2 (see 892236137db9). Please use the configparser backport for Python 2.7. |
|||
| History | |||
|---|---|---|---|
| Date | User | Action | Args |
| 2022年04月11日 14:57:52 | admin | set | github: 63660 |
| 2014年09月05日 02:10:34 | lukasz.langa | set | status: open -> closed type: enhancement -> behavior messages: + msg226396 resolution: fixed stage: needs patch -> resolved |
| 2014年03月08日 09:51:30 | Norman.Denayer | set | files:
+ 89484_issue19461.patch nosy: + Norman.Denayer messages: + msg212924 keywords: + patch |
| 2013年11月10日 21:36:51 | lukasz.langa | set | assignee: lukasz.langa |
| 2013年10月31日 17:47:42 | r.david.murray | set | type: behavior -> enhancement versions: + Python 3.5, - Python 2.7 keywords: + easy nosy: + r.david.murray, lukasz.langa messages: + msg201834 stage: needs patch |
| 2013年10月31日 17:25:41 | Nacsa.Kristóf | set | messages: + msg201830 |
| 2013年10月31日 17:24:06 | Nacsa.Kristóf | create | |