From: ludo@gnu.org (Ludovic Courtès)
To: Manolis Ragkousis <manolis837@gmail.com>
Cc: Guix-devel@gnu.org
Subject: Re: [PATCH] gnu: base: Add Glibc-Hurd.
Date: Mon, 09 Jun 2014 21:27:08 +0200 [thread overview]
Message-ID: <871tux994j.fsf@gnu.org> (raw)
In-Reply-To: <CAFtzXzPd3ikPqvUWSfgoH99jqQE0NEnC7CGy1nKm014dcPsSTw@mail.gmail.com> (Manolis Ragkousis's message of "Fri, 6 Jun 2014 22:23:39 +0000")
Γειά σας!
Manolis Ragkousis <manolis837@gmail.com> skribis:
>>
>>> + (propagated-inputs `(("gnumach-headers" ,gnumach-headers)
>>> + ("hurd-headers" ,hurd-headers)
>>> + ("hurd-minimal" ,hurd-minimal)))
>>
>> Add a one-line comment saying why they’re propagated.
>
> Actually I am not sure, should they be propagated?
Libc provides <hurd.h>, which includes a bunch of Hurd and Mach headers,
so both should be propagated.
Can you add this explanation in a comment above ‘propagated-inputs’?
[...]
>> Is --host the only thing that differs from ‘glibc’? If so, we need to
>> factorize things.
>
> In order for glibc/hurd to build, we need to remove
>
>> (string-append "--with-headers="
>> (assoc-ref %build-inputs "linux-headers")
>> "/include")
>> "--enable-kernel=2.6.30"
>> ;; XXX: Work around "undefined reference to `__stack_chk_guard'".
>> "libc_cv_ssp=no"
>
> which are linux specific, and add
>
>> "--host=i686-pc-gnu"
>> "--disable-profile"
>> "--disable-multi-arch"
>> "--disable-nscd"
>> "--enable-obsolete-rpc"
>
> How do you suggest doing it? :-)
Something like:
(define glibc-hurd
(package (inherit glibc)
(arguments
(substitute-keyword-arguments (package-arguments glibc)
((#:configure-flags cf)
`("--host=i686-pc-gnu"
"--disable-multi-arch"
,@cf))))))
There are examples of that in base.scm.
Could you add a comment justifying each of these --enable/disable flags?
For instance, I don’t see why nscd would need to be disabled.
>> However, why do the headers need to be copied in the first place? I
>> believe the sysdeps headers of add-ons are automatically picked up the
>> libc’s build system normally. Could you check what’s going on?
>
> When these headers are included, they are used like that
>> <bits/...>
> so, while they are normally included in the path, it only looks in the
> directory "bits/" for them not in libpthread/sysdeps/generic/bits/.
>
> Any suggestions?
No, I think that’s more a question for the Hurd hackers, but I’m pretty
sure this ought to work by default. Could you check with bug-hurd or on
IRC?
>>> --- /dev/null
>>> +++ b/gnu/packages/patches/glibc-manual-fix.patch
>>> @@ -0,0 +1,12 @@
>>> +diff --git a/manual/contrib.texi b/manual/contrib.texi
>>> +index 3b9d23c..376b40d 100644
>>> +--- a/manual/contrib.texi
>>> ++++ b/manual/contrib.texi
>>> +@@ -1,3 +1,4 @@
>>> ++@end deftypefun
>>> + @node Contributors, Free Manuals, Platform, Top
>>> + @c %MENU% Who wrote what parts of the GNU C Library
>>> + @appendix Contributors to @theglibc{}
>>
>> What’s this? (Missing explanation.)
>
> Without this, I get the error
>> ./contrib.texi:1: @node seen before @end deftypefun
I suppose you could do without the patch by using ‘texinfo-4’ instead of
‘texinfo’, which would be easier. Could you check that?
Thanks,
Ludo’.
next prev parent reply other threads:[~2014-06-09 19:27 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-06-02 21:09 [PATCH] gnu: base: Add Glibc-Hurd Manolis Ragkousis
2014-06-02 21:38 ` Ludovic Courtès
2014-06-06 22:23 ` Manolis Ragkousis
2014-06-09 19:27 ` Ludovic Courtès [this message]
2014-06-18 19:56 ` Manolis Ragkousis
2014-06-21 15:20 ` Ludovic Courtès
2014-07-17 15:39 ` Manolis Ragkousis
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
List information: https://guix.gnu.org/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=871tux994j.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 public inbox
https://git.savannah.gnu.org/cgit/guix.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).