* tests: add linking warnings/error Signed-off-by: Henry Schreiner <henryschreineriii@gmail.com> * tests: check pedantic C++14 * fix: some pedantic warnings Signed-off-by: Henry Schreiner <henryschreineriii@gmail.com> * feat: add py::mod_gil_used() Signed-off-by: Henry Schreiner <henryschreineriii@gmail.com> * fix: use not_supported() Signed-off-by: Henry Schreiner <henryschreineriii@gmail.com> * style: pre-commit fixes * Update CMakeLists.txt * fix: remove the true/false parameter from mod_gil_not_used Signed-off-by: Henry Schreiner <henryschreineriii@gmail.com> * style: pre-commit fixes * Update tests/CMakeLists.txt * fix: deprecate mod_gil_not_used(bool) instead of removing it Keep the bool constructor public and mark it deprecated, so existing code that spells mod_gil_not_used(false) still compiles. mod_gil_used() now sets the flag through its friendship, which keeps the deprecated constructor the only bool overload and stops pybind11 warning against itself. Also use the py:: alias in exo_planet_pybind11.cpp to match its sibling home_planet_very_lonely_traveler.cpp. Assisted-by: ClaudeCode:claude-opus-5 * fix: add trailing comma to macros picked up in rebase test_smart_ptr.cpp and standalone_enum_module.cpp came from master after this branch was written, so they still invoked variadic macros with no variadic argument. That is a pedantic error below C++20. Assisted-by: ClaudeCode:claude-opus-5 --------- Signed-off-by: Henry Schreiner <henryschreineriii@gmail.com> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
154 lines
4.1 KiB
Python
154 lines
4.1 KiB
Python
from __future__ import annotations
|
|
|
|
import os
|
|
import subprocess
|
|
import sys
|
|
from textwrap import dedent
|
|
|
|
import pytest
|
|
|
|
DIR = os.path.abspath(os.path.dirname(__file__))
|
|
MAIN_DIR = os.path.dirname(os.path.dirname(DIR))
|
|
WIN = sys.platform.startswith("win32") or sys.platform.startswith("cygwin")
|
|
|
|
|
|
@pytest.mark.parametrize("parallel", [False, True])
|
|
@pytest.mark.parametrize("std", [11, 0])
|
|
def test_simple_setup_py(monkeypatch, tmpdir, parallel, std):
|
|
monkeypatch.chdir(tmpdir)
|
|
monkeypatch.syspath_prepend(MAIN_DIR)
|
|
|
|
(tmpdir / "setup.py").write_text(
|
|
dedent(
|
|
f"""\
|
|
import sys
|
|
sys.path.append({MAIN_DIR!r})
|
|
|
|
from setuptools import setup, Extension
|
|
from pybind11.setup_helpers import build_ext, Pybind11Extension
|
|
|
|
std = {std}
|
|
|
|
ext_modules = [
|
|
Pybind11Extension(
|
|
"simple_setup",
|
|
sorted(["main.cpp"]),
|
|
cxx_std=std,
|
|
),
|
|
]
|
|
|
|
cmdclass = dict()
|
|
if std == 0:
|
|
cmdclass["build_ext"] = build_ext
|
|
|
|
|
|
parallel = {parallel}
|
|
if parallel:
|
|
from pybind11.setup_helpers import ParallelCompile
|
|
ParallelCompile().install()
|
|
|
|
setup(
|
|
name="simple_setup_package",
|
|
cmdclass=cmdclass,
|
|
ext_modules=ext_modules,
|
|
)
|
|
"""
|
|
),
|
|
encoding="ascii",
|
|
)
|
|
|
|
(tmpdir / "main.cpp").write_text(
|
|
dedent(
|
|
"""\
|
|
#include <pybind11/pybind11.h>
|
|
|
|
int f(int x) {
|
|
return x * 3;
|
|
}
|
|
PYBIND11_MODULE(simple_setup, m, pybind11::mod_gil_used()) {
|
|
m.def("f", &f);
|
|
}
|
|
"""
|
|
),
|
|
encoding="ascii",
|
|
)
|
|
|
|
out = subprocess.check_output(
|
|
[sys.executable, "setup.py", "build_ext", "--inplace"],
|
|
)
|
|
if not WIN:
|
|
assert b"-g0" in out
|
|
out = subprocess.check_output(
|
|
[sys.executable, "setup.py", "build_ext", "--inplace", "--force"],
|
|
env=dict(os.environ, CFLAGS="-g"),
|
|
)
|
|
if not WIN:
|
|
assert b"-g0" not in out
|
|
|
|
# Debug helper printout, normally hidden
|
|
print(out)
|
|
for item in tmpdir.listdir():
|
|
print(item.basename)
|
|
|
|
assert (
|
|
len([f for f in tmpdir.listdir() if f.basename.startswith("simple_setup")]) == 1
|
|
)
|
|
assert len(list(tmpdir.listdir())) == 4 # two files + output + build_dir
|
|
|
|
(tmpdir / "test.py").write_text(
|
|
dedent(
|
|
"""\
|
|
import simple_setup
|
|
assert simple_setup.f(3) == 9
|
|
"""
|
|
),
|
|
encoding="ascii",
|
|
)
|
|
|
|
subprocess.check_call(
|
|
[sys.executable, "test.py"], stdout=sys.stdout, stderr=sys.stderr
|
|
)
|
|
|
|
|
|
def test_intree_extensions(monkeypatch, tmpdir):
|
|
monkeypatch.syspath_prepend(MAIN_DIR)
|
|
|
|
from pybind11.setup_helpers import intree_extensions
|
|
|
|
monkeypatch.chdir(tmpdir)
|
|
root = tmpdir
|
|
root.ensure_dir()
|
|
subdir = root / "dir"
|
|
subdir.ensure_dir()
|
|
src = subdir / "ext.cpp"
|
|
src.ensure()
|
|
relpath = src.relto(tmpdir)
|
|
(ext,) = intree_extensions([relpath])
|
|
assert ext.name == "ext"
|
|
subdir.ensure("__init__.py")
|
|
(ext,) = intree_extensions([relpath])
|
|
assert ext.name == "dir.ext"
|
|
|
|
|
|
def test_intree_extensions_package_dir(monkeypatch, tmpdir):
|
|
monkeypatch.syspath_prepend(MAIN_DIR)
|
|
|
|
from pybind11.setup_helpers import intree_extensions
|
|
|
|
monkeypatch.chdir(tmpdir)
|
|
root = tmpdir / "src"
|
|
root.ensure_dir()
|
|
subdir = root / "dir"
|
|
subdir.ensure_dir()
|
|
src = subdir / "ext.cpp"
|
|
src.ensure()
|
|
(ext,) = intree_extensions([src.relto(tmpdir)], package_dir={"": "src"})
|
|
assert ext.name == "dir.ext"
|
|
(ext,) = intree_extensions([src.relto(tmpdir)], package_dir={"foo": "src"})
|
|
assert ext.name == "foo.dir.ext"
|
|
subdir.ensure("__init__.py")
|
|
(ext,) = intree_extensions([src.relto(tmpdir)], package_dir={"": "src"})
|
|
assert ext.name == "dir.ext"
|
|
(ext,) = intree_extensions([src.relto(tmpdir)], package_dir={"foo": "src"})
|
|
assert ext.name == "foo.dir.ext"
|