Skip to content

Update cardano-base dependencies to reflect the retirement of cardano-crypto-tests#5494

Closed
neilmayhew wants to merge 7 commits intomasterfrom
nm/update-cardano-base
Closed

Update cardano-base dependencies to reflect the retirement of cardano-crypto-tests#5494
neilmayhew wants to merge 7 commits intomasterfrom
nm/update-cardano-base

Conversation

@neilmayhew
Copy link
Contributor

Description

cardano-crypto-tests was retired in IntersectMBO/cardano-base#590 and replaced by testlibs in cardano-crypto-{class,praos}. This PR adjusts the dependencies in ledger to use the testlibs and adds lower bounds to ensure consistency.

Checklist

  • Commits in meaningful sequence and with useful messages.
  • Tests added or updated when needed.
  • CHANGELOG.md files updated for packages with externally visible changes.
    NOTE: New section is never added with the code changes. (See RELEASING.md).
  • Versions updated in .cabal and CHANGELOG.md files when necessary, according to the
    versioning process.
  • Version bounds in .cabal files updated when necessary.
    NOTE: If bounds change in a cabal file, that package itself must have a version increase. (See RELEASING.md).
  • Code formatted (use scripts/fourmolize.sh).
  • Cabal files formatted (use scripts/cabal-format.sh).
  • CDDL files are up to date (use scripts/gen-cddl.sh)
  • hie.yaml updated (use scripts/gen-hie.sh).
  • Self-reviewed the diff.

@neilmayhew
Copy link
Contributor Author

The commit containing the srp can be removed after new cardano-base packages are released to CHaP.

@neilmayhew neilmayhew force-pushed the nm/update-cardano-base branch from 11afccb to 3513308 Compare December 23, 2025 16:28
Copy link
Collaborator

@lehins lehins left a comment

Choose a reason for hiding this comment

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

Yeah. Looks good.

@neilmayhew neilmayhew force-pushed the nm/update-cardano-base branch 3 times, most recently from cad8584 to b27b315 Compare December 23, 2025 23:31
@neilmayhew neilmayhew force-pushed the nm/update-cardano-base branch from b27b315 to e1da26b Compare December 30, 2025 23:56
@neilmayhew neilmayhew force-pushed the nm/update-cardano-base branch from e1da26b to 59ebba5 Compare January 19, 2026 19:58
This brings in the revised upper bounds in cardano-ledger-tests
@neilmayhew neilmayhew force-pushed the nm/update-cardano-base branch from 59ebba5 to f4781a0 Compare January 29, 2026 19:30
@neilmayhew neilmayhew force-pushed the nm/update-cardano-base branch from f4781a0 to 9e0f59e Compare January 29, 2026 21:42
@neilmayhew
Copy link
Contributor Author

Closing this in favour of #5553 (which includes these commits) now that IntersectMBO/cardano-base#612 is merged

@neilmayhew neilmayhew closed this Feb 2, 2026
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