GitLab

Ticket #215 (Overhaul support for packages' tests).  The update to the
TestSuite parser which stopped disallowed configurations during parsing also
broke the existing test suite with API changes.

Ticket #215 (Overhaul support for packages' tests).

Ticket #215 (Overhaul support for packages' tests).

Change the messages slightly. Make configure return the
new lbi, rather than having to re-read the lbi from file
(avoiding potential infinite loop if the IO failed).

This patch adds "ConfigFlags" to the "LocalBuildInfo" and reuses them to
perform "configureAction" when .cabal file is changed. This has
the same effect as re-running "configure" with the most recent used
set of options, which should be the sensible thing to do.

Closes #294, #477, #309 and #518.

Original patch was:
  Sun Oct  7 13:41:53 BST 2007  Thorkil Naur <naur@post11.tele.dk>
    * Fix processing of shared options
Re-recorded due to code churn

Original patch by:
  Thu Aug 21 21:19:51 MDT 2008  dbueno@gmail.com
    * Correct spelling of 'transative'.
Re-recorded due to conflict.

print a warning if repository location isn't specified and the cabal project looks like being tracked by a version control system

Currently Cabal just appends the pid to the package name-version
which makes it hard to see the actual package version number.
This patch just inserts a '-' between name-version and pid.

While here, simplify the argument list construction in
invocationAsShellScript a little bit.

We previously checked during preprocessing that there is some source
file for every module and produce a nice error message if not (rather
than having ghc --make fail later). We only looked for source files in
the source directories, ie not the target/output build dir, since the
build dir is where pre-processed files go, not original source files.
Igloo's change should allow builds where there is no original source file
at all, so long as there is some generated file in the build dir. In
particular this allows custom Setup.hs files to generate modules out of
thin air and for things to mostly work, even though the rest of the build
system is unaware about how the file got built.

Fixes hscolour'ing Config.hs in GHC

We were generating things like
    #define VERSION_ghc_prim"0.2.0.0"
rather than
    #define VERSION_ghc_prim "0.2.0.0"

I'm adding these type signatures to satisfy the "do not generalise
local let/where" rule that GHC is taking on.

The signatures are clearly correct, but I was surprised at the 
polymorphism needed.  For example

parseOptVersion :: ReadP r Version
parseOptVersion = parseQuoted ver <++ ver
  where ver :: ReadP r Version
        ver = parse <++ return noVersion
        noVersion = Version{ versionBranch=[], versionTags=[] }

Note that 'ver' really is called at two different types!  That 
in turn is because of the type of (<++)

  (<++) :: ReadP a a -> ReadP r a -> ReadP r a
  (+++) :: ReadP r a -> ReadP r a -> ReadP r a

Note the "a a" in the first arg, which is very unusual.
For example, compare the type of (+++).

Changing it to match the type of (+++) makes ReadP fail to compile,
though, so I assume it's right as it stands. But surely this deserves
a comment?!

Ticket #215 (Overhaul support for packages' tests).  Previously, the test-suite
stanza was allowed to contain a test-suite field, such as the executable stanza
parser allows.  This was removed because it is only required by legacy support
for executables and no legacy support is required for test suites.

Ticket #215 (Overhaul support for packages' tests).

Ticket #215 (Overhaul support for packages' tests).

Ticket #215 (Overhaul support for packages' tests).

The --help option output now prints only the first (if any)
long option. Because of this, --{enable,disable}-optimisation
(british spelling) is simply added to the list of options
without testing showOrParseArgs. --hyperlink-sources is now
also accepted for the haddock command.

The executable field is only for the legacy pre-Cabal-1.2 stanza syntax.

Ticket #215 (Overhaul support for packages' tests).  By parsing the test suite
stanza through an intermediate data structure, we can isolate errors due to
missing required fields in the parsing stage and avoid having error handlers
througout the code.

Ticket #215 (Overhaul support for packages' tests).

Ticket #215 (Overhaul support for packages' tests).

Ticket #215 (Overhaul support for packages' tests).  The previous output format
made it difficult to distinguish successful test suites from failed test
suites.

Ticket #215 (Overhaul support for packages' tests).  The use of the old
interface also makes it possible to log stdout and stderr without separating
the two.

Ticket #215 (Overhaul support for packages' tests).  Using a new format for the
test-suite stanza in the .cabal file, similar to the source-repository stanza.

Ticket #215 (Overhaul support for packages' tests).