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.

Sign up for GitHub

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

fix: Set __file__ constant when using eval_file (#1300) #3233

Merged
merged 13 commits into from
Sep 9, 2021
Merged

fix: Set __file__ constant when using eval_file (#1300) #3233

Show file tree
Hide file tree
Changes from 6 commits
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
82dc139
Set __file__ constant when using eval_file
Skylion007 Sep 1, 2021
f76ecc7
Use const ref
Skylion007 Sep 1, 2021
5ce8222
Use a move instead
Skylion007 Sep 1, 2021
38c57fb
Revert
Skylion007 Sep 1, 2021
f7ff7d5
Improve test
Skylion007 Sep 1, 2021
e9eb2c8
Guard test with Python version
Skylion007 Sep 1, 2021
93887fe
Fix tests
Skylion007 Sep 1, 2021
1c4d86c
Dont support Python2 API
Skylion007 Sep 1, 2021
bff887c
Drop Python2 eval __file__ support
Skylion007 Sep 1, 2021
3e34089
Hack
Skylion007 Sep 1, 2021
8b3c3ae
Semisupport Python2
Skylion007 Sep 2, 2021
2d5c38f
Take2
Skylion007 Sep 2, 2021
d5e56c3
Remove Python2 support
Skylion007 Sep 9, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 4 additions & 0 deletions include/pybind11/eval.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -136,6 +136,10 @@ object eval_file(str fname, object global = globals(), object local = object())
pybind11_fail("File \"" + fname_str + "\" could not be opened!");
}

if (!global.contains("__file__")) {
global["__file__"] = std::move(fname);
}

#if PY_VERSION_HEX < 0x03000000 && defined(PYPY_VERSION)
PyObject *result = PyRun_File(f, fname_str.c_str(), start, global.ptr(),
local.ptr());
Expand Down
5 changes: 5 additions & 0 deletions tests/test_cmake_build/test.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,5 +3,10 @@

import test_cmake_build

from .. import env

if env.PY3:
Copy link
Collaborator

Choose a reason for hiding this comment

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

Without the matching guard in C++ we are as much creating a problem as we are solving one.

assert isinstance(__file__, str) # Test this is properly set

assert test_cmake_build.add(1, 2) == 3
print("{} imports, runs, and adds: 1 + 2 = 3".format(sys.argv[1]))