Skip to content

Add pybind11 patches - #326

Closed
KyleFromKitware wants to merge 2 commits into
Kitware:masterfrom
KyleFromKitware:pybind11_use_gilstate
Closed

Add pybind11 patches#326
KyleFromKitware wants to merge 2 commits into
Kitware:masterfrom
KyleFromKitware:pybind11_use_gilstate

Conversation

@KyleFromKitware

Copy link
Copy Markdown
Contributor

This adds a set of patches to pybind11 that fixpybind/pybind11#1276 (the changes I've made can be seen in pybind/pybind11#1286)

@hughed2

Copy link
Copy Markdown
Contributor

LGTM

@dstoup

Copy link
Copy Markdown
Collaborator

Jenkins test this please

@dstoup

Copy link
Copy Markdown
Collaborator

Looks good to me too. I will merge once the dashboard cycles.

@kwcvrobot

Copy link
Copy Markdown
Collaborator

@kwcvrobot

Copy link
Copy Markdown
Collaborator

@kwcvrobot

Copy link
Copy Markdown
Collaborator

@KyleFromKitware

Copy link
Copy Markdown
ContributorAuthor

Closed because I am going to take a new approach that doesn't involve patching pybind11.

Sign up for freeto 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.

Double-nested gil_scoped_release in a pythread causes fatal interpreter error

4 participants

@KyleFromKitware@hughed2@dstoup@kwcvrobot