From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ben Woodcroft Subject: Re: [Patch] ruby-spring Date: Thu, 2 Jun 2016 09:03:54 +1000 Message-ID: <574F69DA.7080803@uq.edu.au> References: <87twhexms6.fsf@mailerver.i-did-not-set--mail-host-address--so-tickle-me> <574ED9E0.7070505@uq.edu.au> <87d1o0ybjj.fsf@mailerver.i-did-not-set--mail-host-address--so-tickle-me> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:55262) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1b8FBG-0001k4-DK for guix-devel@gnu.org; Wed, 01 Jun 2016 19:04:07 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1b8FBC-0003Gl-5x for guix-devel@gnu.org; Wed, 01 Jun 2016 19:04:05 -0400 Received: from mailhub1.soe.uq.edu.au ([130.102.132.208]:41310 helo=newmailhub.uq.edu.au) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1b8FBB-0003E7-Is for guix-devel@gnu.org; Wed, 01 Jun 2016 19:04:02 -0400 In-Reply-To: <87d1o0ybjj.fsf@mailerver.i-did-not-set--mail-host-address--so-tickle-me> 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" To: Matthew Jordan Cc: guix-devel On 02/06/16 05:05, Matthew Jordan wrote: > Good Day, It is cloudy and cold outside here and the air conditioning is being its usual overzealous self. But at least we won the football. I hope it is a good day for you too. > I have updated the synopsis and the description. Please see the > following comment also; > >> + (arguments >> + `(#:test-target "test:unit" >> + #:phases >> + (modify-phases %standard-phases >> + (add-before 'check 'remove-bump >> + (lambda _ >> + (substitute* "spring.gemspec" >> + (("gem.add_development_dependency 'bump'") "")) >> + (substitute* "Rakefile" >> + (("require \\\"bump/tasks\\\"") "")) >> + #t))))) > The files spring.gemspec and Rakefile do not exist. Not in the gem they don't, but I changed the source field to download the release from GitHub. Unfortunately not including test files in the gem is not uncommon for Ruby libraries (especially rails ones), but we should endeavour to include them where possible. > +(define-public ruby-spring > + (package > + (name "ruby-spring") > + (version "1.7.1") > + (source > + (origin > + (method url-fetch) > + (uri (rubygems-uri "spring" version)) > + (sha256 > + (base32 > + "165f6n6am36md1lzaivcg9bx09f7iph4vab0f8av2rlza4yi791p")))) > + (build-system ruby-build-system) > + (arguments > + `(#:tests? #f)) ;; no Rakefile found We should add tests as I suggested in the last email, so this doesn't matter now, but for future reference comments at the end of lines like this by convention are only prefixed by a single ';' not ';;' as you have done here. > + (synopsis > + "Spring is a Rails application preloader") I think it looks better if the synopsis is kept on one line, if possible. > + (description > + "Spring is a Rails application preloader. It speeds up development by > +keeping your application running in the background so you don't need to boot > +it every time you run a test, rake task or migration.") s/your/the rails/, otherwise looks good. > + (home-page"https://github.com/rails/spring") > + (license expat))) Shall we keep the same license: prefix in rails.scm as ruby.scm? Then here it would be "license:expat" with appropriate changes to the top of the file. Thanks, ben