FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

bpo-46315: Add ifdef HAVE_ feature checks for WASI compatibility (GH-30507) by tiran · Pull Request #30507 · python/cpython · GitHub

/ cpython Public

bpo-46315: Add ifdef HAVE_ feature checks for WASI compatibility (GH-30507) - #30507

Merged
tiran merged 1 commit into
python:mainfrom
tiran:bpo-46315-wasi
Jan 13, 2022
Merged

bpo-46315: Add ifdef HAVE_ feature checks for WASI compatibility (GH-30507)#30507
tiran merged 1 commit into
python:mainfrom
tiran:bpo-46315-wasi

Conversation

tiran commented Jan 9, 2022
edited by bedevere-bot
Loading

Copy link
Copy Markdown
Member
  • add check for HAVE_CLOCK
  • use HAVE_PTHREAD_H to guard include of pthread.h
  • guard getpid() with HAVE_GETPID
  • guard clock() with HAVE_CLOCK
  • add checks for HAVE_UMASK and HAVE_CHMOD
  • guard include of setjmp.h
  • use signal emulation on WASI

https://bugs.python.org/issue46315

erlend-aasland left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

lgtm

* add check for HAVE_CLOCK
* use HAVE_PTHREAD_H to guard include of pthread.h
* guard getpid() with HAVE_GETPID
* guard clock() with HAVE_CLOCK
* add checks for HAVE_UMASK and HAVE_CHMOD
* guard include of setjmp.h
* use signal emulation on WASI
tiran changed the title bpo-46315: Add ifdef HAVE_ feature checks for WASI compatibility bpo-46315: Add ifdef HAVE_ feature checks for WASI compatibility (GH-30507) Jan 13, 2022
tiran merged commit a6ca8ee into python:main Jan 13, 2022
tiran deleted the bpo-46315-wasi branch January 13, 2022 08:46
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants


Back | FazBrowse Home | New Git URL