[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#26966: [PATCH 06/22] gnu: Add ecj-javac-wrapper.
From: |
Ricardo Wurmus |
Subject: |
bug#26966: [PATCH 06/22] gnu: Add ecj-javac-wrapper. |
Date: |
Mon, 22 May 2017 18:28:39 +0200 |
User-agent: |
mu4e 0.9.18; emacs 25.2.1 |
Hi Ludo,
> Hi!
>
> Roel Janssen <address@hidden> skribis:
>
>>> + (with-output-to-file target
>>> + (lambda _
>>> + (format #t "#!~a --no-auto-compile\n!#\n" guile)
>>> + (map write
>>> + `((use-modules (ice-9 match)
>>> + (ice-9 receive)
>>> + (ice-9 hash-table)
>>> + (srfi srfi-1)
>>> + (srfi srfi-26))
>>> + (define defaults
>>> + '(("-bootclasspath" ,bootcp)
>>> + ("-source" "1.5")
>>> + ("-target" "1.5")
>>> + ("-cp" ".")))
>
> Minor nitpick: I would rather write this as
>
> (write `(begin
> (use-modules …)
> (define …)
> …)
> port)
>
> because (1) ‘map’ returns a list of values, so ‘for-each’ would be more
> appropriate, and (2) I think sexps should be considered as a whole
> rather than as a list of sexps in general.
Yes, I totally forgot about “begin”! That’s much nicer indeed!
--
Ricardo
GPG: BCA6 89B6 3655 3801 C3C6 2150 197A 5888 235F ACAC
https://elephly.net
bug#26966: [PATCH 08/22] gnu: Add jamvm-bootstrap., Ricardo Wurmus, 2017/05/17
bug#26966: [PATCH 07/22] gnu: Add classpath., Ricardo Wurmus, 2017/05/17
bug#26966: [PATCH 09/22] gnu: Add classpath-jamvm-wrappers., Ricardo Wurmus, 2017/05/17
bug#26966: [PATCH 11/22] gnu: Add classpath-devel., Ricardo Wurmus, 2017/05/17
bug#26966: [PATCH 13/22] gnu: Add ecj-javac-on-jamvm-wrapper-final., Ricardo Wurmus, 2017/05/17