Repository navigation
BytesWarnings triggerred by test suite #64562
Description
| BPO | 20363 |
|---|---|
| Nosy | @tarekziade, @merwok, @ambv, @berkerpeksag, @serhiy-storchaka |
| Files |
Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.
Show more details
GitHub fields:
assignee = 'https://github.com/serhiy-storchaka'
closed_at = <Date 2014-03-12.20:59:53.685>
created_at = <Date 2014-01-23.09:47:31.664>
labels = ['type-bug', 'library']
title = 'BytesWarnings triggerred by test suite'
updated_at = <Date 2014-03-12.20:59:53.684>
user = 'https://bugs.python.org/Arfrever'bugs.python.org fields:
activity = <Date 2014-03-12.20:59:53.684>
actor = 'eric.araujo'
assignee = 'serhiy.storchaka'
closed = True
closed_date = <Date 2014-03-12.20:59:53.685>
closer = 'eric.araujo'
components = ['Library (Lib)']
creation = <Date 2014-01-23.09:47:31.664>
creator = 'Arfrever'
dependencies = []
files = ['33715']
hgrepos = []
issue_num = 20363
keywords = ['patch']
message_count = 19.0
messages = ['208903', '208990', '208993', '208995', '208997', '209009', '209011', '209043', '209263', '209266', '209269', '210415', '210416', '210418', '211178', '211185', '213292', '213293', '213296']
nosy_count = 7.0
nosy_names = ['tarek', 'eric.araujo', 'Arfrever', 'lukasz.langa', 'python-dev', 'berker.peksag', 'serhiy.storchaka']
pr_nums = []
priority = 'normal'
resolution = 'fixed'
stage = 'resolved'
status = 'closed'
superseder = None
type = 'behavior'
url = 'https://bugs.python.org/issue20363'
versions = ['Python 3.3', 'Python 3.4']
Activity
berkerpeksag commentedon Jan 26, 2014
Thanks for the review, Arfrever and Serhiy. Here's a new patch.
Arfrever commentedon Jan 26, 2014
The new patch has %s and %r reversed in Lib/distutils/command/register.py.
berkerpeksag commentedon Jan 26, 2014
Ah, my bad. Thanks Arfrever.
serhiy-storchaka commentedon Feb 6, 2014
LGTM.
python-dev commentedon Feb 6, 2014
New changeset 791674a74e47 by Serhiy Storchaka in branch '3.3':
Issue bpo-20363. Fixed BytesWarning triggerred by test suite.
http://hg.python.org/cpython/rev/791674a74e47
New changeset a4431dce107a by Serhiy Storchaka in branch 'default':
Issue bpo-20363. Fixed BytesWarning triggerred by test suite.
http://hg.python.org/cpython/rev/a4431dce107a
serhiy-storchaka commentedon Feb 6, 2014
Thank you Arfrever for your report. Thank you Berker for your patch.
merwok commentedon Feb 13, 2014
Thanks for applying the patch. distutils tests don’t cover 100% of the codebase; did you test manually that the behavior of the changed code was still correct?
serhiy-storchaka commentedon Feb 13, 2014
At least it wasn't changed.
As I see, affected line is used to output debugging message (disabled by default). It produces different result in 2.7 and 3.x ("----------xxx----------" vs "----------b'xxx'----------") due to the difference between 2.x str and 3.x bytes.
merwok commentedon Mar 12, 2014
I don’t understand one thing: you said the output wasn’t changed, then show an example of changed output: “"----------xxx----------" vs "----------b'xxx'----------"”.
The “data” that is displayed is supposed to be text; showing “b'” and “'” is a regression for the 3.x line.
Arfrever commentedon Mar 12, 2014
Output is different between Python 2 and 3, but not different between different versions in Python 3.
Output in Python 3 is e.g. "----------b'xxx'----------" both before and after commit 791674a74e47.
I suggest to create a separate issue for discussion about this message in distutils.
merwok commentedon Mar 12, 2014
Thanks for correcting me. I created bpo-20900.