From: Manolis Ragkousis <manolis837@gmail.com>
To: "Ludovic Courtès" <ludo@gnu.org>
Cc: Guix-devel <Guix-devel@gnu.org>
Subject: Re: [PATCH] gnu: base: Add Glibc-Hurd Headers.
Date: Wed, 24 Sep 2014 16:01:15 +0300 [thread overview]
Message-ID: <CAFtzXzNSie=fQ0ENV7M_LskUc7gA5QdnT5Wpw17Pt5-wYgY4kw@mail.gmail.com> (raw)
In-Reply-To: <8761hbcspb.fsf@gnu.org>
[-- Attachment #1: Type: text/plain, Size: 251 bytes --]
Taken care of the FIXME.
Just wanted to add that in base.scm, instead of gnu-gettext we needed
perl. Probably I changed it by mistake while rebasing some local
commits.
And thread_terminate does exist, that's why it's enough for our needs.
Manolis
[-- Attachment #2: 0001-Taken-care-of-FIXME.patch --]
[-- Type: text/x-patch, Size: 4393 bytes --]
From 0f169f370f4f3a6baf45b0020a461ae9844befb2 Mon Sep 17 00:00:00 2001
From: Manolis Ragkousis <manolis837@gmail.com>
Date: Thu, 18 Sep 2014 22:21:19 +0000
Subject: [PATCH] Taken care of FIXME.
---
gnu/packages/base.scm | 12 +++++-------
gnu/packages/patches/hurd-minimal.patch | 4 +---
gnu/packages/patches/libpthread-glibc-preparation.patch | 11 ++++-------
3 files changed, 10 insertions(+), 17 deletions(-)
diff --git a/gnu/packages/base.scm b/gnu/packages/base.scm
index 234194f..27c8fc1 100644
--- a/gnu/packages/base.scm
+++ b/gnu/packages/base.scm
@@ -533,7 +533,7 @@ with the Linux kernel.")
`(,@(package-native-inputs glibc)
("patch/libpthread-patch" ,(search-patch "libpthread-glibc-preparation.patch"))
("mig" ,mig)
- ("gettext" ,gnu-gettext) ;needed for FIXME
+ ("perl" ,perl)
("libpthread" ,(origin
(method git-fetch)
(uri (git-reference
@@ -548,12 +548,9 @@ with the Linux kernel.")
(substitute-keyword-arguments (package-arguments glibc)
((#:configure-flags cf)
`(append (list "--host=i686-pc-gnu"
- ;; Since Hurd supports only i686 disable everything
- ;; else. FIXME: needed?
- ;"--disable-multi-arch"
;; nscd fails to build for GNU/Hurd:
- ;; <http://lists.gnu.org/FIXME>. Disable it.
+ ;; <https://lists.gnu.org/archive/html/bug-hurd/2014-07/msg00006.html>. Disable it.
"--disable-nscd")
,cf))
((#:phases phases)
@@ -582,8 +579,9 @@ with the Linux kernel.")
(lambda* (#:key outputs #:allow-other-keys)
(and (zero? (system* "make" "install-headers"))
- ;; Make an empty stubs.h to work around FIXME. See
- ;; <http://lists.gnu.org/FIXME>.
+ ;; Make an empty stubs.h to work around not being able to
+ ;; produce a valid stubs.h and causing the build to fail. See
+ ;; <http://lists.gnu.org/archive/html/guix-devel/2014-04/msg00233.html>.
(let ((out (assoc-ref outputs "out")))
(close-port
(open-output-file
diff --git a/gnu/packages/patches/hurd-minimal.patch b/gnu/packages/patches/hurd-minimal.patch
index 60eb344..b551d5c 100644
--- a/gnu/packages/patches/hurd-minimal.patch
+++ b/gnu/packages/patches/hurd-minimal.patch
@@ -2,9 +2,7 @@ We want to build the Hurd's libihash very early so that we can then
build libc. Because of that, we cannot link against libc, and thus
we miss 'clnt_create' and friends.
-See <http://lists.gnu.org/FIXME>.
-
-FIXME: Is this explanation correct?
+See <http://lists.gnu.org/archive/html/bug-hurd/2014-04/msg00116.html>.
diff --git a/configure.ac b/configure.ac
index f8856db..a381219 100644
diff --git a/gnu/packages/patches/libpthread-glibc-preparation.patch b/gnu/packages/patches/libpthread-glibc-preparation.patch
index 2862b76..fb13dd3 100644
--- a/gnu/packages/patches/libpthread-glibc-preparation.patch
+++ b/gnu/packages/patches/libpthread-glibc-preparation.patch
@@ -15,13 +15,10 @@ index 0000000..2cdbc71
--
1.9.0
-We are using a version of GNU Mach that lacks 'thread_terminate' (not
-introduced yet) and 'thread_terminate_release' (recently removed.)
-FIXME: really? Add URLs.
-
-The 'thread_terminate' RPC call will be enough for our needs.
+We are using a version of GNU Mach that lacks 'thread_terminate_release'
+(not introduced yet).The 'thread_terminate' RPC call will be enough for
+our needs.
http://lists.gnu.org/archive/html/bug-hurd/2014-05/msg00127.html
-FIXME: then why not use it?
diff --git a/libpthread/sysdeps/mach/pt-thread-terminate.c b/libpthread/sysdeps/mach/pt-thread-terminate.c
index 6672065..129a611 100644
@@ -45,7 +42,7 @@ index 6672065..129a611 100644
The __PTHREAD_SPIN_LOCK_INITIALIZER definition is missing, so we
define it to __SPIN_LOCK_INITIALIZER which already exists.
-See <http://lists.gnu.org/FIXME>.
+See http://lists.gnu.org/archive/html/commit-hurd/2009-04/msg00006.html
diff --git a/libpthread/sysdeps/mach/bits/spin-lock.h b/libpthread/sysdeps/mach/bits/spin-lock.h
index 537dac9..fca0e5a 100644
--
2.1.0
next prev parent reply other threads:[~2014-09-24 13:01 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-06-02 21:16 [PATCH] gnu: base: Add Glibc-Hurd Headers Manolis Ragkousis
2014-08-29 13:48 ` Ludovic Courtès
2014-09-24 13:01 ` Manolis Ragkousis [this message]
2014-09-27 9:31 ` Ludovic Courtès
2014-09-27 13:50 ` Manolis Ragkousis
2014-09-27 17:30 ` Manolis Ragkousis
2014-09-28 9:51 ` 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
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='CAFtzXzNSie=fQ0ENV7M_LskUc7gA5QdnT5Wpw17Pt5-wYgY4kw@mail.gmail.com' \
--to=manolis837@gmail.com \
--cc=Guix-devel@gnu.org \
--cc=ludo@gnu.org \
/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).