[glibc] branch sid updated (b891546 -> 0945cf6)

2017-04-10 Thread Aurelien Jarno
This is an automated email from the git hooks/post-receive script.

aurel32 pushed a change to branch sid
in repository glibc.

  from  b891546   patches/any/submitted-if.h-conflict.diff: new patch to 
fix the conflict between  and .  Closes: #824442.
   new  0945cf6   Revert "patches/any/submitted-if.h-conflict.diff: new 
patch to fix the conflict between  and .  Closes: 
#824442."

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "adds" were already present in the repository and have only
been added to this reference.


Summary of changes:
 debian/changelog|  2 -
 debian/patches/any/submitted-if.h-conflict.diff | 74 -
 debian/patches/series   |  1 -
 3 files changed, 77 deletions(-)
 delete mode 100644 debian/patches/any/submitted-if.h-conflict.diff

-- 
Alioth's /usr/local/bin/git-commit-notice on 
/srv/git.debian.org/git/pkg-glibc/glibc.git



Bug#824442: and conflict needs to be resolved

2017-04-10 Thread Aurelien Jarno
tag 824442 - pending
clone 824442 -1
reassign -1 linux
retitle -1 linux: libc-compat.h sets __UAPI_DEF_IF_* to 1 even when if.h hasn't 
been included
thanks

> On 2017-03-04 17:23, Niels Thykier wrote:
> > On Tue, 14 Feb 2017 20:36:43 +0100 Aurelien Jarno 
> > wrote:
> > > [...]
> > > 
> > > Now that the change have been done on the kernel side, the bug is not
> > > really visible anymore, so nobody really cared about this bug.
> > > 
> > > I'll see if I can come with a patch in the next days.
> > > 
> > > Aurelien
> > > 
> > > -- 
> > > Aurelien Jarno  GPG: 4096R/1DDD8C9B
> > > aurel...@aurel32.net http://www.aurel32.net
> > > 
> > > 
> > 
> > Thanks for the reply.  I am looking forward to the patch. :)
> 
> I have opened a bug upstream and sent a patch to the libc-alpha mailing
> list. I'll include the patch in the next upload, either today or
> tomorrow.

Unfortunately I have been pointed on the libc-alpha mailing list that
it doesn't work if another file which includes 
(e.g. ) is included before . The problem is
that the __UAPI_DEF_IF_* constants are set to 1 in 
even if  is not included.

Even if we are able to workaround that on the glibc side (for example
by checking for _LINUX_IF_H), the following sequence doesn't work:
  #include 
  #include 
  #include 

In that case  tries to redefine the same enum and struct
already defined by , which is obviously a kernel header bug.

I am therefore cloning this bug to the kernel headers. A fix is needed
on both sides.

-- 
Aurelien Jarno  GPG: 4096R/1DDD8C9B
aurel...@aurel32.net http://www.aurel32.net



Processed: Re: Bug#824442: and conflict needs to be resolved

2017-04-10 Thread Debian Bug Tracking System
Processing commands for cont...@bugs.debian.org:

> tag 824442 - pending
Bug #824442 [src:glibc]  and  conflict needs to be 
resolved
Removed tag(s) pending.
> clone 824442 -1
Bug #824442 [src:glibc]  and  conflict needs to be 
resolved
Bug 824442 cloned as bug 860013
> reassign -1 linux
Bug #860013 [src:glibc]  and  conflict needs to be 
resolved
Bug reassigned from package 'src:glibc' to 'linux'.
Ignoring request to alter found versions of bug #860013 to the same values 
previously set
Ignoring request to alter fixed versions of bug #860013 to the same values 
previously set
> retitle -1 linux: libc-compat.h sets __UAPI_DEF_IF_* to 1 even when if.h 
> hasn't been included
Bug #860013 [linux]  and  conflict needs to be resolved
Changed Bug title to 'linux: libc-compat.h sets __UAPI_DEF_IF_* to 1 even when 
if.h hasn't been included' from ' and  conflict needs to 
be resolved'.
> thanks
Stopping processing here.

Please contact me if you need assistance.
-- 
824442: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=824442
860013: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=860013
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems



[glibc] 01/01: Revert "patches/any/submitted-if.h-conflict.diff: new patch to fix the conflict between and . Closes: #824442."

2017-04-10 Thread Aurelien Jarno
This is an automated email from the git hooks/post-receive script.

aurel32 pushed a commit to branch sid
in repository glibc.

commit 0945cf60fcfe0cf0383f2389aa42e90ea0881034
Author: Aurelien Jarno 
Date:   Mon Apr 10 10:49:17 2017 +0200

Revert "patches/any/submitted-if.h-conflict.diff: new patch to fix the 
conflict between  and .  Closes: #824442."

This reverts commit b891546d948f797c034a9ac49f07a4faf1426025.

It still has some bugs.
---
 debian/changelog|  2 -
 debian/patches/any/submitted-if.h-conflict.diff | 74 -
 debian/patches/series   |  1 -
 3 files changed, 77 deletions(-)

diff --git a/debian/changelog b/debian/changelog
index 30f41e9..909783e 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -21,8 +21,6 @@ glibc (2.24-10) UNRELEASED; urgency=medium
 regressions when building a +debXuX version. That way changes in the
 environment (e.g.: kernel) do not prevent security or stable versions
 to be built.
-  * patches/any/submitted-if.h-conflict.diff: new patch to fix the conflict
-between  and .  Closes: #824442.
 
  -- Aurelien Jarno   Mon, 16 Jan 2017 18:46:54 +0100
 
diff --git a/debian/patches/any/submitted-if.h-conflict.diff 
b/debian/patches/any/submitted-if.h-conflict.diff
deleted file mode 100644
index d9c7de4..000
--- a/debian/patches/any/submitted-if.h-conflict.diff
+++ /dev/null
@@ -1,74 +0,0 @@
-2017-04-09 Aurelien Jarno  
-
-   [BZ #21367]
-   * sysdeps/gnu/net/if.h: wrap IFF_* enums in
-   !defined __UAPI_DEF_IF_NET_DEVICE_FLAGS defines.
-   (struct ifmap): Wrap in !defined __UAPI_DEF_IF_IFMAP defines.
-   (struct ifreq): Wrap in !defined __UAPI_DEF_IF_IFREQ defines.
-   (struct ifconf): Wrap in !defined __UAPI_DEF_IF_IFCONF defines.
-
 a/sysdeps/gnu/net/if.h
-+++ b/sysdeps/gnu/net/if.h
-@@ -39,6 +39,7 @@ struct if_nameindex
- 
- #ifdef __USE_MISC
- /* Standard interface flags. */
-+# ifndef __UAPI_DEF_IF_NET_DEVICE_FLAGS
- enum
-   {
- IFF_UP = 0x1, /* Interface is up.  */
-@@ -79,6 +80,7 @@ enum
- IFF_DYNAMIC = 0x8000  /* Dialup device with changing addresses.  */
- # define IFF_DYNAMIC  IFF_DYNAMIC
-   };
-+# endif /* !defined __UAPI_DEF_IF_NET_DEVICE_FLAGS */
- 
- /* The ifaddr structure contains information about one address of an
-interface.  They are maintained by the different address families,
-@@ -108,6 +110,7 @@ struct ifaddr
-handy for debugging things. The set side is fine for now and being
-very small might be worth keeping for clean configuration.  */
- 
-+# ifndef __UAPI_DEF_IF_IFMAP
- struct ifmap
-   {
- unsigned long int mem_start;
-@@ -118,11 +121,13 @@ struct ifmap
- unsigned char port;
- /* 3 bytes spare */
-   };
-+# endif /* !defined __UAPI_DEF_IF_IFMAP */
- 
- /* Interface request structure used for socket ioctl's.  All interface
-ioctl's must have parameter definitions which begin with ifr_name.
-The remainder may be interface specific.  */
- 
-+# ifndef __UAPI_DEF_IF_IFREQ
- struct ifreq
-   {
- # define IFHWADDRLEN  6
-@@ -148,6 +153,8 @@ struct ifreq
-   __caddr_t ifru_data;
-   } ifr_ifru;
-   };
-+# endif /* !__UAPI_DEF_IF_IFREQ */
-+
- # define ifr_name ifr_ifrn.ifrn_name  /* interface name   */
- # define ifr_hwaddr   ifr_ifru.ifru_hwaddr/* MAC address  */
- # define ifr_addr ifr_ifru.ifru_addr  /* address  */
-@@ -173,6 +180,7 @@ struct ifreq
-configuration for machine (useful for programs which must know all
-networks accessible).  */
- 
-+# ifndef __UAPI_DEF_IF_IFCONF
- struct ifconf
-   {
- int   ifc_len;/* Size of buffer.  */
-@@ -182,6 +190,7 @@ struct ifconf
-   struct ifreq *ifcu_req;
-   } ifc_ifcu;
-   };
-+# endif /* !defined __UAPI_DEF_IF_IFCONF */
- # define ifc_buf  ifc_ifcu.ifcu_buf   /* Buffer address.  */
- # define ifc_req  ifc_ifcu.ifcu_req   /* Array of structures.  */
- # define _IOT_ifconf _IOT(_IOTS(struct ifconf),1,0,0,0,0) /* not right */
diff --git a/debian/patches/series b/debian/patches/series
index 0aebc6c..16e7df0 100644
--- a/debian/patches/series
+++ b/debian/patches/series
@@ -218,4 +218,3 @@ any/submitted-dlopen-noload.diff
 any/submitted-perl-inc.diff
 any/submitted-unicode-9.0.0.diff
 any/cvs-remove-pid-tid-cache-clone.diff
-any/submitted-if.h-conflict.diff

-- 
Alioth's /usr/local/bin/git-commit-notice on 
/srv/git.debian.org/git/pkg-glibc/glibc.git