GNU bug report logs -
#53656
[PATCH 0/2] 2 Julia packages
Previous Next
Reported by: Efraim Flashner <efraim <at> flashner.co.il>
Date: Mon, 31 Jan 2022 10:39: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
#53656: [PATCH 0/2] 2 Julia packages
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 53656 <at> debbugs.gnu.org.
--
53656: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=53656
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems
[Message part 2 (message/rfc822, inline)]
[Message part 3 (text/plain, inline)]
On Thu, Feb 10, 2022 at 05:22:33PM +0100, zimoun wrote:
> Hi Efraim,
>
> Both patches look good to me.
>
>
> I notice this warning:
>
> --8<---------------cut here---------------start------------->8---
> ┌ Warning: Package OptimTestProblems does not have LinearAlgebra in its dependencies:
> │ - If you have OptimTestProblems checked out for development and have
> │ added LinearAlgebra as a dependency but haven't updated your primary
> │ environment's manifest file, try `Pkg.resolve()`.
> │ - Otherwise you may need to report an issue with OptimTestProblems
> └ Loading LinearAlgebra into OptimTestProblems from project dependency, future warnings for OptimTestProblems are suppressed.
> --8<---------------cut here---------------end--------------->8---
It ends up not being a problem, but I'm left wondering if we should
extend the #:julia-package-foo arguments to add more fields or if we
should just add a custom phase to build a Package.toml that includes all
the fields needed.
> and...
>
>
> On Mon, 31 Jan 2022 at 12:40, Efraim Flashner <efraim <at> flashner.co.il> wrote:
>
> > * gnu/packages/julia-xyz.scm (julia-optim): New variable.
>
> [...]
>
> > + #:use-module (guix gexp)
>
> [...]
>
> > + #:phases
> > + #~(modify-phases %standard-phases
> > + (add-after 'unpack 'adjust-tests
> > + (lambda _
> > + ;; TODO: Figure out why this test fails.
> > + (substitute* "test/runtests.jl"
> > + ((".*l_bfgs.*") "")))))))
>
> ...I am thinking loudly if this gexp is required. There are other many
> similar patter without a gexp. Therefore, for consistency with the
> rest, I would be in favor to switch to regular modification of phases.
> Or let me know if this is becoming the new patter and I would adapt for
> the others.
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.
--
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)]
Two more julia packages, ready for upstreaming.
Efraim Flashner (2):
gnu: Add julia-linesearches.
gnu: Add julia-optim.
gnu/packages/julia-xyz.scm | 93 +++++++++++++++++++++++++++++++++++++-
1 file changed, 92 insertions(+), 1 deletion(-)
base-commit: 43dd34c7777a212c99a97da7a2c237158faa9a1b
prerequisite-patch-id: 0db9d2565885888d76f51e5e9ab3287fdaa3877a
--
2.34.0
This bug report was last modified 3 years and 102 days ago.
Previous Next
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997,2003 nCipher Corporation Ltd,
1994-97 Ian Jackson.