Use of deprecated / removed pkg_resources

[loqs]

pkg_resources pkg_resources was deprecated in Setuptools v67.5.0. It was removed in Setuptools v82.0.0. It is used in barectf/config_parse_common.py. There is a pkg_resources migration guide.

[icobg]

Hi, use this patch for Python 3.11+

--- a/barectf/config_parse_common.py 2023-05-17 02:38:55.000000000 +0300
+++ b/barectf/config_parse_common.py 2026-03-28 18:09:56.630399079 +0200
@@ -21,7 +21,7 @@

TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE

SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.

-import pkg_resources
+from importlib import resources
import collections
import jsonschema # type: ignore
import os.path
@@ -126,7 +126,7 @@

validate as well as a schema short ID.

class _SchemaValidator:
def init(self, subdirs: Iterable[str]):

•    schemas_dir = pkg_resources.resource_filename(__name__, 'schemas')
  

•    schemas_dir = str(resources.files(__name__) / "schemas")
     self._store: Dict[str, str] = {}
  
     for subdir in subdirs:
  

@@ -350,7 +350,7 @@
self._include_dirs = copy.copy(include_dirs)

     if with_pkg_include_dir:

•        self._include_dirs.append(pkg_resources.resource_filename(__name__, f'include/{major_version}'))
  

•        self._include_dirs.append(str(resources.files(__name__) / f"include/{major_version}"))
  
     self._ignore_include_not_found = ignore_include_not_found
     self._include_stack: List[str] = []
  

[loqs]

@icobg please consider opening a merge request for your fix.
Edit:
Python 3.8's importlib.resources lacks files:

diff --git a/barectf/config_parse_common.py b/barectf/config_parse_common.py
index 61fff44..b3f95a8 100644
--- a/barectf/config_parse_common.py
+++ b/barectf/config_parse_common.py
@@ -21,7 +21,11 @@
 # TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE
 # SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
 
-import pkg_resources
+try:
+    from importlib import resources
+    getattr(resources, "files")
+except AttributeError:
+    import importlib_resources as resources
 import collections
 import jsonschema  # type: ignore
 import os.path
@@ -126,7 +130,7 @@ _MapNode = MutableMapping[str, Any]
 # validate as well as a schema short ID.
 class _SchemaValidator:
     def __init__(self, subdirs: Iterable[str]):
-        schemas_dir = pkg_resources.resource_filename(__name__, 'schemas')
+        schemas_dir = str(resources.files(__name__) / "schemas")
         self._store: Dict[str, str] = {}
 
         for subdir in subdirs:
@@ -350,7 +354,7 @@ class _Parser:
         self._include_dirs = copy.copy(include_dirs)
 
         if with_pkg_include_dir:
-            self._include_dirs.append(pkg_resources.resource_filename(__name__, f'include/{major_version}'))
+            self._include_dirs.append(str(resources.files(__name__) / f"include/{major_version}"))
 
         self._ignore_include_not_found = ignore_include_not_found
         self._include_stack: List[str] = []
diff --git a/pyproject.toml b/pyproject.toml
index 1e19544..6afc1a1 100644
--- a/pyproject.toml
+++ b/pyproject.toml
@@ -54,6 +54,7 @@ packages = [{include = 'barectf'}]
 
 [tool.poetry.dependencies]
 python = '^3.8'
+importlib-resources = { version = ">=1.3", python = "<3.9" }
 termcolor = '^1.1'
 pyyaml = '^6.0'
 jsonschema = '^3.2'

[icobg]

Sorry, for my late response. I hope one of the project’s developers will do it.

[Mirochill]

I checked the current master branch and this appears to be fixed already by commit e16d289 (Replace deprecated pkg_resources with importlib.resources). barectf/config_parse_common.py now uses importlib.resources.files() / as_file() and no longer imports pkg_resources.

The latest tag I can see, v3.1.2, still has the old pkg_resources.resource_filename() calls, so this looks like a release/publication gap rather than a missing change on master.

[eepp]

Yes sorry!

We fixed this upstream indeed.

We'll release next week.

[loqs]

The latest tag I can see, v3.1.2, still has the old pkg_resources.resource_filename() calls, so this looks like a release/publication gap rather than a missing change on master.

e16d289 was authored on April 2 while this bug report was opened on Feb 27.

@eepp will you close this issue with the next release or would you like me to?

[eepp]

I will close!