search

kadena-io / chainweb-node

Chainweb: A Proof-of-Work Parallel-Chain Architecture for Massive Throughput
https://docs.kadena.io/basics/whitepapers/overview
BSD 3-Clause "New" or "Revised" License
248 stars 95 forks source link

remove upper bound on crypton-connection #1975

Closed larskuhtz closed 5 months ago

larskuhtz commented 5 months ago

The PR removes / fixes the upper bound on crypto-connection. This fixes the build with the latest versions of the HTTP and TLS related packages.

Along the way the PR also cleans up some issues and inconsistency in the cabal file -- which are small and trivial changes.

larskuhtz commented 5 months ago

The unit test failures look concerning. I wonder if those are related to the new version of unordered-containers. In best case it's just test code that makes illegal assumptions about ordering of the items in the container.

larskuhtz commented 5 months ago

OK, I think, the unit test failures are independent of the code changes in this PR. The have been appearing on other branches, too. I am going to investigate those based on master.

larskuhtz commented 5 months ago

Closing this because it got merged into #1976