Address review comments.

.github/workflows/build.yaml

@@ -28,7 +28,7 @@ jobs:
       - name: Install dependencies (macOS)
         if: matrix.os == 'macos-latest'
         run: |
-          brew install qt@5 python@3.8
+          brew install qt@5 python@3.12
           echo PATH=/usr/local/opt/qt@5/bin:$PATH >> ${GITHUB_ENV}
       - run: qmake
       - run: make

README.md

@@ -11,7 +11,7 @@ Requirements
 ------------
 
 * Qt 5.1.0 or newer (Qt 6.x also supported)
-* Python 3.3.0 or newer
+* Python 3.8.0 or newer
 
 
 Building
@@ -28,7 +28,7 @@ make install
 To build against a specific Python version, use:
 
 ```
-qmake PYTHON_CONFIG=python3.3-config   # use "qmake6" for Qt 6
+qmake PYTHON_CONFIG=python3.8-config   # use "qmake6" for Qt 6
 make
 make install
 ```

docs/index.rst

@@ -1169,7 +1169,7 @@ Building PyOtherSide
 The following build requirements have to be satisfied to build PyOtherSide:
 
 * Qt 5.1.0 or newer (Qt 6.x also supported)
-* Python 3.3.0 or newer
+* Python 3.8.0 or newer
 
 If you have the required build-dependencies installed, building and installing
 the PyOtherSide plugin should be as simple as:
@@ -1185,7 +1185,7 @@ pass a suitable ``python-config`` to ``qmake`` at configure time:
 
 .. code-block:: sh
 
-    qmake PYTHON_CONFIG=python3.3-config   # For Qt 6, use "qmake6"
+    qmake PYTHON_CONFIG=python3.8-config   # For Qt 6, use "qmake6"
     make
     make install
 
@@ -1199,6 +1199,8 @@ ChangeLog
 Version UNRELEASED (YYYY-MM-DD)
 -------------------------------
 
+* Dropped support for Python < 3.8 (PR#131)
+* Added support for Python 3.12 (PR#131)
 * Support for Qt 6.5 and newer (backwards-incompatible ``Q_RETURN_ARG()`` change) (fixes #128)
 
 Version 1.6.0 (2022-08-05)

src/qrc_importer.py

@@ -35,19 +35,14 @@ def exec_module(self, module):
         exec(code, module.__dict__)
 
 
-class PyOtherSideQtRCImporter(abc.MetaPathFinder, abc.SourceLoader):
+class PyOtherSideQtRCImporter(abc.MetaPathFinder):
     def find_spec(self, fullname, path, target=None):
         if path is None:
             fname = self.get_filename(fullname)
             if fname:
                 return spec_from_loader(fullname, PyOtherSideQtRCLoader(fname))
         return
 
-    def find_module(self, fullname, path):
-        if path is None or all(x.startswith('qrc:') for x in path):
-            if self.get_filename(fullname):
-                return self
-
     def get_filename(self, fullname):
         basename = fullname.replace('.', '/')
 
@@ -60,10 +55,4 @@ def get_filename(self, fullname):
                 if pyotherside.qrc_is_file(filename[len('qrc:'):]):
                     return filename
 
-    def get_data(self, path):
-        return pyotherside.qrc_get_file_contents(path[len('qrc:'):])
-
-    def module_repr(self, m):
-        return "<module '{}' from '{}'>".format(m.__name__, m.__file__)
-
 sys.meta_path.append(PyOtherSideQtRCImporter())