From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:470:142:3::10]:36814) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1j1XKV-0004FM-S9 for guix-patches@gnu.org; Tue, 11 Feb 2020 10:20:05 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1j1XKU-0001rG-I5 for guix-patches@gnu.org; Tue, 11 Feb 2020 10:20:03 -0500 Received: from debbugs.gnu.org ([209.51.188.43]:51393) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1j1XKU-0001rC-Ex for guix-patches@gnu.org; Tue, 11 Feb 2020 10:20:02 -0500 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1j1XKU-0002oP-B9 for guix-patches@gnu.org; Tue, 11 Feb 2020 10:20:02 -0500 Subject: [bug#38546] [00/11] Update Julia, Fix precompilation, add HTTP.jl Resent-Message-ID: MIME-Version: 1.0 References: <87v9qpvbyr.fsf@guixSD.i-did-not-set--mail-host-address--so-tickle-me> <87wo9ouaq5.fsf@guixSD.i-did-not-set--mail-host-address--so-tickle-me> <87d0bcly8f.fsf@guixSD.i-did-not-set--mail-host-address--so-tickle-me> <877e1jdd62.fsf@guixSD.i-did-not-set--mail-host-address--so-tickle-me> <87lfp9dusm.fsf@guixSD.i-did-not-set--mail-host-address--so-tickle-me> In-Reply-To: <87lfp9dusm.fsf@guixSD.i-did-not-set--mail-host-address--so-tickle-me> From: zimoun Date: Tue, 11 Feb 2020 16:19:32 +0100 Message-ID: Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable 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: =?UTF-8?Q?Nicol=C3=B2?= Balzarotti Cc: Ludovic =?UTF-8?Q?Court=C3=A8s?= , 38546@debbugs.gnu.org On Tue, 11 Feb 2020 at 16:10, Nicol=C3=B2 Balzarotti = wrote: > > 1. > >> Do you have a special motivation to rename 'llvm-patch' to 'julia-patc= h'? > >> (I agree, it is better :-)) > > > Yes, because now it's not only used by llvm-julia but also by the new > libunwind-julia. So calling llvm-patch a function that takes a patch > from the julia repo and that is applied to a "julia module" is measleadin= g. Ok. > > 2. > >> Well, only 2 inputs are missing. Do you think it is affordable to pack > >> them and so remove the commentary? > >> > >> --8<---------------cut here---------------start------------->8--- > >> ;; FIXME: The following inputs are downloaded from upstream to allow u= s > >> ;; to use the lightweight Julia release tarball. Ideally, these input= s > >> ;; would eventually be replaced with proper Guix packages. > >> ;; Find dependencies versions here: > >> ;; https://raw.githubusercontent.com/JuliaLang/julia/v1.3.0/deps/Versi= ons.make > >> --8<---------------cut here---------------end--------------->8--- > > I indeed tried. The main problem is that in the Makefile there's no > USE_SYSTEM_{OBJCONV/LIBWHICH}, and I didn't want to mess with the it. > But yes, I think I could at least try again. Let me check how it will be crawled by SWH to be sure that Guix will be able to still build Julia 1.3 even if GitHub does not exist anymore. And I am in favour to let these 2 inputs. > > 3. > >> Otherwise, there is some issues about indentation -- I think you use > >> Emacs, so check that '.dir-locals.el' is correctly setup -- double > >> space for sentences in description, > >> 'patch/julia-SOURCE_DATE_EPOCH-mtime.patch' should be added with the > >> same commit that "Update", etc. > >> I have half-corrected so I can send you these 2 v2-patches if you want= to. :-) > > > If you already fixed them, yes please send a v2 patch and I'll continue > working from there. Stay tuned. :-) I try to send it today... > Fine for me, do we need also to debug a broken test right? I do not know if the test are broken. I am not able to run them on my desktop machine. Not enough memory. Cheers, simon