From mboxrd@z Thu Jan 1 00:00:00 1970 From: ludo@gnu.org (Ludovic =?utf-8?Q?Court=C3=A8s?=) Subject: Re: [PATCH] Add php Date: Mon, 14 Nov 2016 13:59:44 +0100 Message-ID: <87y40m8ben.fsf@gnu.org> References: <20161030130828.3797d37d@polymos.lepiller.eu> <20161030175105.1f6eeff2@polymos.lepiller.eu> <87ins9s9y1.fsf@duckhunt.i-did-not-set--mail-host-address--so-tickle-me> <20161102224052.7ec98d2d@lepiller.eu> <87eg2k8xp2.fsf@gnu.org> <20161111173123.51375f43@polymos.lepiller.eu> <87eg2etms7.fsf@kirby.i-did-not-set--mail-host-address--so-tickle-me> <87bmxitmdi.fsf@kirby.i-did-not-set--mail-host-address--so-tickle-me> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:50075) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1c6GrW-0006pC-Vk for guix-devel@gnu.org; Mon, 14 Nov 2016 07:59:51 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1c6GrT-0001hq-3k for guix-devel@gnu.org; Mon, 14 Nov 2016 07:59:51 -0500 In-Reply-To: <87bmxitmdi.fsf@kirby.i-did-not-set--mail-host-address--so-tickle-me> (Marius Bakke's message of "Mon, 14 Nov 2016 09:57:13 +0000") 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: Marius Bakke Cc: guix-devel@gnu.org Hi Marius, Marius Bakke skribis: > Marius Bakke writes: > >> Julien Lepiller writes: >> >>>> Could you send an updated patch to address those issues? Then we can >>>> happily apply it. >>> here is the updated patch. I let the tests be done because it doesn't >>> harm, but it does no good either, so feel free to disable them if you >>> prefer. >> >> Hello! I've gone through this package and made some improvements. I also >> managed to enable the test suite, but had to resort to strace to figure >> out what was going on (/bin/sh calls..). > > Oops, here is a version that actually passes `guix lint`. > > From 83b2913549784e5d3bafd29641e314b308f7df11 Mon Sep 17 00:00:00 2001 > From: Julien Lepiller > Date: Fri, 11 Nov 2016 15:18:29 +0100 > Subject: [PATCH] gnu: Add php. > > * gnu/packages/php.scm: New file. > * gnu/local.mk (GNU_SYSTEM_MODULES): Add it. Thanks for the update! > + ;;"bcmath/libbcmath" So this one is very different from =E2=80=98bc=E2=80=99? > + ;;"fileinfo/libmagic" ; This is a patched ve= rsion of libmagic. Here adding a dependency on =E2=80=98file=E2=80=99 instead (the package tha= t provides libmagic) didn=E2=80=99t cut it, right? > + (add-before 'check 'set-test-environment > + (lambda _ > + ;; Without this variable, `make test' passes regardless of = failures. > + (setenv "REPORT_EXIT_STATUS" "1") > + #t))) > + #:test-target "test" > + #:tests? #t)) Do you have more info as to what fails? I wouldn=E2=80=99t be surprised if there were lots of issues having to do with the use of /bin/sh, host name lookups, and other things not available in the build environment. Could you gather more info as to what fails so that we can either fix it, or at least convince ourselves that these are not serious issues and document them in a comment? Thanks Julien & Marius! Ludo=E2=80=99.