From mboxrd@z Thu Jan  1 00:00:00 1970
From: Mark H Weaver <mhw@netris.org>
Subject: Re: [PATCH] Add Julia.
Date: Wed, 25 Mar 2015 17:00:55 -0400
Message-ID: <878uekpzaw.fsf@netris.org>
References: <idjoanndimb.fsf@bimsb-sys02.mdc-berlin.net>
	<87384vqtwh.fsf@netris.org> <87pp7yphom.fsf@netris.org>
	<idjvbhq1foh.fsf@bimsb-sys02.mdc-berlin.net>
	<87d23xoxp2.fsf@netris.org> <87k2y5q6k1.fsf@mango.localdomain>
Mime-Version: 1.0
Content-Type: text/plain
Return-path: <guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org>
Received: from eggs.gnu.org ([2001:4830:134:3::10]:50641)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <mhw@netris.org>) id 1YasPn-0008QB-Eb
	for guix-devel@gnu.org; Wed, 25 Mar 2015 17:00:40 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <mhw@netris.org>) id 1YasPj-0006Bl-6T
	for guix-devel@gnu.org; Wed, 25 Mar 2015 17:00:39 -0400
Received: from world.peace.net ([50.252.239.5]:34328)
	by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <mhw@netris.org>) id 1YasPj-0006Bf-1u
	for guix-devel@gnu.org; Wed, 25 Mar 2015 17:00:35 -0400
In-Reply-To: <87k2y5q6k1.fsf@mango.localdomain> (Ricardo Wurmus's message of
	"Wed, 25 Mar 2015 19:24:14 +0100")
List-Id: "Development of GNU Guix and the GNU System distribution."
	<guix-devel.gnu.org>
List-Unsubscribe: <https://lists.gnu.org/mailman/options/guix-devel>,
	<mailto:guix-devel-request@gnu.org?subject=unsubscribe>
List-Archive: <http://lists.gnu.org/archive/html/guix-devel>
List-Post: <mailto:guix-devel@gnu.org>
List-Help: <mailto:guix-devel-request@gnu.org?subject=help>
List-Subscribe: <https://lists.gnu.org/mailman/listinfo/guix-devel>,
	<mailto:guix-devel-request@gnu.org?subject=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: Ricardo Wurmus <ricardo.wurmus@mdc-berlin.de>
Cc: guix-devel <guix-devel@gnu.org>

Ricardo Wurmus <ricardo.wurmus@mdc-berlin.de> writes:

> Mark H Weaver writes:
>
>> Ricardo Wurmus <ricardo.wurmus@mdc-berlin.de> writes:
>>
>>> May I suggest adding passing the "DYNAMIC_ARCH=1" flag in openblas
>>> instead of propagating non-substitutability to Julia?  When built with
>>> "DYNAMIC_ARCH=1" the CPU type can be picked with the environment
>>> variable OPENBLAS_CORETYPE=<type>.  If I'm not mistaken this makes
>>> substitutions for openblas possible.
>>>
>>> What do you think?
>>
>> Yes, we should do this.
>
> Attached is a patch to make openblas substitutable.
>
> From 326fa66415fe8e9ed2b28a249b903f4a654b769e Mon Sep 17 00:00:00 2001
> From: Ricardo Wurmus <rekado@elephly.net>
> Date: Wed, 25 Mar 2015 19:20:11 +0100
> Subject: [PATCH] gnu: openblas: build for all supported CPUs.
>
> * gnu/packages/maths.scm (openblas)[arguments]: Pass make flag
>   "DYNAMIC_ARCH=1" to build library for all CPUs, making it substitutable.

This explanation of the purpose of "DYNAMIC_ARCH=1" belongs as a comment
in the source code, rather than in the commit log.  You might also make
it clear that this enables runtime CPU detection.

The commit log should only summarize what changed, maybe something along
these lines:

* gnu/packages/maths.scm (openblas)[arguments]: Add "DYNAMIC_ARCH=1" to
  make flags.  Remove "#:substitutable? #f".

Otherwise it looks good to me.  Please push with these changes.

    Thanks!
      Mark