Skip to content

Diff-2.0.0 - #47

Merged
ddssff merged 4 commits into
masterfrom
2.0.0
Sep 6, 2026
Merged

Diff-2.0.0#47
ddssff merged 4 commits into
masterfrom
2.0.0

Conversation

@ddssff

@ddssff ddssff commented Sep 4, 2026

Copy link
Copy Markdown
Member

Thanks to Xavier Góngora and Facundo Domínguez who have put some real polish on this package. This version has significant performance optimizations (see PR #25, #30 and #37, leading to a ~2.4x and ~8.5x getDiff improvement on benchmarks) and removes groupBy' from the API (see #26.)

Thanks to Xavier Góngora and Facundo Domínguez who have put some real
polish on this package.  This version has significant performance
optimizations (see PR #25, #30 and #37, leading to a ~2.4x and ~8.5x
getDiff improvement on benchmarks) and removes groupBy' from the API
(see #26.)
@ninioArtillero ninioArtillero mentioned this pull request Sep 4, 2026
@ninioArtillero

Copy link
Copy Markdown
Collaborator

@ddssff Seems like the CHANGELOG.md update is still missing, you can use what you think is relevant from what I proposed in #46

ddssff and others added 2 commits September 4, 2026 09:58
* Minor version bump to 2.0.1 to allow new hackage upload
Diff.cabal declared cabal-version: 3.14 solely to use the new
extra-files field for bundling Diff-liquidhaskell.cabal in the
sdist. However, such version is not supported by Hackage and
could break dependency solving for some comsumers.

Switch to extra-source-files, which achieves the same sdist
packaging with no functional difference for this use case, and
drop cabal-version back to 1.18.

This also requires switching the license field from the SPDX
identifier BSD-3-Clause back to the legacy BSD3, since SPDX license
expressions aren't recognized before cabal-version: 2.2.
@ninioArtillero

Copy link
Copy Markdown
Collaborator

We just saw https://hackage.haskell.org/package/Diff-2.0.0/reports/2 reporting:
rejecting: Diff-2.0.0 (unsupported spec-version 3.14). This version was introduced in #45 to be able to use the extra-files field, but perhaps we can use extra-source-files instead (which has the small downside of tracking the modification time of the file for cabal build, which should be OK here). This last field is supported since 1.6, so perhaps we can revert to 1.18.

#48 stacks such changes onto this PR

Lower cabal-version back to 1.18
@ddssff
ddssff merged commit a56c189 into master Sep 6, 2026
13 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.

2 participants