Issue 570655: bdist_rpm and the changelog option (original) (raw)

Created on 2002-06-18 16:07 by misa, last changed 2022-04-10 16:05 by admin. This issue is now closed.

Messages (6)
msg11246 - (view)	Author: Mihai Ibanescu (misa)	Date: 2002-06-18 16:07
I think the changelog option for bdist_rpm is not correctly handled. The documentation says it should expect a path to a file containing the changelog entries, but in bdist_rpm.py, line 200, it tries to read it as a string (ensure_string etc). The changelogs are multiple lines, and I was unable to add multi-lined entries in setup.cfg. Even so, the documentation is still misleading.
msg11247 - (view)	Author: Mihai Ibanescu (misa)	Date: 2002-06-28 19:57
Logged In: YES user_id=205865 Uploaded a patch
msg11248 - (view)	Author: A.M. Kuchling (akuchling) *	Date: 2002-11-04 13:28
Logged In: YES user_id=11375 Hmm... looking at old versions of bdist_rpm, changelog has always been a string. We can't change it to a path now because that might break existing setup.py file; instead, the documentation should be fixed. Thanks for reporting this!
msg11249 - (view)	Author: A.M. Kuchling (akuchling) *	Date: 2002-11-04 13:36
Logged In: YES user_id=11375 Option hint fixed in revision 1.33 of bdist_rpm.py; fix also backported to 2.2 branch.
msg11250 - (view)	Author: Mihai Ibanescu (misa)	Date: 2002-11-05 20:54
Logged In: YES user_id=205865 akuchling: how can you set changelog then? Changelog entries are multi-lined, I did not find a way to set it in setup.cfg. Do I miss something?
msg11251 - (view)	Author: A.M. Kuchling (akuchling) *	Date: 2002-11-06 14:54
Logged In: YES user_id=11375 setup.cfg is parsed using ConfigParser.py, which supports multiple-line entries by embedding a newline followed by whitespace. If that doesn't work for some reason, that's another bug; let me know and I'll look into it.