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

Jump to bottom

Threading and THREAD_SAFE for AIX #81190

Open
aixtools opened this issue May 22, 2019 · 3 comments
Open

Threading and THREAD_SAFE for AIX #81190

aixtools opened this issue May 22, 2019 · 3 comments
Labels
3.8 only security fixes 3.9 only security fixes 3.10 only security fixes build The build process and cross-build

Comments

@aixtools
Copy link
Contributor

BPO 37009
Nosy @aixtools, @kadler, @isidentical

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 = None
created_at = <Date 2019-05-22.12:01:09.176>
labels = ['build', '3.8', '3.9', '3.10']
title = 'Threading and THREAD_SAFE for AIX'
updated_at = <Date 2020-11-18.10:12:55.031>
user = 'https://github.com/aixtools'

bugs.python.org fields:

activity = <Date 2020-11-18.10:12:55.031>
actor = 'christian.heimes'
assignee = 'none'
closed = False
closed_date = None
closer = None
components = ['Build']
creation = <Date 2019-05-22.12:01:09.176>
creator = 'Michael.Felt'
dependencies = []
files = []
hgrepos = []
issue_num = 37009
keywords = []
message_count = 3.0
messages = ['343184', '365359', '365898']
nosy_count = 3.0
nosy_names = ['Michael.Felt', 'kadler', 'BTaskaya']
pr_nums = []
priority = 'normal'
resolution = None
stage = None
status = 'open'
superseder = None
type = None
url = 'https://bugs.python.org/issue37009'
versions = ['Python 3.8', 'Python 3.9', 'Python 3.10']
@aixtools
Copy link
Contributor Author

For years Python includes the file /usr/include/pthread.h. The AIX documentation states that this needs to be the first include file included OR the define _THREAD_SAFE needs to be defined.

As this may have been true, might still be true, or might have never been true - this patch assures that the define is added to BASECFLAGS for AIX - and will not be forgotten during builds.

It may be advisable to include this in backports. This "conditional requirement" has been accurate for over 20 years.

@aixtools aixtools added 3.7 (EOL) end of life 3.8 only security fixes build The build process and cross-build labels May 22, 2019
@isidentical
Copy link
Sponsor Member

this patch assures that the define is added to BASECFLAGS for AIX - and will not be forgotten during builds.

@Micheal.Felt I can't see an attached patch or PR, FYI.

@aixtools
Copy link
Contributor Author

aixtools commented Apr 7, 2020

Yes, looks like I need to find that. thx for the reminder.

@tiran tiran added 3.9 only security fixes 3.10 only security fixes and removed 3.7 (EOL) end of life labels Nov 18, 2020
@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
3.8 only security fixes 3.9 only security fixes 3.10 only security fixes build The build process and cross-build
Projects
Threading issues 🧵
Status: No status
Milestone
No milestone
Development

No branches or pull requests
3 participants
@tiran @aixtools @isidentical