Hi,

This is supposed to fix the missing unit test modules in the darcs 2.4
tarball, part of issue1757. It also contains a patch that makes the release
script build and run the unit tests, and buid with witnesses, before
approving of a release tarball.

Reinier

3 patches for repository http://darcs.net/:

Sat Mar 13 23:14:28 CET 2010  Reinier Lamers <tux_rocker@reinier.de>
  * Add missing unit test modules to darcs.cabal

Sat Mar 13 23:54:54 CET 2010  Reinier Lamers <tux_rocker@reinier.de>
  * Test release tarballs more thoroughly in release.sh

Sat Mar 13 23:56:11 CET 2010  Reinier Lamers <tux_rocker@reinier.de>
  * Also add PatchInfo test module to darcs.cabal

Attachments

• add-missing-unit-test-modules-to-darcs_cabal.dpatch
• unnamed

On Sat, Mar 13, 2010 at 23:28:02 +0000, Reinier Lamers wrote:
> Sat Mar 13 23:14:28 CET 2010  Reinier Lamers <tux_rocker@reinier.de>
>   * Add missing unit test modules to darcs.cabal
> 
> Sat Mar 13 23:54:54 CET 2010  Reinier Lamers <tux_rocker@reinier.de>
>   * Test release tarballs more thoroughly in release.sh
> 
> Sat Mar 13 23:56:11 CET 2010  Reinier Lamers <tux_rocker@reinier.de>
>   * Also add PatchInfo test module to darcs.cabal

I've pushed these.  Nothing out of the ordinary there...

Add missing unit test modules to darcs.cabal
--------------------------------------------
> +  -- list all unit test modules not exported by libdarcs; otherwise Cabal won't
> +  -- include them in the tarball
> +  other-modules:    Darcs.Test.Email
> +                    Darcs.Test.Patch.Check
> +                    Darcs.Test.Patch.QuickCheck
> +                    Darcs.Test.Patch.Test
> +                    Darcs.Test.Patch.Unit

This sort of thing kinda frustrates me about Cabal... or rather, what
really gets on my nerves is when I have non-exposed modules in my
libraries, and then when building something which depends on my package,
it gets a mysterious linker error...

-- 
Eric Kow <http://www.nltg.brighton.ac.uk/home/Eric.Kow>
PGP Key ID: 08AC04F9

This patch bundle (with 3 patches) was just applied to the repository http://darcs.net/.
This message was brought to you by DarcsWatch
http://darcswatch.nomeata.de/repo_http:__darcs.net_.html#bundle-b817a08535a2406abebbe5b355e7c3b6884fb01a

This patch bundle (with 3 patches) was just applied to the repository http://darcs.net/reviewed.
This message was brought to you by DarcsWatch
http://darcswatch.nomeata.de/repo_http:__darcs.net_reviewed.html#bundle-b817a08535a2406abebbe5b355e7c3b6884fb01a