From mboxrd@z Thu Jan 1 00:00:00 1970 From: Manolis Ragkousis Subject: Re: [PATCH] gnu: Add fontconfig-path-max. Date: Mon, 4 Jul 2016 01:24:08 +0300 Message-ID: References: <7b313a7566d04932f94fb33e7a63c222@openmailbox.org> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:38148) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bJpoH-0003zA-No for guix-devel@gnu.org; Sun, 03 Jul 2016 18:24:18 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bJpoC-0004bg-Ek for guix-devel@gnu.org; Sun, 03 Jul 2016 18:24:16 -0400 Received: from mail-wm0-x243.google.com ([2a00:1450:400c:c09::243]:35110) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bJpoC-0004bM-6u for guix-devel@gnu.org; Sun, 03 Jul 2016 18:24:12 -0400 Received: by mail-wm0-x243.google.com with SMTP id a66so17733646wme.2 for ; Sun, 03 Jul 2016 15:24:11 -0700 (PDT) In-Reply-To: <7b313a7566d04932f94fb33e7a63c222@openmailbox.org> 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: rennes@openmailbox.org Cc: guix-devel@gnu.org Hello Rennes, I am sorry for the long delay, I somehow missed the patch. Leo thank you for telling me. On 06/18/16 22:02, rennes@openmailbox.org wrote: > Hello Guix team, > > i'm doing tests whith GNU Guix on GNU Hurd, compiling fontconfig and > there is an error during compilation: Once again thank you :-) > > a) fontconfig uses the constant PATH_MAX. > > Reviewing the documentation about the treatment of constant for Hurd; > i've attached a patch for review. > > References: > https://www.gnu.org/software/hurd/community/gsoc/project_ideas/maxpath.html > https://www.gnu.org/software/hurd/hurd/porting/guidelines.html > > and i've a couple of questions about: > > a) How Guix identify if it is a Linux or Hurd system at compile or > install the package?. > b) i searches in ML an example, but i not found. Currently we apply patches regardless of where it is running. But one way to check the system is like in (gnu packages base) (define* (glibc-for-target #:optional (target (or (%current-target-system) (%current-system)))) "Return the glibc for TARGET, GLIBC/LINUX for a Linux host or GLIBC/HURD for a Hurd host" (match target ((or "i586-pc-gnu" "i586-gnu") glibc/hurd) (_ glibc/linux))) If %current-target-system is not #f then we are cross-building for the value inside it. %current-system has the value of the system we are running on. Now regarding the patch, what is the status on upstream? Are those fontconfig patches present in fontconfig upstream? Other than that, it looks good to me. Thank you for helping on this, Manolis.