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

Missing semicolon at Modules/posixsubprocess.c:4511 #57548

Closed
nneonneo mannequin opened this issue Nov 4, 2011 · 4 comments
Closed

Missing semicolon at Modules/posixsubprocess.c:4511 #57548

nneonneo mannequin opened this issue Nov 4, 2011 · 4 comments
Labels
build The build process and cross-build extension-modules C modules in the Modules dir

Comments

@nneonneo
Copy link
Mannequin

nneonneo mannequin commented Nov 4, 2011

BPO 13339

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 = None
closed_at = <Date 2011-11-04.06:41:20.098>
created_at = <Date 2011-11-04.04:34:23.841>
labels = ['extension-modules', 'build']
title = 'Missing semicolon at Modules/posixsubprocess.c:4511'
updated_at = <Date 2012-05-16.05:07:32.551>
user = 'https://bugs.python.org/nneonneo'

bugs.python.org fields:

activity = <Date 2012-05-16.05:07:32.551>
actor = 'python-dev'
assignee = 'rosslagerwall'
closed = True
closed_date = <Date 2011-11-04.06:41:20.098>
closer = 'rosslagerwall'
components = ['Extension Modules']
creation = <Date 2011-11-04.04:34:23.841>
creator = 'nneonneo'
dependencies = []
files = []
hgrepos = []
issue_num = 13339
keywords = []
message_count = 4.0
messages = ['146995', '146996', '146997', '160803']
nosy_count = 3.0
nosy_names = ['nneonneo', 'rosslagerwall', 'python-dev']
pr_nums = []
priority = 'normal'
resolution = 'fixed'
stage = 'resolved'
status = 'closed'
superseder = None
type = 'compile error'
url = 'https://bugs.python.org/issue13339'
versions = ['Python 3.1', 'Python 3.2', 'Python 3.3', 'Python 3.4']

@nneonneo
Copy link
Mannequin Author

nneonneo mannequin commented Nov 4, 2011

Line 4511 of Modules/posixsubprocess.c is missing a semicolon, so it would not compile successfully if the relevant build flags were enabled (PYOS_OS2).

Trivial patch:
@@ -4508,7 +4508,7 @@
 static PyObject *
 posix_spawnvpe(PyObject *self, PyObject *args)
 {
-    PyObject *opath
+    PyObject *opath;
     char *path;
     PyObject *argv, *env;
     char **argvlist;

No tests needed since it's a compile error.

@nneonneo nneonneo mannequin added extension-modules C modules in the Modules dir build The build process and cross-build labels Nov 4, 2011
@python-dev
Copy link
Mannequin

python-dev mannequin commented Nov 4, 2011

New changeset fa9c69dcf31b by Ross Lagerwall in branch '3.2':
Issue bpo-13339: Fix compile error in posixmodule.c due to missing semicolon.
http://hg.python.org/cpython/rev/fa9c69dcf31b

New changeset 9e5d14aadca4 by Ross Lagerwall in branch 'default':
(Merge 3.2) Issue bpo-13339.
http://hg.python.org/cpython/rev/9e5d14aadca4

@rosslagerwall
Copy link
Mannequin

rosslagerwall mannequin commented Nov 4, 2011

Thanks!

@rosslagerwall rosslagerwall mannequin closed this as completed Nov 4, 2011
@rosslagerwall rosslagerwall mannequin self-assigned this Nov 4, 2011
@python-dev
Copy link
Mannequin

python-dev mannequin commented May 16, 2012

New changeset 91ac9c36f09e by Éric Araujo in branch 'default':
Harmonize error messages and add one missing test for bpo-13339
http://hg.python.org/distutils2/rev/91ac9c36f09e

@ezio-melotti ezio-melotti transferred this issue from another repository Apr 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build The build process and cross-build extension-modules C modules in the Modules dir
Projects
None yet
Development

No branches or pull requests

0 participants