From: ludo@gnu.org (Ludovic Courtès)
To: Manolis Ragkousis <manolis837@gmail.com>
Cc: Guix-devel@gnu.org
Subject: Re: [PATCH] gnu: Add glibc-hurd-headers
Date: Tue, 15 Apr 2014 00:20:01 +0200 [thread overview]
Message-ID: <87vbubzh7y.fsf@gnu.org> (raw)
In-Reply-To: <CAFtzXzOZG6iYNacWOcmsp06qeAN_NQNboCczuTsy6oUBu5c2ag@mail.gmail.com> (Manolis Ragkousis's message of "Mon, 14 Apr 2014 22:00:09 +0000")
Manolis Ragkousis <manolis837@gmail.com> skribis:
> Take notice that it needs the patch "glibc-make-4.0.patch" for the
> configure check to pass. I should bring up this issue to the hurd
> guys.
That’s a sign that the Hurd’s libc should be rebased on top of libc 2.19.
> +(define-public glibc/hurd-headers
> + (package (inherit glibc)
> + (name "glibc-hurd-headers")
The indentation is wrong (see this file for examples.)
> + (arguments
> + (substitute-keyword-arguments
> + `(#:strip-binaries? #f
> + #:out-of-source? #t
> + #:tests? #f
> + #:configure-flags (list
> + "--host=i686-pc-gnu"
> + "--enable-addons"
> + ;;"--disable-sanity-checks"
> + (string-append "libc_cv_localedir="
> + (assoc-ref %outputs "locales")
> + "/share/locale")
> + (string-append "--with-headers="
> + (assoc-ref %build-inputs "gnumach-headers")
> + "/include:"
> + (assoc-ref %build-inputs "hurd-headers")
> + "/include")
> +
> + ;; Use our Bash instead of /bin/sh.
> + (string-append "BASH_SHELL="
> + (assoc-ref %build-inputs "bash")
> + "/bin/bash"))
> + ,@(package-arguments glibc))
> + ((#:phases phases)
> + `(alist-replace
> + 'install
> + (lambda _
> + (zero? (system* "make" "install-headers")))
> + (alist-delete
> + 'build
> + ,phases)))))))
This looks good to me, but unnecessarily complex: get rid of
‘substitute-keyword-arguments’:
(arguments
'(#:out-of-source? #t
#:tests? #f
#:configure-flags ...
#:phases (alist-replace ... %standard-phases)))
HTH,
Ludo’.
next prev parent reply other threads:[~2014-04-14 22:20 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-04-14 22:00 [PATCH] gnu: Add glibc-hurd-headers Manolis Ragkousis
2014-04-14 22:20 ` Ludovic Courtès [this message]
2014-04-14 22:31 ` Manolis Ragkousis
[not found] ` <CAFtzXzM3yy-WJ7EpmKBK8Vf4FnwQhG3gs1+bw5T=+Wmz_qZJXA@mail.gmail.com>
[not found] ` <87d2gjat2d.fsf@gnu.org>
2014-04-15 14:00 ` Manolis Ragkousis
2014-04-15 19:39 ` Ludovic Courtès
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87vbubzh7y.fsf@gnu.org \
--to=ludo@gnu.org \
--cc=Guix-devel@gnu.org \
--cc=manolis837@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/guix.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.