Skip to content

Commit

Permalink
More cabal file tidying
Browse files Browse the repository at this point in the history
Remove most constraints from non-library components. I'm unsure if this
is quite right: possibly we should have bounds for any public component,
but this is significantly more work to maintain (witness the odd bound
on `sorted-list` from a test suite). I think this strikes a reasonable
balance for now.
  • Loading branch information
michaelpj committed Dec 23, 2023
1 parent c92e6fe commit 816a47a
Show file tree
Hide file tree
Showing 3 changed files with 21 additions and 18 deletions.
10 changes: 5 additions & 5 deletions lsp-test/lsp-test.cabal
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ library
build-depends:
, aeson >=2 && <2.3
, aeson-pretty ^>=0.8
, ansi-terminal >= 0.10 && < 1.1
, ansi-terminal >=0.10 && <1.1
, async ^>=2.2
, base >=4.10 && <5
, bytestring >=0.10 && <0.13
Expand All @@ -54,7 +54,7 @@ library
, conduit-parse ^>=0.2
, containers ^>=0.6
, data-default ^>=0.7
, Diff >= 0.4 && < 0.6
, Diff >=0.4 && <0.6
, directory ^>=1.3
, exceptions ^>=0.10
, filepath ^>=1.4
Expand All @@ -63,7 +63,7 @@ library
, lens-aeson ^>=1.2
, lsp ^>=2.3
, lsp-types ^>=2.1
, mtl >= 2.2 && < 2.4
, mtl >=2.2 && <2.4
, parser-combinators ^>=1.3
, process ^>=1.6
, row-types ^>=1.0
Expand Down Expand Up @@ -106,9 +106,9 @@ test-suite tests
, filepath
, hspec
, lens
, lsp ^>=2.3
, lsp
, lsp-test
, mtl <2.4
, mtl
, parser-combinators
, process
, text
Expand Down
23 changes: 13 additions & 10 deletions lsp-types/lsp-types.cabal
Original file line number Diff line number Diff line change
Expand Up @@ -76,7 +76,7 @@ library
, containers ^>=0.6
, data-default ^>=0.7
, deepseq >=1.4 && <1.6
, Diff >= 0.4 && < 0.6
, Diff >=0.4 && <0.6
, dlist ^>=1.0
, exceptions ^>=0.10
, hashable ^>=1.4
Expand All @@ -85,7 +85,7 @@ library
, lens >=5.1 && <5.3
, lens-aeson ^>=1.2
, mod ^>=0.2
, mtl >= 2.2 && < 2.4
, mtl >=2.2 && <2.4
, network-uri ^>=2.6
, prettyprinter ^>=1.7
, row-types ^>=1.0
Expand All @@ -94,11 +94,14 @@ library
, template-haskell >=2.7 && <2.22
, text >=1 && <2.2

-- This version of filepath comes with GHC 9.6, so
-- we should be able to drop this flag once we only
-- support 9.6 or higher
if flag(force-ospath)
build-depends: filepath ^>=1.4.100.0

else
build-depends: filepath
build-depends: filepath ^>=1.4

ghc-options:
-Wall -Wmissing-deriving-strategies
Expand Down Expand Up @@ -532,11 +535,11 @@ library metamodel

build-depends:
, aeson >=2
, base >=4.11 && <5
, file-embed
, lens >=4.15.2
, template-haskell
, text
, base >=4.11 && <5
, file-embed ^>=0.0.15
, lens >=5.1 && <5.3
, template-haskell >=2.7 && <2.22
, text >=1 && <2.2

executable generator
hs-source-dirs: generator
Expand Down Expand Up @@ -578,11 +581,11 @@ test-suite lsp-types-test

ghc-options: -threaded -rtsopts -with-rtsopts=-N -Wall
build-depends:
, aeson >=2.0.3.0
, aeson
, base
, filepath
, hspec
, lens >=4.15.2
, lens
, lsp-types
, network-uri
, QuickCheck
Expand Down
6 changes: 3 additions & 3 deletions lsp/lsp.cabal
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,7 @@ library
, lens >=5.1 && <5.3
, lens-aeson ^>=1.2
, lsp-types ^>=2.1
, mtl >= 2.2 && < 2.4
, mtl >=2.2 && <2.4
, prettyprinter ^>=1.7
, random ^>=1.2
, row-types ^>=1.0
Expand All @@ -88,7 +88,7 @@ executable lsp-demo-reactor-server
, aeson
, base
, co-log-core
, lens >=4.15.2
, lens
, lsp
, prettyprinter
, stm
Expand Down Expand Up @@ -132,7 +132,7 @@ test-suite lsp-test
, hspec
, lsp
, row-types
, sorted-list >=0.2.1 && <0.2.3
, sorted-list
, text
, text-rope
, unordered-containers
Expand Down

0 comments on commit 816a47a

Please sign in to comment.