From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:49916) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1de73G-0002gE-L6 for guix-patches@gnu.org; Sat, 05 Aug 2017 17:56:09 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1de73C-0004rG-PQ for guix-patches@gnu.org; Sat, 05 Aug 2017 17:56:06 -0400 Received: from debbugs.gnu.org ([208.118.235.43]:41140) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1de73C-0004qo-L0 for guix-patches@gnu.org; Sat, 05 Aug 2017 17:56:02 -0400 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1de73C-0005vG-3B for guix-patches@gnu.org; Sat, 05 Aug 2017 17:56:02 -0400 Subject: [bug#27438] [PATCH] Specify native search path for all ruby packages Resent-Message-ID: Date: Sat, 5 Aug 2017 22:55:52 +0100 From: Christopher Baines Message-ID: <20170805225552.19c95c6d@cbaines.net> In-Reply-To: <794765de-67a1-040b-3877-a19bfe83183f@uq.edu.au> References: <0c3c92c6-05ca-bb36-2f24-69077033e91f@cbaines.net> <878tkliix9.fsf@gnu.org> <6a666d6f-f2c2-d786-8fda-422ab28867ca@cbaines.net> <20170716183705.18b5025e@cbaines.net> <20170722110630.19b5fce0@cbaines.net> <794765de-67a1-040b-3877-a19bfe83183f@uq.edu.au> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; boundary="Sig_/psk0ngvS=LI/00_jEqZYtnd"; protocol="application/pgp-signature" List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-patches-bounces+kyle=kyleam.com@gnu.org Sender: "Guix-patches" To: Ben Woodcroft Cc: 27438@debbugs.gnu.org --Sig_/psk0ngvS=LI/00_jEqZYtnd Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: quoted-printable On Sat, 5 Aug 2017 13:59:56 +1000 Ben Woodcroft wrote: > Hi Chris, sorry for the delay on this. No problem :) > On 22/07/17 20:06, Christopher Baines wrote: > > On Thu, 20 Jul 2017 09:39:13 +1000 > > Ben Woodcroft wrote: > > =20 > >> Hi Chris, > >> > >> > >> [..] > >> > >> What happens to the default gems that come bundled with ruby > >> itself? I'm interpreting from your patch that these will not be > >> available? =20 > > They seem to be: =20 > OK, excellent. >=20 > >> In general, except for some special circumstances, we don't support > >> old versions of software. To fix the issue that you are > >> encountering properly with nokogiri probably requires new package > >> definitions using "package-with-ruby-2.3" or similar to be made, I > >> suppose. Ludo did some nice work making this easier (see > >> https://lists.gnu.org/archive/html/guix-patches/2017-04/msg00126.html), > >> but I worry in general about the resources required to support > >> older Ruby versions. WDYT? =20 > > I'm not aware of any particular problems if you are working with the > > package definitions in Guile, as it should be possible to make them > > use the single ruby version that you want. > > > > With the guix environment command I posted: > > > > guix environment --pure --ad-hoc ruby-nokogiri ruby@2.1 -- ruby > > -e "puts require 'nokogiri'" > > > > It would be ideal if there was some way of telling guix environment > > to rewrite the package definitions of all packages to use ruby@2.1 > > in place of whatever ruby they might be using. =20 > Is "package-mapping" sufficient? I don't think so. The ruby used is in the case of the ruby-build-system is an argument to the build system, so you need to traverse part of the dependency graph, altering the arguments of packages using the ruby-build-system. Or, perhaps do the transformation at a lower level abstraction than the package record... > >> Perhaps I'm slow, but what are the advantages of the "vendor_ruby" > >> method over exporting multiple GEM_PATHs as Ludo and I suggested? > >> Changing the directory seems like a heavier touch and so more > >> likely to misbehave. WDYT? =20 > > I agree that it is heavier in some sense, but I like the simplicity > > of getting rid of the version from the path. > > > > The best documentation I've found for this is the NEWS of the > > release where it was added [1]. While Guix blurs the lines between > > the "package system" and the "user", using this vendor directory > > might come in useful. > > > > 1: http://svn.ruby-lang.org/repos/ruby/tags/v1_8_7/NEWS =20 > Ah, OK. I hadn't realised there was support for this baked into Ruby=20 > itself. Seems obvious in hindsight. >=20 > If all Ruby dependencies build with this change, then the change > seems reasonable to me, details aside. Ok, does anyone know a good process for testing if lots of packages build? I think I've heard of Hydra building branches? --Sig_/psk0ngvS=LI/00_jEqZYtnd Content-Type: application/pgp-signature Content-Description: OpenPGP digital signature -----BEGIN PGP SIGNATURE----- iQKTBAEBCgB9FiEEPonu50WOcg2XVOCyXiijOwuE9XcFAlmGPuhfFIAAAAAALgAo aXNzdWVyLWZwckBub3RhdGlvbnMub3BlbnBncC5maWZ0aGhvcnNlbWFuLm5ldDNF ODlFRUU3NDU4RTcyMEQ5NzU0RTBCMjVFMjhBMzNCMEI4NEY1NzcACgkQXiijOwuE 9XdVKg/+Lj1EDfiYAoRN8+SrvmbJ7a+jAkfILbxFHcm9n99tD7IgGGcTApTun/is TKCoLZs+rcVtdhS4GToU4QCKaoAyAg5BbWT2r/lumpy7YP5g9DRKQOQzgy0EcXvK 66MHoat524tzQqBrEj1gQsSy41ia+0ABb1VVqWgC2JAlv5gZ3n1UFfpc+MghzCQv vWzZahd9E1l8PfBDAFr4QTiU0WEoX4uIL2EGly+pi9lVkePSY24YzIxXyTPs+9xM Fj0ul4cfGnfqDNTcG5lrRJ9IOmxl5W3Si1VDMXIVcmrZaBWJyDbYId2VCD4xqnlB 7uCoTNubHrlIVgStym5nnJ5nE+PKx/cKa7QXwWiy6F9Fx8wgixBa2jKMNbJYir1k hFRAgtx4X1h0+foFU+4W2JWFWsn5gO0QWJwDPXmJ25Y+jRgkdkjkCXHlcnDfwid9 6EBZdPu/gHinLfroUt6NCbH60mdS/d3OUZWCMnG3FG6rm2IVxEqtzaUtBPhRxkAR toHhrnzXvvFfzT1k/vuiwN6FcJVKmFMqJ6EWBINGIbBAk8sa6t3uZvvvmHOK7nEF hf9W5hoFVa+qtTMGywTxD2YnDEvflt2gdSEL2DJhGrfkdELgIYq0XoSu3Nr11bq1 jAH/5lUzRbE0dgeoGCIqS9LLAH5QCouhtZUJd0xOknP4e24NBCk= =nuhU -----END PGP SIGNATURE----- --Sig_/psk0ngvS=LI/00_jEqZYtnd--