[sthibault@xxxxxxxxxx: Re: Bug#891812: util-linux: FTBFS on hurd-i386: calls pty_init_slave unconditionally]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,

Please consider Samuels patch as forwarded below to fix 
(debian package) build on Hurd.

Sorry for messy formatting and lack of s-o-b.
If it helps, raw patch can be downloaded from:
https://bugs.debian.org/cgi-bin/bugreport.cgi?att=1;bug=891812;filename=patch;msg=34

( Original discussion at http://bugs.debian.org/891812 )

Regards,
Andreas Henriksson

----- Forwarded message from Samuel Thibault <sthibault@xxxxxxxxxx> -----

Date: Sun, 27 May 2018 18:00:02 +0200
From: Samuel Thibault <sthibault@xxxxxxxxxx>
To: Andreas Henriksson <andreas@xxxxxxxx>
Cc: "Aaron M. Ucko" <ucko@xxxxxxxxxx>, 891812@xxxxxxxxxxxxxxx
Subject: Re: Bug#891812: util-linux: FTBFS on hurd-i386: calls pty_init_slave
	unconditionally
User-Agent: NeoMutt/20170113 (1.7.2)

Hello,

Andreas Henriksson, le lun. 21 mai 2018 22:45:54 +0200, a ecrit:
> On Thu, Mar 01, 2018 at 01:55:22AM +0100, Samuel Thibault wrote:
> > Aaron M. Ucko, on mer. 28 févr. 2018 19:47:40 -0500, wrote:
> > >   login-utils/su-common.c:1427:3: warning: implicit declaration of function 'pty_init_slave'; did you mean 'initstate'? [-Wimplicit-function-declaration]
> > >   [...]
> > >   ./login-utils/su-common.c:1427: undefined reference to `pty_init_slave'
> > 
> > IIRC I got a fix commited upstream
> 
> Your fix should now be part of the latest upload (2.32-0.1),
> unfortunately there's another problem now.
> 
> Any chance you can test if by any chance the following upstream commit
> fixes the (libmount-related) build issue on hurd (or if additional
> upstream fixing is needed for hurd)?
> 
> https://git.kernel.org/pub/scm/utils/util-linux/util-linux.git/commit/?id=061d1a51097c3c025ff46173f10aa135f9a610d4

As Svante mentioned it will not work because GNU/Hurd does not provide
the linuxish mount() interface (which is about to change, btw ;)
The attached patch fixes the build.

Samuel

diff --git a/libmount/src/libmount.h.in b/libmount/src/libmount.h.in
index 11fd759fa..39568a37c 100644
--- a/libmount/src/libmount.h.in
+++ b/libmount/src/libmount.h.in
@@ -34,7 +34,9 @@ extern "C" {
  * don't want to include sys/mount.h at all to avoid collisions.
  */
 #ifndef MS_RDONLY
-# include <sys/mount.h>
+# ifdef HAVE_SYS_MOUNT_H
+#  include <sys/mount.h>
+# endif
 #endif
 
 #define LIBMOUNT_VERSION   "@LIBMOUNT_VERSION@"


----- End forwarded message -----
--
To unsubscribe from this list: send the line "unsubscribe util-linux" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Netdev]     [Ethernet Bridging]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]

  Powered by Linux