[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: 'python-build-system'
From: |
Ludovic Courtès |
Subject: |
Re: 'python-build-system' |
Date: |
Mon, 29 Apr 2013 21:33:03 +0200 |
User-agent: |
Gnus/5.130005 (Ma Gnus v0.5) Emacs/24.3 (gnu/linux) |
Oh, I see now!
Nikita Karetnikov <address@hidden> skribis:
> --- a/guix/build/utils.scm
> +++ b/guix/build/utils.scm
> @@ -680,8 +680,8 @@ contents:
> This is useful for scripts that expect particular programs to be in $PATH,
> for
> programs that expect particular shared libraries to be in $LD_LIBRARY_PATH,
> or
> modules in $GUILE_LOAD_PATH, etc."
> - (let ((prog-real (string-append "." prog "-real"))
> - (prog-tmp (string-append "." prog "-tmp")))
> + (let ((prog-real (string-append (dirname prog) "/." (basename prog)
> "-real"))
> + (prog-tmp (string-append (dirname prog) "/." (basename prog)
> "-tmp")))
OK.
> (define (export-variable lst)
> ;; Return a string that exports an environment variable.
> (match lst
> @@ -709,11 +709,11 @@ modules in $GUILE_LOAD_PATH, etc."
> (with-output-to-file prog-tmp
> (lambda ()
> (format #t
> - "#!~a~%~a~%exec ~a~%"
> + "#!~a~%~a~%exec ~a address@hidden"
Make it "#!~a~%~a~%exec ~a \"address@hidden"~%".
> (which "bash")
> (string-join (map export-variable vars)
> "\n")
> - (canonicalize-path prog-real))))
> + prog-real)))
Keep ‘canonicalize-path’ here, it doesn’t hurt.
I think this can go in (in ‘core-updates’) with these two small
changes. And then we can merge ‘core-updates’.
Thanks!
Ludo’.
- ERROR: missing interface for module (gnutls) (was: 'python-build-system'), (continued)
Re: 'python-build-system', Ludovic Courtès, 2013/04/27
Re: 'python-build-system',
Ludovic Courtès <=
Re: 'python-build-system', Ludovic Courtès, 2013/04/30
Re: 'python-build-system', Cyril Roelandt, 2013/04/28
Re: 'python-build-system', Ludovic Courtès, 2013/04/29
Re: 'python-build-system', Cyril Roelandt, 2013/04/29
Re: 'python-build-system', Nikita Karetnikov, 2013/04/30