GNU bug report logs - #70065
[PATCH 0/6] gnu: Update to Racket 8.12, Chez Scheme 10, and Zuo 1.9.

Previous Next

Package: guix-patches;

Reported by: Philip McGrath <philip <at> philipmcgrath.com>

Date: Fri, 29 Mar 2024 05:17:01 UTC

Severity: normal

Tags: patch

Done: Ludovic Courtès <ludo <at> gnu.org>

Bug is archived. No further changes may be made.

Full log


Message #35 received at 70065 <at> debbugs.gnu.org (full text, mbox):

From: Skyler Ferris <skyvine <at> protonmail.com>
To: Philip McGrath <philip <at> philipmcgrath.com>, 70065 <at> debbugs.gnu.org
Subject: Re: [bug#70065] [PATCH 2/6] gnu: racket: Update to 8.12.
Date: Sat, 30 Mar 2024 14:35:06 +0000
On 3/28/24 22:18, Philip McGrath wrote:
> +(define chez-configure
> +  ;; The custom Chez 'configure' script doesn't allow unrecognized flags, such
> +  ;; as those automatically added by `gnu-build-system`. This replacement
> +  ;; phase uses only the explicitly provided `#:configure-flags`.
> +  #~(lambda* (#:key inputs (configure-flags '()) out-of-source?
> +              #:allow-other-keys)
> +      (let* ((abs-srcdir (getcwd))
> +             (srcdir (if out-of-source?
> +                         (string-append "../" (basename abs-srcdir))
> +                         ".")))
> +        (format #t "source directory: ~s (relative from build: ~s)~%"
> +                abs-srcdir srcdir)
> +        (when out-of-source?
> +            (begin
> +              (mkdir "../build")
> +              (chdir "../build")))
> +        (format #t "build directory: ~s~%" (getcwd))
> +        (format #t "configure flags: ~s~%" configure-flags)
> +        (apply invoke
> +               (string-append srcdir "/configure")
> +               configure-flags))))
> +
Are the format statements supposed to be here or are they left over from 
debugging?
> +          ;; Building the documentation requires stex and a running scheme.
> +          ;; TODO: improve cross support upstream.
I believe this comment should be associated with the build-docs phase, 
not the install-docs phase, correct?

Additionally, I am not too familiar with autotools, so while I did look 
over it and the changes seem reasonable my perspective on that aspect of 
this patch should be taken with a grain of salt.





This bug report was last modified 1 year and 38 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.