Skip to content
GitLab
Menu
Projects
Groups
Snippets
Help
Help
Support
Community forum
Keyboard shortcuts
?
Submit feedback
Sign in / Register
Toggle navigation
Menu
Open sidebar
Glasgow Haskell Compiler
Packages
Cabal
Commits
3007be18
Commit
3007be18
authored
Nov 02, 2016
by
tulcod
Committed by
GitHub
Nov 02, 2016
Browse files
Merge pull request #4072 from abooij/pr4065-fixup
Deduplicate version checks in cabal-testsuite
parents
acc460dc
df647b6b
Changes
3
Hide whitespace changes
Inline
Side-by-side
Cabal/doc/developing-packages.rst
View file @
3007be18
...
...
@@ -1443,7 +1443,7 @@ executables when the executable is run (or even lazily during
execution), while static libraries (``.a`` files on Linux/OSX, ``.lib``
files on Windows) get linked against the executable at compile time.
Foreign libraries only work with GHC 7.
4
and later.
Foreign libraries only work with GHC 7.
8
and later.
A typical stanza for a foreign library looks like
...
...
cabal-testsuite/PackageTests/ForeignLibs/Check.hs
View file @
3007be18
...
...
@@ -16,9 +16,9 @@ import Distribution.Version
import
PackageTests.PackageTester
-- Foreign libraries don't work with GHC 7.
2
and earlier
-- Foreign libraries don't work with GHC 7.
6
and earlier
suite
::
TestM
()
suite
=
whenGhcVersion
(
>=
mkVersion
[
7
,
4
])
.
withPackageDb
$
do
suite
=
whenGhcVersion
(
>=
mkVersion
[
7
,
8
])
.
withPackageDb
$
do
cabal_install
[]
dist_dir
<-
distDir
pkg_dir
<-
packageDir
...
...
cabal-testsuite/PackageTests/Tests.hs
View file @
3007be18
...
...
@@ -58,8 +58,7 @@ tests config = do
tc
"AutogenModules/SrcDist"
PackageTests
.
AutogenModules
.
SrcDist
.
Check
.
suite
-- Test that foreign libraries work
tc
"ForeignLibs"
.
whenGhcVersion
(
>=
mkVersion
[
7
,
8
])
$
PackageTests
.
ForeignLibs
.
Check
.
suite
tc
"ForeignLibs"
PackageTests
.
ForeignLibs
.
Check
.
suite
---------------------------------------------------------------------
-- * Test suite tests
...
...
Write
Preview
Supports
Markdown
0%
Try again
or
attach a new file
.
Attach a file
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Cancel
Please
register
or
sign in
to comment