Avoid a potential import of fcntl

sebastinas · sebastinas · commit 9ce99705b98c · 2022-04-25T22:59:29.000+02:00
diff --git a/bpython/config.py b/bpython/config.py
@@ -1,7 +1,7 @@
 # The MIT License
 #
 # Copyright (c) 2009-2015 the bpython authors.
-# Copyright (c) 2015-2020 Sebastian Ramacher
+# Copyright (c) 2015-2022 Sebastian Ramacher
 #
 # Permission is hereby granted, free of charge, to any person obtaining a copy
 # of this software and associated documentation files (the "Software"), to deal
@@ -35,9 +35,13 @@
 from xdg import BaseDirectory
 
 from .autocomplete import AutocompleteModes
-from .curtsiesfrontend.parse import CNAMES
 
 default_completion = AutocompleteModes.SIMPLE
+# All supported letters for colors for themes
+#
+# Instead of importing it from .curtsiesfrontend.parse, we define them here to
+# avoid a potential import of fcntl on Windows.
+COLOR_LETTERS = tuple("krgybmcwd")
 
 
 class UnknownColorCode(Exception):
@@ -381,7 +385,7 @@ def load_theme(
             colors[k] = theme.get("syntax", k)
         else:
             colors[k] = theme.get("interface", k)
-        if colors[k].lower() not in CNAMES:
+        if colors[k].lower() not in COLOR_LETTERS:
             raise UnknownColorCode(k, colors[k])
 
     # Check against default theme to see if all values are defined
diff --git a/bpython/curtsiesfrontend/parse.py b/bpython/curtsiesfrontend/parse.py
@@ -8,11 +8,12 @@
 )
 from functools import partial
 
+from ..config import COLOR_LETTERS
 from ..lazyre import LazyReCompile
 
 
 COLORS = CURTSIES_COLORS + ("default",)
-CNAMES = dict(zip("krgybmcwd", COLORS))
+CNAMES = dict(zip(COLOR_LETTERS, COLORS))
 # hack for finding the "inverse"
 INVERSE_COLORS = {
     CURTSIES_COLORS[idx]: CURTSIES_COLORS[
