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.

Sign up for GitHub

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

Jump to bottom

[Patch] Make presence of SCHED_* optional #71843

Closed
EdSchouten mannequin opened this issue Jul 30, 2016 · 2 comments
Closed

[Patch] Make presence of SCHED_* optional #71843

EdSchouten mannequin opened this issue Jul 30, 2016 · 2 comments
Labels
extension-modules C modules in the Modules dir

Comments

@EdSchouten
Copy link
Mannequin

EdSchouten mannequin commented Jul 30, 2016

BPO 27656
Nosy @EdSchouten
Files
  • sched.diff: Patch for declaring SCHED_* conditionally
    		•

    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 2016-07-31.06:22:40.100>
created_at = <Date 2016-07-30.15:49:07.658>
labels = ['extension-modules']
title = '[Patch] Make presence of SCHED_* optional'
updated_at = <Date 2016-07-31.06:22:40.097>
user = 'https://github.com/EdSchouten'

    bugs.python.org fields:

    activity = <Date 2016-07-31.06:22:40.097>
actor = 'python-dev'
assignee = 'none'
closed = True
closed_date = <Date 2016-07-31.06:22:40.100>
closer = 'python-dev'
components = ['Extension Modules']
creation = <Date 2016-07-30.15:49:07.658>
creator = 'EdSchouten'
dependencies = []
files = ['43952']
hgrepos = []
issue_num = 27656
keywords = ['patch']
message_count = 2.0
messages = ['271697', '271723']
nosy_count = 2.0
nosy_names = ['python-dev', 'EdSchouten']
pr_nums = []
priority = 'normal'
resolution = 'fixed'
stage = 'resolved'
status = 'closed'
superseder = None
type = None
url = 'https://bugs.python.org/issue27656'
versions = ['Python 3.6']
    @EdSchouten
    Copy link
    Mannequin Author

    EdSchouten mannequin commented Jul 30, 2016

    The SCHED_* constants that are part of POSIX's <sched.h> are all optional:

    http://pubs.opengroup.org/onlinepubs/9699919799/basedefs/sched.h.html

    Python already declares the SCHED_SPORADIC constant as part of the POSIX module optionally, depending on whether it is present in C, but doesn't do this for the other SCHED_* constants.

    This is problematic for CloudABI (https://mail.python.org/pipermail/python-dev/2016-July/145708.html), which doesn't support any scheduler interaction at all.

    Attached is a patch to only define these if present.

    @EdSchouten EdSchouten mannequin added the extension-modules C modules in the Modules dir label Jul 30, 2016
    @python-dev
    Copy link
    Mannequin

    python-dev mannequin commented Jul 31, 2016

    New changeset 095b424127e7 by Benjamin Peterson in branch '3.5':
    all SCHED_ constants are optional (closes bpo-27656)
    https://hg.python.org/cpython/rev/095b424127e7

    New changeset 64b763290da9 by Benjamin Peterson in branch 'default':
    merge 3.5 (bpo-27656)
    https://hg.python.org/cpython/rev/64b763290da9

    @python-dev python-dev mannequin closed this as completed Jul 31, 2016
    @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
    Assignees
    No one assigned
    Labels
    extension-modules C modules in the Modules dir
    Projects
    None yet
    Milestone
    No milestone
    Development

    No branches or pull requests
    0 participants