check for argp header

DoneSubmitted by Nikita Gillmann.
Details
2 participants
  • Ludovic Courtès
  • Nikita Gillmann
Owner
unassigned
Severity
normal
N
N
Nikita Gillmann wrote on 15 May 14:30 +0200
(address . guix-patches@gnu.org)
20200515123016.gmeicqvo67qrqpuh@hex
Hi,
as mentioned in IRC I have begun porting Guix to NetBSD (with the pathtaken not yet decided upon, just plain building guix itself for now).
Glibc provides argp. Arguably we don't have to check for argp becauseGuix targets glibc. But I am quiete certain that there will be peoplewho will attempt to do what I am doing and run into this.
I haven't tested this, but semantically it should check out. I don'tknow how much changed in guix's bootstrap but configure.ac should still bethe right place for this patch.
From eb8214d2cd6b2170f6e05b89dbd8e47e1f0f4326 Mon Sep 17 00:00:00 2001From: nikita <nikita@n0.is>Date: Fri, 15 May 2020 14:23:48 +0200Subject: [PATCH] configure: check for arpg header as required by nix.
This is provided by glibc, porting to a platform without glibcrevealed this lack of check.--- configure.ac | 3 +++ 1 file changed, 3 insertions(+)
Toggle diff (16 lines)diff --git a/configure.ac b/configure.acindex 0a20b476eb..174633785a 100644--- a/configure.ac+++ b/configure.ac@@ -266,6 +266,9 @@ else fi AC_SUBST([LIBLZ]) +# check for standard headers, required in Nix daemon+AC_CHECK_HEADERS([argp.h])+ dnl Check for Guile-SSH, for the (guix ssh) module. GUIX_CHECK_GUILE_SSH AM_CONDITIONAL([HAVE_GUILE_SSH],-- 2.25.2
L
L
Ludovic Courtès wrote on 16 May 19:22 +0200
(name . Nikita Gillmann)(address . nikita@n0.is)(address . 41286@debbugs.gnu.org)
87ftbzrdzw.fsf@gnu.org
Hi Nikita,
Nikita Gillmann <nikita@n0.is> skribis:
Toggle quote (7 lines)> as mentioned in IRC I have begun porting Guix to NetBSD (with the path> taken not yet decided upon, just plain building guix itself for now).>> Glibc provides argp. Arguably we don't have to check for argp because> Guix targets glibc. But I am quiete certain that there will be people> who will attempt to do what I am doing and run into this.
Guix targets glibc-based systems, so as you write, it’s reasonable toassume argp is present.
Also, using AC_CHECK_HEADERS doesn’t achieve anything: it only defines‘HAVE_ARGP_H’ to zero or one.
Last, I don’t want to discourage anyone from porting, but I also want tobe clear about what it entails. I’m strongly in favor of supportingonly glibc because: (1) after all, it’s about GNU as a system, and (2)my experience with Nixpkgs is that supporting multiple C libraries isjust too much work to maintain good support.
Thus I’m closing for now.
Thanks,Ludo’.
L
L
Ludovic Courtès wrote on 16 May 19:22 +0200
control message for bug #41286
(address . control@debbugs.gnu.org)
87eerjrdzk.fsf@gnu.org
tags 41286 wontfixclose 41286quit
?