From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mp2 ([2001:41d0:8:6d80::]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)) by ms0.migadu.com with LMTPS id gEiXLB3cmGBjAQAAgWs5BA (envelope-from ) for ; Mon, 10 May 2021 09:09:17 +0200 Received: from aspmx1.migadu.com ([2001:41d0:8:6d80::]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)) by mp2 with LMTPS id OGYvKB3cmGA6PgAAB5/wlQ (envelope-from ) for ; Mon, 10 May 2021 07:09:17 +0000 Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by aspmx1.migadu.com (Postfix) with ESMTPS id 1DBE311910 for ; Mon, 10 May 2021 09:09:16 +0200 (CEST) Received: from localhost ([::1]:47436 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1lg02V-0007BE-NI for larch@yhetil.org; Mon, 10 May 2021 03:09:15 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:35272) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1lg02I-0007AU-Dz for guix-patches@gnu.org; Mon, 10 May 2021 03:09:02 -0400 Received: from debbugs.gnu.org ([209.51.188.43]:45672) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1lg02I-0004h5-6T for guix-patches@gnu.org; Mon, 10 May 2021 03:09:02 -0400 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1lg02H-0003V7-Vi for guix-patches@gnu.org; Mon, 10 May 2021 03:09:01 -0400 X-Loop: help-debbugs@gnu.org Subject: [bug#48325] julia-1.6 guix Resent-From: =?UTF-8?Q?Nicol=C3=B2?= Balzarotti Original-Sender: "Debbugs-submit" Resent-CC: guix-patches@gnu.org Resent-Date: Mon, 10 May 2021 07:09:01 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 48325 X-GNU-PR-Package: guix-patches X-GNU-PR-Keywords: To: Jean-Baptiste Volatier Cc: 48325@debbugs.gnu.org Received: via spool by 48325-submit@debbugs.gnu.org id=B48325.162063048913377 (code B ref 48325); Mon, 10 May 2021 07:09:01 +0000 Received: (at 48325) by debbugs.gnu.org; 10 May 2021 07:08:09 +0000 Received: from localhost ([127.0.0.1]:57217 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1lg01Q-0003Th-BA for submit@debbugs.gnu.org; Mon, 10 May 2021 03:08:08 -0400 Received: from mail-ej1-f49.google.com ([209.85.218.49]:35563) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1lg01N-0003TB-TE for 48325@debbugs.gnu.org; Mon, 10 May 2021 03:08:07 -0400 Received: by mail-ej1-f49.google.com with SMTP id m12so22918259eja.2 for <48325@debbugs.gnu.org>; Mon, 10 May 2021 00:08:05 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:cc:subject:in-reply-to:references:date:message-id :mime-version; bh=mHBpqZZU/QfbRnjojxOy64JAY1wxLqiQtz01WJvcBoM=; b=eGdqe3kVFDXGRcQMsXXiB9MKgvdcfUOOtk/0YQFcUPkM8Tao29jUISyfFg7Mutnf+7 aR2N2To65WHs2lIbAg8jT+HWVmKrYJaAZem5y56705j98wQSjhH+e8+aiz/4iKZ2X6n/ 9XaML3M4qQsiM7gzaW3Z6y+7OpnuSDF2EirLrddzzGwLeJzAr8102fp677SKKgs2LlEm PbG/w/qIoOF687JbZhHvZoqOvZ05CWbv4yqcQ4XkaVIta1ZndytbrsShqRN1DwLgr9gq 7a6SwFGFXpw9+fGYyHBaoJ3SkbUzuWMi0WKpTkeEWRmKQPd31GDJsg3FupYClskHTohH 0Zvg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:in-reply-to:references:date :message-id:mime-version; bh=mHBpqZZU/QfbRnjojxOy64JAY1wxLqiQtz01WJvcBoM=; b=TWp3PkLYLX4WJnAX3cptqBCoUsLKTXdRzwy6z93NQvoXpTplPIniCLBbTLr8y6UsAs wcGTlqgsqBSMsJsIujRZcnFgQgo+akeAelwS2OOLfHQvvA2p9Q9TQH9tzFXsZbQQbmkj LtdvGi6fGLOJzq0w1MqyhpjrYHdndB8qfqMMJEOCCwSjMrLB5h+VRwGZDvxAwEuhX/ZU jQ8NA8LaESzVDYGfTGSjXWxFDVAVxpbEIMBPfJG3mHItiCrZUOMikJ569UtRptxK3FuE WSXs4+2piCX9/+4g7MdbD9dJYhW3Q+yJ2IJovLcSuTDr3i9+IMUE2Im15nB06Mdv5J5K 2ehw== X-Gm-Message-State: AOAM5304qaSy/YdHHDyx1kvoAGdaaokF3TteQy7okfbrYChI5WvsZu1t KQ29jYRSeAiz1IWyjoOsgv+TFCk9Pfg= X-Google-Smtp-Source: ABdhPJzMWCOJCh1z6M8shGPX7kDHEsRg++zWyaeQ0GaO87YduxIeZi4unIYnTPH7djNyc3ZcAk3qcQ== X-Received: by 2002:a17:906:a2d1:: with SMTP id by17mr25175217ejb.426.1620630480136; Mon, 10 May 2021 00:08:00 -0700 (PDT) Received: from guixSD (host-79-17-142-89.retail.telecomitalia.it. [79.17.142.89]) by smtp.gmail.com with ESMTPSA id h4sm9321549edv.97.2021.05.10.00.07.59 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 10 May 2021 00:07:59 -0700 (PDT) From: =?UTF-8?Q?Nicol=C3=B2?= Balzarotti In-Reply-To: <87pmxzfth8.fsf@guixSD.i-did-not-set--mail-host-address--so-tickle-me> References: <87bl9xqx5c.fsf@guixSD.i-did-not-set--mail-host-address--so-tickle-me> <87pmxzfth8.fsf@guixSD.i-did-not-set--mail-host-address--so-tickle-me> Date: Mon, 10 May 2021 09:07:58 +0200 Message-ID: <87mtt3f4mp.fsf@guixSD.i-did-not-set--mail-host-address--so-tickle-me> MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="=-=-=" X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list X-BeenThere: guix-patches@gnu.org List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-patches-bounces+larch=yhetil.org@gnu.org Sender: "Guix-patches" X-Migadu-Flow: FLOW_IN ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=yhetil.org; s=key1; t=1620630557; h=from:from:sender:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type:resent-cc:resent-from:resent-sender: resent-message-id:in-reply-to:in-reply-to:references:references: list-id:list-help:list-unsubscribe:list-subscribe:list-post: dkim-signature; bh=mHBpqZZU/QfbRnjojxOy64JAY1wxLqiQtz01WJvcBoM=; b=IPch/bEZl+y0j1Ve+wxsiUQkg3sdtforu6cUhsa+0UOJK5RLrZvpZ18xKL8Igl48o5Ynfa l6zfZFyEj4SYDRuihEZx6Mi2qiovEkHpzJagqnI2L1OT9g8QbJ5YerqmUwwf3gSsXxKDBo TunmCETiibieoSdJsdwPue4F/q6Y0tOpt9++CHIoO1CmlRVza238nVtg3ScC2Dlx2tSaBA 3bQQn5yEhEbfYAP94v8K0//FYQ5p7/sn/KeT8sgOim6AaL6Yb1BhPkDQHoQIGH0YlFUrx5 AKdoBlHCWtm6tQ84xC4m3urU4lcXw3Qa1zbZDmvmutgXm8Lf+jz9yVqlC3EHLQ== ARC-Seal: i=1; s=key1; d=yhetil.org; t=1620630557; a=rsa-sha256; cv=none; b=cd1zUXfVGSZYQwVF5K50rXDmFKS/sD2s5UeOw8BfLtV8eomWIJntCiH3dU2Q4xn5UVCADP IhfmvNgoVfMGbjaMmAdeJ0LIeC7s9Ydw6EO68dNhtd/m55GtkxQrRa4fNsScxlW77yI+7W TYotiGEbp3Fxn0c0OeNVikk3PE9HH4ihf2IqOZBPIQw0RONyF6mH59fO/L2dSBOJsO2sz3 izujJ8sUr1g47Zhw1DjfA/oQyWsvdxzV+DYpZU2HImBFCzwue2IULF7f6XhNnjsg2Yc2u8 VZfUTQ0l1jtnvn4oYN3AkXYbyRT8MUN1obmT2LRBnvI2Cp2ImQHD/ALTN/n/oA== ARC-Authentication-Results: i=1; aspmx1.migadu.com; dkim=fail ("headers rsa verify failed") header.d=gmail.com header.s=20161025 header.b=eGdqe3kV; dmarc=fail reason="SPF not aligned (relaxed)" header.from=gmail.com (policy=none); spf=pass (aspmx1.migadu.com: domain of guix-patches-bounces@gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=guix-patches-bounces@gnu.org X-Migadu-Spam-Score: -1.35 Authentication-Results: aspmx1.migadu.com; dkim=fail ("headers rsa verify failed") header.d=gmail.com header.s=20161025 header.b=eGdqe3kV; dmarc=fail reason="SPF not aligned (relaxed)" header.from=gmail.com (policy=none); spf=pass (aspmx1.migadu.com: domain of guix-patches-bounces@gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=guix-patches-bounces@gnu.org X-Migadu-Queue-Id: 1DBE311910 X-Spam-Score: -1.35 X-Migadu-Scanner: scn0.migadu.com X-TUID: kn1KbpyQ3ZL3 --=-=-= Content-Type: text/plain I'll forward here the review I sent you privately. I want to add that it built ok. I tried building but julia-benchmarktools and julia-staticarrays are failing. Version 0.7.0 works fine and should work both on 1.6 (tested) and 1.5 (untested) [fn:1] so we should apply it before the julia update. I tried updating StaticArrays, but the build still fails. I opened a bug report here [fn:2]. I attached here the patch, if you can please apply it before yours, apply my suggestions and send the updated patch [fn:1] https://github.com/JuliaCI/BenchmarkTools.jl/blob/e058ff249215671c196f2c24a0a3f401de27b718/test/TrialsTests.jl#L217 [fn:2] https://github.com/JuliaArrays/StaticArrays.jl/issues/912 --=-=-= Content-Type: text/x-patch Content-Disposition: attachment; filename=0001-gnu-julia-benchmarktools-Update-to-0.7.0.patch >From 1376c9103b315492b123eeea1ffc93f81163a809 Mon Sep 17 00:00:00 2001 From: nixo Date: Mon, 10 May 2021 08:55:16 +0200 Subject: [PATCH] gnu: julia-benchmarktools: Update to 0.7.0. * gnu/packages/julia-xyz.scm (julia-benchmarktools): Update to 0.7.0. --- gnu/packages/julia-xyz.scm | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/gnu/packages/julia-xyz.scm b/gnu/packages/julia-xyz.scm index 9812162ce1..04eddbd8b2 100644 --- a/gnu/packages/julia-xyz.scm +++ b/gnu/packages/julia-xyz.scm @@ -76,7 +76,7 @@ be GPU compatible without throwing away the wrapper.") (define-public julia-benchmarktools (package (name "julia-benchmarktools") - (version "0.5.0") + (version "0.7.0") (source (origin (method git-fetch) @@ -85,7 +85,7 @@ be GPU compatible without throwing away the wrapper.") (commit (string-append "v" version)))) (file-name (git-file-name name version)) (sha256 - (base32 "0nsx21m3i5h22lkgyrmfj6r085va6ag40khwssqs8y7l0wz98lvp")))) + (base32 "000izw9pj7cbh9r35nnwg2ijkb9dpfd5nkl2889b8b2dpsh4fi63")))) (build-system julia-build-system) (propagated-inputs `(("julia-json" ,julia-json))) (home-page "https://github.com/JuliaCI/BenchmarkTools.jl") -- 2.31.1 --=-=-= Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Nicol=C3=B2 Balzarotti writes: > seems to be ok (applied fine). A few notes about the patch > > [...] >> `(list ;; Taken from NixOS. Only way I could get libLLVM-6.0.so >> - "-DCMAKE_BUILD_TYPE=3DRelease" >> + "-DCMAKE_BUILD_TYPE=3DRelease" >> + >> + ;; Build a native compiler and the NVPTX backend (NVIDIA) sin= ce >> + ;; Julia insists on it, nothing more. This reduces build tim= es and >> + ;; disk usage. >> + ,(string-append "-DLLVM_TARGETS_TO_BUILD=3D" (system->llvm-ta= rget)) >> + "-DLLVM_EXPERIMENTAL_TARGETS_TO_BUILD=3DNVPTX" >>=20=20 >> - ;; Build a native compiler and the NVPTX backend (NVIDIA) si= nce >> - ;; Julia insists on it, nothing more. This reduces build ti= mes and >> - ;; disk usage. >> - ,(string-append "-DLLVM_TARGETS_TO_BUILD=3D" (system->llvm-t= arget)) >> - "-DLLVM_EXPERIMENTAL_TARGETS_TO_BUILD=3DNVPTX" > Here it's difficult to see if there are changes, you should undo the > indent edit. If indentation is wrong, it should be fixed in a separate c= ommit. > >> + ;; "-DLLVM_HOST_TRIPLE=3D${stdenv.hostPlatform.config}" >> + ;; "-DLLVM_DEFAULT_TARGET_TRIPLE=3D${stdenv.hostPlatform.conf= ig}" >> + ;; "-DLLVM_EXPERIMENTAL_TARGETS_TO_BUILD=3DWebAssembly" > Those seems to be taken from nixos, but is there a reason why are those > commented? > >> + (invoke "patch" flag "-i" (string-append >> "julia-src/deps/patches/" patchname ".patch"))))) > Line too long > >> + (mkdir-p "julia-src") >> + (invoke "tar" "xf" (assoc-ref inputs "julia-source") >> "-C" "julia-src" "--strip-components=3D1") > DITTO > >> + (map (lambda (patchname) >> + (patch patchname "-p1")) (list >> "llvm-D27629-AArch64-large_model_6.0.1" > Here we can slpit the line before (list > >>=20=20 >> - "-DLLVM_LINK_LLVM_DYLIB=3DON")))))) >> + #t))))))) > Returning #t should not be needed anymore > >> + (inputs >> + `(("julia-source" ,(package-source julia)) >> + ,@(package-inputs llvm-11))))) >>=20=20 >> - "suitesparse")) >> + "suitesparse" "libfortran")) >> ":")) >> #t)) > We can remove this #t now > >> + (string-append "stdlib/" pkgname "_jll/src/" >> pkgname "_jll.jl"))) > Long line >> + (lambda* (#:key inputs #:allow-other-keys) >> + ;; some tests execute julia in an environment that needs >> to propagate GUIX_LOCPATH > Good catch, I was wondering which ENV variable was needed > >> + (substitute* "test/cmdlineargs.jl" >> + (("\"HOME\"\\s=3D>\\shomedir\\(\\)") "\"HOME\" =3D> >> homedir(), \"GUIX_LOCPATH\" =3D> ENV[\"GUIX_LOCPATH\"]")) >> #t)) > again, long line and #t >> + (add-before 'install 'symlink-libraries ;; FIXME change >> build to install > What does this comment refer to? >> (string-append "prefix=3D" (assoc-ref %outputs "out")) >>=20=20 >> - ;; Passing the MARCH flag is necessary to build binary substit= utes for >> - ;; the supported architectures. >> - ,(match (or (%current-target-system) >> - (%current-system)) >> - ("x86_64-linux" "MARCH=3Dx86-64") >> - ("i686-linux" "MARCH=3Dpentium4") >> - ("aarch64-linux" "MARCH=3Darmv8-a") >> - ;; Prevent errors when querying this package on unsupp= orted >> - ;; platforms, e.g. when running "guix package --search= =3D" >> - (_ "MARCH=3DUNSUPPORTED")) >> + ;; Passing the MARCH flag is necessary to build binary substitu= tes for >> + ;; the supported architectures. >> + ,(match (or (%current-target-system) >> + (%current-system)) >> + ("x86_64-linux" "MARCH=3Dx86-64") >> + ("i686-linux" "MARCH=3Dpentium4") >> + ("aarch64-linux" "MARCH=3Darmv8-a") >> + ;; Prevent errors when querying this package on unsupported >> + ;; platforms, e.g. when running "guix package --search=3D" >> + (_ "MARCH=3DUNSUPPORTED")) >> > Again the indentation stuff > > --=-=-=--