Re: Bug#778790: glusterfs: FTBFS on new archs: #includes sysctl.h

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

 



Looking at the attached patch file:

  --- glusterfs-3.5.2.orig/xlators/mount/fuse/src/fuse-helpers.c
  +++ glusterfs-3.5.2/xlators/mount/fuse/src/fuse-helpers.c
  @@ -14,7 +14,7 @@
   #include "fuse-bridge.h"
   #if defined(GF_SOLARIS_HOST_OS)
   #include <sys/procfs.h>
  -#else
  +#elif defined GF_BSD_HOST_OS || defined GF_DARWIN_HOST_OS
   #include <sys/sysctl.h>
   #endif

We'd need to update this for current master first wouldn't we, then
backport it to release-3.6 and release-3.5?

The header code in master for this file seems a bit different than
the above, so it's not a straight apply.  Kind of suspecting the
patch may be incorrect, as in master the current code seems to
specifically exclude sys/sysctl.h from being included in FreeBSD.

Anyone have ideas?

+ Justin


On 23 Feb 2015, at 08:10, Patrick Matthäi <pmatthaei@xxxxxxxxxx> wrote:
> Hi,
> 
> please see below.
> 
> 
> -------- Weitergeleitete Nachricht --------
> Betreff:	Bug#778790: glusterfs: FTBFS on new archs: #includes sysctl.h
> Weitersenden-Datum:	Thu, 19 Feb 2015 22:03:02 +0000
> Weitersenden-Von:	Adam Borowski <kilobyte@xxxxxxxxxx>
> Weitersenden-An:	debian-bugs-dist@xxxxxxxxxxxxxxxx
> Weitersenden-CC:	Patrick Matthäi <pmatthaei@xxxxxxxxxx>
> Datum:	Thu, 19 Feb 2015 23:00:05 +0100
> Von:	Adam Borowski <kilobyte@xxxxxxxxxx>
> Antwort an:	Adam Borowski <kilobyte@xxxxxxxxxx>, 778790@xxxxxxxxxxxxxxx
> An:	Debian Bug Tracking System <submit@xxxxxxxxxxxxxxx>
> 
> Package: glusterfs
> Version: 3.5.2-1
> Severity: wishlist
> Tags: patch
> User: 
> debian-x32@xxxxxxxxxxxxxxxx
> 
> Usertags: port-x32 ftbfs-x32
> 
> Hi!
> I'm afraid that glusterfs fails to build on new architectures.  This is
> caused by deprecated #include <sysctl.h>, which compiles successfully on
> old architectures, but on new archs like x32, contains nothing but a #error
> with an explanatory message.
> 
> The sysctl(2) syscall isn't used by glusterfs on Linux on runtime (it would
> spam syslog if called), so it's safe to remove its header.  The attached
> patch does this.
> 
> 
> -- System Information:
> Debian Release: 8.0
>   APT prefers unstable
>   APT policy: (600, 'unstable'), (500, 'unreleased'), (50, 'experimental')
> Architecture: x32 (x86_64)
> 
> Kernel: Linux 3.19.0-x32 (SMP w/6 CPU cores)
> Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8)
> Shell: /bin/sh linked to /bin/dash
> Init: sysvinit (via /sbin/init)
> 
> 
> 
> 
> <no-sysctl-on-linux.patch>_______________________________________________
> Gluster-devel mailing list
> Gluster-devel@xxxxxxxxxxx
> http://www.gluster.org/mailman/listinfo/gluster-devel

--
GlusterFS - http://www.gluster.org

An open source, distributed file system scaling to several
petabytes, and handling thousands of clients.

My personal twitter: twitter.com/realjustinclift

_______________________________________________
Gluster-devel mailing list
Gluster-devel@xxxxxxxxxxx
http://www.gluster.org/mailman/listinfo/gluster-devel





[Index of Archives]     [Gluster Users]     [Ceph Users]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [eCos]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux