From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mark H Weaver Subject: Re: [PATCH] gnu: Add ruby 1.8.7 Date: Sun, 26 Oct 2014 21:43:22 -0400 Message-ID: <87siiap9s5.fsf@yeeloong.lan> References: <20141026203111.GA20849@thebird.nl> <87wq7mpfs0.fsf@yeeloong.lan> <20141026234436.GA21721@thebird.nl> Mime-Version: 1.0 Content-Type: text/plain Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:57433) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XiZMC-0004Wk-2q for guix-devel@gnu.org; Sun, 26 Oct 2014 21:44:33 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1XiZM6-0003q2-PB for guix-devel@gnu.org; Sun, 26 Oct 2014 21:44:28 -0400 In-Reply-To: <20141026234436.GA21721@thebird.nl> (Pjotr Prins's message of "Mon, 27 Oct 2014 00:44:36 +0100") 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: Pjotr Prins Cc: guix-devel@gnu.org Pjotr Prins writes: > On Sun, Oct 26, 2014 at 07:33:51PM -0400, Mark H Weaver wrote: >> Pjotr Prins writes: >> > * gnu/packages/ruby.scm: Added older Ruby 1.8.7 >> >> According to our conventions, this should be: >> >> * gnu/packages/ruby.scm (ruby-1.8): New variable. > > Is a new package a variable? 'ruby-1.8' is a Scheme variable that's bound to a package object. I admit that I'm not fond of this convention -- I'd prefer "New package" -- but it's what we've been doing, anyway. > We used the same message with ruby2. Where can I find these > conventions, btw? In broad strokes, we use the GNU changelog conventions: http://www.gnu.org/prep/standards/html_node/Change-Logs.html but with some minor formatting differences. However, the best thing is to look in the commit history for examples close to what you're doing. There is brief mention of this in the "Submitting Patches" section of the HACKING file in the source tree. >> It looks like the 'native-search-paths' field needs to be overridden, >> since the existing one has "2.1.3" in it. > > I don't think it is used by 1.8.7. Does that matter? Yes, otherwise users who install ruby 1.8.7 will be advised to set GEM_PATH to $HOME/.guix-profile/lib/ruby/gems/2.1.3. If setting GEM_PATH is not appropriate when using ruby 1.8.7, then you should override it here to be the empty list, like so: (native-search-paths '()) Thanks, Mark