From 1c79891026c57046f5ac596080ea60c515e0560a Mon Sep 17 00:00:00 2001 From: "Miss Islington (bot)" <31488909+miss-islington@users.noreply.github.com> Date: Wed, 30 Jan 2019 09:36:51 -0800 Subject: [PATCH] bpo-35717: Fix KeyError exception raised when using enums and compile (GH-11523) (GH-11669) MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit https://bugs.python.org/issue17467 (cherry picked from commit 1fd06f1eca80dcbf3a916133919482a8327f3da4) Co-authored-by: Rémi Lapeyre --- Lib/enum.py | 2 +- Lib/test/test_enum.py | 9 +++++++++ Misc/ACKS | 1 + .../Library/2019-01-11-17-56-15.bpo-35717.6TDTB_.rst | 2 ++ 4 files changed, 13 insertions(+), 1 deletion(-) create mode 100644 Misc/NEWS.d/next/Library/2019-01-11-17-56-15.bpo-35717.6TDTB_.rst diff --git a/Lib/enum.py b/Lib/enum.py index 782d37433a6e..8984cac60955 100644 --- a/Lib/enum.py +++ b/Lib/enum.py @@ -427,7 +427,7 @@ class EnumMeta(type): if module is None: try: module = sys._getframe(2).f_globals['__name__'] - except (AttributeError, ValueError) as exc: + except (AttributeError, ValueError, KeyError) as exc: pass if module is None: _make_class_unpicklable(enum_class) diff --git a/Lib/test/test_enum.py b/Lib/test/test_enum.py index b221045328db..fdc3dd9c38a0 100644 --- a/Lib/test/test_enum.py +++ b/Lib/test/test_enum.py @@ -1866,6 +1866,15 @@ class TestEnum(unittest.TestCase): REVERT_ALL = "REVERT_ALL" RETRY = "RETRY" + def test_empty_globals(self): + # bpo-35717: sys._getframe(2).f_globals['__name__'] fails with KeyError + # when using compile and exec because f_globals is empty + code = "from enum import Enum; Enum('Animal', 'ANT BEE CAT DOG')" + code = compile(code, "", "exec") + global_ns = {} + local_ls = {} + exec(code, global_ns, local_ls) + class TestOrder(unittest.TestCase): diff --git a/Misc/ACKS b/Misc/ACKS index 027e01307225..193592290f0e 100644 --- a/Misc/ACKS +++ b/Misc/ACKS @@ -895,6 +895,7 @@ Glenn Langford Andrew Langmead Wolfgang Langner Detlef Lannert +Rémi Lapeyre Soren Larsen Amos Latteier Piers Lauder diff --git a/Misc/NEWS.d/next/Library/2019-01-11-17-56-15.bpo-35717.6TDTB_.rst b/Misc/NEWS.d/next/Library/2019-01-11-17-56-15.bpo-35717.6TDTB_.rst new file mode 100644 index 000000000000..7cae1d1c82c7 --- /dev/null +++ b/Misc/NEWS.d/next/Library/2019-01-11-17-56-15.bpo-35717.6TDTB_.rst @@ -0,0 +1,2 @@ +Fix KeyError exception raised when using enums and compile. Patch +contributed by Rémi Lapeyre. -- 2.47.3