Stop using Py_FrozenFlag

Author: Avasam

Pythonwin/dllmain.cpp

@@ -191,7 +191,7 @@ extern "C" __declspec(dllexport) int __stdcall DllMainwin32ui(HINSTANCE hInstanc
         // insert into resource chain.
         pDLL = new CDynLinkLibrary(extensionDLL);
 
-#else  // Frozen .EXE that embedds win32ui is initializing
+#else  // Frozen .EXE that embeds win32ui is initializing
         TRACE("win32ui in frozen %s initializing.\n", path);
 #endif
     }

com/pythoncom.py

@@ -2,3 +2,23 @@
 import pywintypes
 
 pywintypes.__import_pywin32_system_module__("pythoncom", globals())
+
+
+def __getattr__(name: str) -> "bool | str":
+    if name == "frozen":
+        import sys
+        import warnings
+
+        warnings.warn(
+            DeprecationWarning(
+                f"`pythoncom.frozen` used to expose `Py_FrozenFlag` from the C API. "
+                + "`Py_FrozenFlag` is deprecated since Python 3.12. "
+                + "Ever since pywin32 b200, loading the `win32com` has silently been "
+                + "replacing `pythoncom.frozen` with `sys.frozen`. "
+                + 'Use `getattr(sys, "frozen", False)` directly instead.'
+            ),
+            stacklevel=2,
+        )
+        return getattr(sys, "frozen", False)
+    else:
+        raise AttributeError(f"module {__name__!r} has no attribute {name!r}")

com/win32com/__init__.py

@@ -9,11 +9,7 @@
 import pythoncom
 import win32api
 
-# flag if we are in a "frozen" build.
-_frozen = getattr(sys, "frozen", False)
-# pythoncom dumbly defaults this to zero - we believe sys.frozen over it.
-if _frozen and not getattr(pythoncom, "frozen", 0):
-    pythoncom.frozen = sys.frozen
+__frozen: str | bool = getattr(sys, "frozen", False)
 
 # Add support for an external "COM Extensions" path.
 #  Concept is that you can register a seperate path to be used for
@@ -86,11 +82,11 @@ def SetupEnvironment():
 # (which the win32com developers do!)
 def __PackageSupportBuildPath__(package_path):
     # See if we have a special directory for the binaries (for developers)
-    if not _frozen and __build_path__:
+    if not __frozen and __build_path__:
         package_path.append(__build_path__)
 
 
-if not _frozen:
+if not __frozen:
     SetupEnvironment()
 
 # If we don't have a special __gen_path__, see if we have a gen_py as a

com/win32com/server/register.py

