Skip to content

Hack to avoid dependency on obsolete ed25519 package#4

Open
cajun-rat wants to merge 2 commits intouptane:masterfrom
cajun-rat:fix/debian-stable
Open

Hack to avoid dependency on obsolete ed25519 package#4
cajun-rat wants to merge 2 commits intouptane:masterfrom
cajun-rat:fix/debian-stable

Conversation

@cajun-rat
Copy link
Collaborator

No description provided.

Copy link
Member

@tkfu tkfu left a comment

Choose a reason for hiding this comment

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

LGTM

@tkfu
Copy link
Member

tkfu commented Oct 14, 2025

Ah, CI pipeline failed. Looks like it's because the pipeline is ultimately installing the venv from requirements-dev.{in,txt} Can you change those to use pynacl as well?

Copy link
Member

@tkfu tkfu left a comment

Choose a reason for hiding this comment

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

Changes also need to be done in requirements-dev.in and requirements-dev.txt

This can be tested locally with:

    docker build -t tuf-test-vectors-ci -f ci/gitlab/Dockerfile.tuf-test-vectors  .
    docker run -v "$PWD:$PWD" -w "$PWD" -it tuf-test-vectors-ci make lint test
@cajun-rat
Copy link
Collaborator Author

@tkfu It turns out to be quite an adventure to fix this. Hopefully this fixes it. The CI doesn't run automatically since I'm not a maintainer for this. Any chance you could add me?

@cajun-rat cajun-rat requested a review from tkfu October 27, 2025 21:13
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.

2 participants