Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

xmlrpclib, PEP291, "yield" #40764

Closed
anthonybaxter mannequin opened this issue Aug 16, 2004 · 5 comments
Closed

xmlrpclib, PEP291, "yield" #40764

anthonybaxter mannequin opened this issue Aug 16, 2004 · 5 comments
Assignees
Labels
stdlib Python modules in the Lib dir

Comments

@anthonybaxter
Copy link
Mannequin

anthonybaxter mannequin commented Aug 16, 2004

BPO 1009803
Nosy @loewis

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/loewis'
closed_at = <Date 2004-08-22.16:05:18.000>
created_at = <Date 2004-08-16.04:28:59.000>
labels = ['library']
title = 'xmlrpclib, PEP291, "yield"'
updated_at = <Date 2004-08-22.16:05:18.000>
user = 'https://bugs.python.org/anthonybaxter'

bugs.python.org fields:

activity = <Date 2004-08-22.16:05:18.000>
actor = 'loewis'
assignee = 'loewis'
closed = True
closed_date = None
closer = None
components = ['Library (Lib)']
creation = <Date 2004-08-16.04:28:59.000>
creator = 'anthonybaxter'
dependencies = []
files = []
hgrepos = []
issue_num = 1009803
keywords = []
message_count = 5.0
messages = ['22075', '22076', '22077', '22078', '22079']
nosy_count = 3.0
nosy_names = ['loewis', 'anthonybaxter', 'quiver']
pr_nums = []
priority = 'high'
resolution = 'fixed'
stage = None
status = 'closed'
superseder = None
type = None
url = 'https://bugs.python.org/issue1009803'
versions = []

@anthonybaxter
Copy link
Mannequin Author

anthonybaxter mannequin commented Aug 16, 2004

PEP-291 says that xmlrpclib should be 1.5.2 compatible.

This patch:
revision 1.31
date: 2003/10/31 13:49:36; author: loewis; state:
Exp; lines: +66 -2
Patch bpo-531629: Add multicall support.

however, introduced code with the 'yield' keyword. This
is completely and utterly non-1.5.2 compatible. Either
the PEP (and the comment in the file) should be
updated, or the code should be rewritten to not use yield.

Martin? You checked in the patch, what's your opinion?

@anthonybaxter anthonybaxter mannequin closed this as completed Aug 16, 2004
@anthonybaxter anthonybaxter mannequin assigned loewis Aug 16, 2004
@anthonybaxter anthonybaxter mannequin added the stdlib Python modules in the Lib dir label Aug 16, 2004
@anthonybaxter anthonybaxter mannequin closed this as completed Aug 16, 2004
@anthonybaxter anthonybaxter mannequin assigned loewis Aug 16, 2004
@anthonybaxter anthonybaxter mannequin added the stdlib Python modules in the Lib dir label Aug 16, 2004
@loewis
Copy link
Mannequin

loewis mannequin commented Aug 20, 2004

Logged In: YES
user_id=21627

After discussion with /F, I have now bumped the minimum
supported version to 2.1; fixed in xmlrpclib.py 1.34 and
pep-0291.txt 1.11. Fredrik still has 1.5.2 installations he
needs to support, but thinks he can manage maintenance on
xmlrpclib for these on his own.

@anthonybaxter
Copy link
Mannequin Author

anthonybaxter mannequin commented Aug 20, 2004

Logged In: YES
user_id=29957

The code, however, still has 'yield' in it, and is therefore
_not_ 2.1 compatible.

@quiver
Copy link
Mannequin

quiver mannequin commented Aug 21, 2004

Logged In: YES
user_id=671362

Should I close bug bpo-842600 as a duplicate?
(Although it was submitted last November.)

@loewis
Copy link
Mannequin

loewis mannequin commented Aug 22, 2004

Logged In: YES
user_id=21627

This is now fixed in xmlrpclib.py 1.35.

@ezio-melotti ezio-melotti transferred this issue from another repository Apr 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
stdlib Python modules in the Lib dir
Projects
None yet
Development

No branches or pull requests

0 participants