@@ -7,6 +7,8 @@
 
 """
 
+from __future__ import annotations
+
 import os
 import sys
 
@@ -17,6 +19,8 @@
 
 CATID_PythonCOMServer = "{B3EF80D0-68E2-11D0-A689-00C04FD658FF}"
 
+__frozen: str | bool = getattr(sys, "frozen", False)
+
 
 def _set_subkeys(keyName, valueDict, base=win32con.HKEY_CLASSES_ROOT):
     hkey = win32api.RegCreateKey(base, keyName)
@@ -207,13 +211,9 @@ def RegisterServer(
     # Set default clsctx.
     if not clsctx:
         clsctx = pythoncom.CLSCTX_INPROC_SERVER | pythoncom.CLSCTX_LOCAL_SERVER
-    # And if we are frozen, ignore the ones that don't make sense in this
-    # context.
-    if pythoncom.frozen:
-        assert sys.frozen, (
-            "pythoncom is frozen, but sys.frozen is not set - don't know the context!"
-        )
-        if sys.frozen == "dll":
+    # And if we are frozen, ignore the ones that don't make sense in this context.
+    if __frozen:
+        if __frozen == "dll":
             clsctx &= pythoncom.CLSCTX_INPROC_SERVER
         else:
             clsctx &= pythoncom.CLSCTX_LOCAL_SERVER
@@ -223,7 +223,7 @@ def RegisterServer(
         # nod to Gordon's installer - if sys.frozen and sys.frozendllhandle
         # exist, then we are being registered via a DLL - use this DLL as the
         # file name.
-        if pythoncom.frozen:
+        if __frozen:
             if hasattr(sys, "frozendllhandle"):
                 dllName = win32api.GetModuleFileName(sys.frozendllhandle)
             else:
@@ -261,9 +261,8 @@ def RegisterServer(
         _remove_key(keyNameRoot + "\\InprocServer32")
 
     if clsctx & pythoncom.CLSCTX_LOCAL_SERVER:
-        if pythoncom.frozen:
-            # If we are frozen, we write "{exe} /Automate", just
-            # like "normal" .EXEs do
+        if __frozen:
+            # If we are frozen, we write "{exe} /Automate", just like "normal" .EXEs do
             exeName = win32api.GetShortPathName(sys.executable)
             command = f"{exeName} /Automate"
         else:
@@ -302,7 +301,7 @@ def RegisterServer(
         _remove_key(keyNameRoot + "\\PythonCOMPath")
 
     if addPyComCat is None:
-        addPyComCat = pythoncom.frozen == 0
+        addPyComCat = __frozen == False
     if addPyComCat:
         catids = catids + [CATID_PythonCOMServer]
 
@@ -422,7 +421,7 @@ def RegisterClasses(*classes, **flags):
         clsctx = _get(cls, "_reg_clsctx_")
         tlb_filename = _get(cls, "_reg_typelib_filename_")
         # default to being a COM category only when not frozen.
-        addPyComCat = not _get(cls, "_reg_disable_pycomcat_", pythoncom.frozen != 0)
+        addPyComCat = not _get(cls, "_reg_disable_pycomcat_", __frozen)
         addnPath = None
         if debugging:
             # If the class has a debugging dispatcher specified, use it, otherwise
@@ -455,7 +454,7 @@ def RegisterClasses(*classes, **flags):
 
             spec = moduleName + "." + cls.__name__
             # Frozen apps don't need their directory on sys.path
-            if not pythoncom.frozen:
+            if not __frozen:
                 scriptDir = os.path.split(sys.argv[0])[0]
                 if not scriptDir:
                     scriptDir = "."
@@ -664,7 +663,7 @@ def RegisterPyComCategory():
     regCat.RegisterCategories([(CATID_PythonCOMServer, 0x0409, "Python COM Server")])
 
 
-if not pythoncom.frozen:
+if not __frozen:
     try:
         win32api.RegQueryValue(
             win32con.HKEY_CLASSES_ROOT,

com/win32com/src/PythonCOM.cpp

@@ -2637,9 +2637,6 @@ PYWIN_MODULE_INIT_FUNC(pythoncom)
 
     ADD_CONSTANT(DESCKIND_FUNCDESC);
     ADD_CONSTANT(DESCKIND_VARDESC);
-    // Expose the frozen flag, as Python itself doesn't!!
-    // @prop int|frozen|1 if the host is a frozen program, else 0
-    AddConstant(dict, "frozen", Py_FrozenFlag);
 
     // And finally some gross hacks relating to DCOM
     // I'm really not sure what a better option is!
@@ -2678,7 +2675,10 @@ PYWIN_MODULE_INIT_FUNC(pythoncom)
     // @prop int|dcom|1 if the system is DCOM aware, else 0.  Only Win95 without DCOM extensions should return 0
 
     // ### ALL THE @PROPERTY TAGS MUST COME AFTER THE LAST @PROP TAG!!
-    // @property int|pythoncom|frozen|1 if the host is a frozen program, else 0
+    // @property int|pythoncom|frozen|`pythoncom.frozen` used to expose `Py_FrozenFlag` from the C API.
+    // `Py_FrozenFlag` is deprecated since Python 3.12.
+    // Ever since pywin32 b200, loading the `win32com` has silently been replacing `pythoncom.frozen` with `sys.frozen`.
+    // Use `getattr(sys, "frozen", False)` directly instead.
     // @property int|pythoncom|dcom|1 if the system is DCOM aware, else 0.  Only Win95 without DCOM extensions should
     // return 0
 

com/win32comext/mapi/src/exchangeguids.cpp

@@ -3,8 +3,7 @@
 #define USES_IID_IExchangeFolderACLs
 
 #ifndef BUILD_FREEZE
-/* In a frozen environemt, these are likely to be picked
-up by the MAPI module */
+// In a frozen environemnt, these are likely to be picked up by the MAPI module
 
 #define USES_IID_IMsgStore
 #define USES_IID_IMAPISession

pycln.toml

@@ -6,7 +6,7 @@ skip_imports = [
   "IDLEenvironment", # Injects fast_readline into the IDLE auto-indent extension
   "pythoncom", # pythoncomXX.dll loader
   "pywintypes", # pywintypesXX.dll loader
-  "win32com", # Sets pythoncom.frozen and adds modules to path based on the registry
+  "win32com", # Adds modules to path based on the registry
   "win32traceutil", # Redirects output to win32trace remote collector
   "win32ui", # Must always be imported before dde
 ]