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

CMake: Remember Python Version #1434

Merged
merged 1 commit into from
Jun 24, 2018
Merged

Conversation

ax3l
Copy link
Collaborator

@ax3l ax3l commented Jun 20, 2018

It is useful not only to remember the python libs and includes but also the interpreter version in cache.

If users call pybind11 throught add_subdirectories they will otherwise have no access to the selected interpreter version. The interpreter version is useful for downstream projects, e.g. to select default lib/pythonX.Y/site-packages/ install paths.

Fix pybind/cmake_example#13

ax3l pushed a commit to ax3l/openPMD-api that referenced this pull request Jun 20, 2018
Add an internal version of pybind11, version 2.2.3.

Modifications from the releases:
- add fix pybind/pybind11#1434
- remove test dirs
- remove wheel packaging (setup.py/cfg)
- remove github templates
@ax3l ax3l force-pushed the fix-addSubdirPythonVersion branch from 50a1937 to 0a59021 Compare June 20, 2018 13:26
@ax3l
Copy link
Collaborator Author

ax3l commented Jun 20, 2018

cc @dean0x7d @SylvainCorlay

@ax3l
Copy link
Collaborator Author

ax3l commented Jun 20, 2018

Is it possible the CI linux python 3.6.6 test is broken in dev, too?

@ax3l
Copy link
Collaborator Author

ax3l commented Jun 20, 2018

@jagerman is the CI broken on Python 3.6.6? -> fix proposed in #1436

ax3l pushed a commit to ax3l/openPMD-api that referenced this pull request Jun 21, 2018
Add an internal version of pybind11, version 2.2.3.

Modifications from the releases:
- add fix pybind/pybind11#1434
- remove test dirs
- remove wheel packaging (setup.py/cfg)
- remove github templates
It is useful not only to remember the python libs and includes but
also the interpreter version in cache.

If users call pybind11 throught `add_subdirectories` they will
otherwise have no access to the selected interpreter version.
The interpreter version is useful for downstream projects, e.g.
to select default `lib/pythonX.Y/site-packages/` install paths.
@ax3l ax3l force-pushed the fix-addSubdirPythonVersion branch from 0a59021 to f0601dd Compare June 21, 2018 15:02
@ax3l
Copy link
Collaborator Author

ax3l commented Jun 21, 2018

@chuckatkins any thoughts on this one as well?

@ax3l
Copy link
Collaborator Author

ax3l commented Jun 21, 2018

@jagerman @dean0x7d would be glad if we can improve in-source pybind11 projects with this addition :)

@wjakob
Copy link
Member

wjakob commented Jun 24, 2018

Thanks!

@wjakob wjakob merged commit 97b20e5 into pybind:master Jun 24, 2018
@ax3l ax3l deleted the fix-addSubdirPythonVersion branch June 24, 2018 14:02
wjakob pushed a commit that referenced this pull request Sep 11, 2018
It is useful not only to remember the python libs and includes but
also the interpreter version in cache.

If users call pybind11 throught `add_subdirectories` they will
otherwise have no access to the selected interpreter version.
The interpreter version is useful for downstream projects, e.g.
to select default `lib/pythonX.Y/site-packages/` install paths.
ax3l pushed a commit to ax3l/openPMD-api that referenced this pull request Jun 15, 2019
Update the internal version of pybind11, version 2.3.0.

This release includes several fixes, such as
  - add fix pybind/pybind11#1434

and can compute all function signatures at compile-time in
C++11 mode via `constexpr`, leading to smaller binaries.

  https://github.com/pybind/pybind11/blob/v2.3.0/docs/changelog.rst

Modifications from the releases:
  - remove test dirs
  - remove wheel packaging (setup.py/cfg)
  - remove github templates
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.

None yet

2 participants