Skip to content

fix: replace pkg_resources by importlib#134

Merged
bernhardreiter merged 9 commits into
csaf-tools:mainfrom
wagner-intevation:fix/pkgresources
May 4, 2026
Merged

fix: replace pkg_resources by importlib#134
bernhardreiter merged 9 commits into
csaf-tools:mainfrom
wagner-intevation:fix/pkgresources

Conversation

@wagner-intevation

Copy link
Copy Markdown
Member

pkg_resources is deprecated and no longer available in current python versions
replace it by calls to stdlib importlib

pkg_resources is deprecated and no longer available in current python
versions
replace it by calls to stdlib importlib
  use the Traversable object more directly
Comment thread cvrf2csaf/cvrf2csaf.py Outdated
bernhardreiter and others added 3 commits April 30, 2026 12:02
  use importlib_resources' Traversable objects directly in two cases.
  change variable names to be more explicit about what they contain
convert the Traversable to a Path. if package is a zip, this is a tempfile
Comment thread cvrf2csaf/common/utils.py Outdated
 * remove python 3.9, which is end-of-life.
 * add active python 3.14.
  * add requirements for building lxml, as it seems that the wheel
    is not there yet for python 3.14
  * change suffix _FILE to _TRAV where a Traversable object is used.
    To be more consistent.

@bernhardreiter bernhardreiter left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM & works

@bernhardreiter
bernhardreiter requested a review from reiterl May 4, 2026 08:25

@reiterl reiterl left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@bernhardreiter
bernhardreiter merged commit 95f0ca1 into csaf-tools:main May 4, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants