From mboxrd@z Thu Jan 1 00:00:00 1970 From: Pjotr Prins Subject: Re: [PATCH] build: ruby: Rewrite build system to use gem archives. Date: Tue, 18 Aug 2015 16:55:58 +0200 Message-ID: <20150818145558.GA24785@thebird.nl> References: <87d1zcfmyn.fsf@izanagi.i-did-not-set--mail-host-address--so-tickle-me> <87lhd9nlpk.fsf@elephly.net> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:53270) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZRiKX-0002rZ-Sn for guix-devel@gnu.org; Tue, 18 Aug 2015 10:57:39 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ZRiKT-0000GV-2N for guix-devel@gnu.org; Tue, 18 Aug 2015 10:57:37 -0400 Received: from mail.thebird.nl ([95.154.246.10]:47359) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZRiKS-0000Fq-TZ for guix-devel@gnu.org; Tue, 18 Aug 2015 10:57:33 -0400 Content-Disposition: inline In-Reply-To: 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: "Thompson, David" Cc: guix-devel On Tue, Aug 18, 2015 at 08:34:44AM -0400, Thompson, David wrote: > making packaging better for everyone. So, if we have to disable some > test suites to get our Ruby support off the ground, so be it. It is actually not a big deal for gem writers to include their tests (most have them). Amounts to 2-3 lines in the gemspec, e.g. https://github.com/pjotrp/bioruby-vcf/blob/master/bio-vcf.gemspec#L31 For rapid progress, I think it is a good idea for Guix to disable tests for a gem when they are not included, and at the same time suggest the authors to include tests in their next release (as per the rubygems spec). With luck we can enable testing again down the line. It won't always happen, but I think we can aim for 70%+ of gems to have their tests included. Pj.