GNU bug report logs -
#54088
[PATCH 0/2] julia-build-system: Add missing julia-pkg-deps
Previous Next
Reported by: Efraim Flashner <efraim <at> flashner.co.il>
Date: Mon, 21 Feb 2022 12:49:02 UTC
Severity: normal
Tags: patch
Done: Efraim Flashner <efraim <at> flashner.co.il>
Bug is archived. No further changes may be made.
Full log
View this message in rfc822 format
[Message part 1 (text/plain, inline)]
Your bug report
#54088: [PATCH 0/2] julia-build-system: Add missing julia-pkg-deps
which was filed against the guix-patches package, has been closed.
The explanation is attached below, along with your original report.
If you require more details, please reply to 54088 <at> debbugs.gnu.org.
--
54088: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=54088
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems
[Message part 2 (message/rfc822, inline)]
[Message part 3 (text/plain, inline)]
Great job! Patches pushed (with one change)
On Wed, Feb 23, 2022 at 02:44:43PM +0100, zimoun wrote:
> Hi Efraim,
>
> On Mon, 21 Feb 2022 at 14:47, Efraim Flashner <efraim <at> flashner.co.il> wrote:
>
> > It turns out we didn't expose julia-package-dependencies in the
> > julia-build-system, making the created Package.toml incomplete. This
> > corrects the omission and fixes the one package where it is manually
> > added.
>
> All LGTM. But I take this opportunity to uniform, as discussed:
>
> It's definitely the new pattern we're using now. I suppose it's
> not really necessary here, but having more instances of it
> throughout the codebase also makes it easier to search for
> examples when others are looking to use or modify gexps.
>
> <https://issues.guix.gnu.org/issue/53656#4-lineno49>
>
> The first patch of the series uses G-exps. Note that it does not imply
> any rebuild.
>
> The Julia build system is adapted for the new
> 'julia-package-dependencies' and so it requires the minor tweak to
> default with ''() instead of #f; otherwise it crashes for some packages.
>
> Since we are at it, I fix a minor typo of how Julia uses --procs. Other
> said, currently "guix build julia-media -c 0" works but not "guix build
> julia-media -c 1" which crashes.
>
> Because the build system modifications imply a full Julia rebuild, let
> take the opportunity to remove the trailing #t. It is done package per
> package because each modified package need a rebuild so it eases to
> follow, IMHO. They can be squashed otherwise.
>
>
> Cheers,
> simon
--
Efraim Flashner <efraim <at> flashner.co.il> אפרים פלשנר
GPG key = A28B F40C 3E55 1372 662D 14F7 41AA E7DC CA3D 8351
Confidentiality cannot be guaranteed on emails sent or received unencrypted
[signature.asc (application/pgp-signature, inline)]
[Message part 5 (message/rfc822, inline)]
It turns out we didn't expose julia-package-dependencies in the
julia-build-system, making the created Package.toml incomplete. This
corrects the omission and fixes the one package where it is manually
added.
Efraim Flashner (2):
build: julia: Add julia-package-dependencies as keyword.
gnu: julia-media: Use julia-package-dependencies.
gnu/packages/julia-xyz.scm | 15 ++-------------
guix/build-system/julia.scm | 5 ++++-
guix/build/julia-build-system.scm | 7 ++++++-
3 files changed, 12 insertions(+), 15 deletions(-)
base-commit: 7eefff2054b94f8a7ad850ad8f36b8773bb39ce9
--
2.34.0
This bug report was last modified 3 years and 86 days ago.
Previous Next
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997,2003 nCipher Corporation Ltd,
1994-97 Ian Jackson.