From mboxrd@z Thu Jan 1 00:00:00 1970 From: Paul van der Walt Subject: Re: [PATCH] gnu: Add ghc-doctest. Date: Thu, 22 Oct 2015 14:58:50 +0200 Message-ID: <87twpjqctx.fsf@denknerd.org> References: <1445483479-19104-1-git-send-email-ericbavier@openmailbox.org> <1445484780-25990-1-git-send-email-ericbavier@openmailbox.org> Content-Type: text/plain Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:34090) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZpFSM-0008S5-2J for guix-devel@gnu.org; Thu, 22 Oct 2015 08:58:58 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ZpFSI-00009e-LX for guix-devel@gnu.org; Thu, 22 Oct 2015 08:58:57 -0400 Received: from mx01.mykolab.com ([95.128.36.1]:21780 helo=mx-out03.mykolab.com) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZpFSI-00009Y-9m for guix-devel@gnu.org; Thu, 22 Oct 2015 08:58:54 -0400 In-reply-to: <1445484780-25990-1-git-send-email-ericbavier@openmailbox.org> List-Id: "Development of GNU Guix and the GNU System distribution." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org Sender: guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org To: ericbavier@openmailbox.org Cc: guix-devel@gnu.org On 2015-10-22 at 05:32, quoth ericbavier@openmailbox.org: > Testing for and with doctest is tricky. I'm not sure your comment about > missing test frameworks is accurate; it looks, from what I can tell, like all > the necessary inputs are present. > > I believe the issue is that tests that call the 'doctest' function may not > (probably do not) include the "-package-db" argument in the parameter list > given, but since haskell-build-system relies on the fact that cabal passes > that argument to GHC, it is essential that that information get there somehow. > I would say it's a deficiency of most package's build systems that they do not > take this situation into account. I see, i finally understand the issue. That sounds exactly right, indeed. > So, either the tests files can be patched to include "-package-db" in the GHC > arguments given to the 'doctest' function, or GHC_PACKAGE_PATH can be defined > appropriately before the 'check' phase (at that point in the build cabal will > not complain, as it would if GHC_PACKAGE_PATH were defined during the > 'configure' phase). I think this is unnecessarily complex, especially given the upcoming native-search-path patch. > It looks to me like tests are disabled for all packages that use doctest in > this patch series: > > ghc-trifecta > ghc-lens > ghc-wai-logger > ghc-parsers > ghc-semigroupoids > ghc-comonad > ghc-distributive > ghc-http-types > ghc-unix-time > ghc-iproute > > I would suggest that tests for these packages simply be disabled (and the > corresponding test-only inputs be removed) for the time being. The pending > GHC native-search-path patches should trivially resolve this issue, at which > time tests for these packages can be revisited. I have indeed simply removed the test-only inputs on these packages, and fixed the comments to correspond to the real problem. Thanks, p.