[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#72925] Adding JPM package for Janet
From: |
Ludovic Courtès |
Subject: |
[bug#72925] Adding JPM package for Janet |
Date: |
Sat, 12 Oct 2024 19:22:01 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) |
Suhail Singh <suhailsingh247@gmail.com> skribis:
> * gnu/packages/lisp.scm (jpm): Ensure jpm respects JANET_HEADERPATH and
> JANET_LIBPATH if set by user. Ensure gcc/g++ is able to find header files and
> compilation-related utilities.
>
> Change-Id: Ic7218dbd10e6fabddded50894b82492de8cabc88
[...]
> + (wrap-program (string-append #$output "/bin/jpm")
> + `("JANET_HEADERPATH" ":" prefix
> + (,(string-append #$janet "/include/janet")))
> + `("JANET_LIBPATH" ":" prefix
> + (,(string-append #$janet "/lib")))
> + `("C_INCLUDE_PATH" ":" prefix
> + (,(string-append gcc-toolchain "/include")))
> + `("CPLUS_INCLUDE_PATH" ":" prefix
> + (,(string-append gcc-toolchain "/include/c++")
> + ,(string-append gcc-toolchain "/include")))
> + `("LIBRARY_PATH" ":" prefix
> + (,(string-append gcc-toolchain "/lib")
> + ,(string-append gcc-toolchain "/lib64")))
> + `("PATH" ":" prefix
> + (,(string-append gcc-toolchain "/bin")
> + ,(string-append #$coreutils "/bin"))))))))))
> + (inputs (list bash-minimal
> + ;; Lazily resolve the gcc-toolchain to avoid a circular
> + ;; dependency.
> + (module-ref (resolve-interface '(gnu packages
> commencement))
> + 'gcc-toolchain)))
I suppose JPM shells out to GCC to compiler Janet (or C?) code, right?
I’d recommend adding ‘gcc’, ‘glibc’, ‘binutils’, and ‘ld-wrapper’ to
‘inputs’; that’d less us avoid the ‘gcc-toolchain’ dance.
Then in, the phase above, make sure to ‘search-input-file’ rather than
direct references to these variables.
How does that sound?
Alternatively, depending on how important this is for JPM, you could
also leave it up to users to install ‘gcc-toolchain’ alongside JPM when
they need it.
Thanks,
Ludo’.