Skip to content

Fix interspersed positional arguments in cythonize.py#2988

Merged
scoder merged 5 commits intocython:masterfrom
realead:fix_interspersed_args_in_cythonize
Jun 11, 2019
Merged

Fix interspersed positional arguments in cythonize.py#2988
scoder merged 5 commits intocython:masterfrom
realead:fix_interspersed_args_in_cythonize

Conversation

@realead
Copy link
Contributor

@realead realead commented Jun 7, 2019

This is fix for a regression in #2952: in optparse interspersed arguments are on per default, but not in argparse (I was not able to parse this information from https://docs.python.org/3.7/library/argparse.html#upgrading-optparse-code), thus the following

cythonize.py file1 -a file2

works with optparse but not argparse.

As the native-argparse support for interspersed/intermixed arguments is only available since Python3.7 (https://bugs.python.org/issue14191) we need to emulate it.

@realead realead force-pushed the fix_interspersed_args_in_cythonize branch from 8891fe3 to 6fb2d5b Compare June 8, 2019 10:55
@scoder
Copy link
Contributor

scoder commented Jun 11, 2019

Thanks!

@scoder scoder merged commit 838f26c into cython:master Jun 11, 2019
@scoder scoder added this to the 3.0 milestone Jun 11, 2019
@scoder scoder added the defect label Jun 11, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants