Add patch armel_ftbfs.patch: Rename struct user to struct _user in the
authorgregor herrmann <gregoa@debian.org>
Thu, 10 Feb 2011 15:51:13 +0000 (15:51 -0000)
committergregor herrmann <gregoa@debian.org>
Thu, 10 Feb 2011 15:51:13 +0000 (15:51 -0000)
code to avoid naming conflict caused by including <sys/user.h> as part of
the armel build. Taken from Ubuntu; not needed currently in Debian but
saves Ubuntu from carrying the patch and might be helpful later.

debian/changelog
debian/patches/armel_ftbfs.patch [new file with mode: 0644]
debian/patches/series

index 8538f9ea3d6a38ffead0f87c0337e0169e8d5fdf..927864a61a69c833336f375eefe2ee2b92edd0c6 100644 (file)
@@ -6,6 +6,10 @@ iodine (0.6.0~rc1-5) UNRELEASED; urgency=low
     Thanks to "Debian bug at v.nix.is" for the bug report.
   * Add a logcheck filter; thanks to martin f krafft for the bug report and
     the logcheck file (closes: #612721).
+  * Add patch armel_ftbfs.patch: Rename struct user to struct _user in the
+    code to avoid naming conflict caused by including <sys/user.h> as part of
+    the armel build. Taken from Ubuntu; not needed currently in Debian but
+    saves Ubuntu from carrying the patch and might be helpful later.
 
  -- gregor herrmann <gregoa@debian.org>  Thu, 10 Feb 2011 15:33:18 +0100
 
diff --git a/debian/patches/armel_ftbfs.patch b/debian/patches/armel_ftbfs.patch
new file mode 100644 (file)
index 0000000..84a3e56
--- /dev/null
@@ -0,0 +1,50 @@
+Description: Rename struct user to struct _user in the code to avoid naming
+ conflict caused by including <sys/user.h> as part of the armel build. Fixes
+ FTBFS.
+Origin: Ubuntu
+Reviewed-by: gregor herrmann <gregoa@debian.org>
+Last-Update: 2011-02-10
+
+--- a/src/user.c
++++ b/src/user.c
+@@ -33,7 +33,7 @@
+ #include "encoding.h"
+ #include "user.h"
+-struct user users[USERS];
++struct _user users[USERS];
+ int
+ init_users(in_addr_t my_ip, int netbits)
+@@ -58,7 +58,7 @@
+       maxusers = (1 << (32-netbits)) - 3; /* 3: Net addr, broadcast addr, iodined addr */
+       
+-      memset(users, 0, USERS * sizeof(struct user));
++      memset(users, 0, USERS * sizeof(struct _user));
+       for (i = 0; i < USERS; i++) {
+               in_addr_t ip;
+               users[i].id = i;
+--- a/src/user.h
++++ b/src/user.h
+@@ -33,7 +33,10 @@
+ #define QMEMDATA_LEN 15
+ /* Max advisable: 36/2 = 18. Total mem usage: QMEMDATA_LEN * USERS * 6 bytes */
+-struct user {
++/* Renamed to struct _user to avoid naming conflict with struct user found in <sys/user.h>
++ * which gets included in some builds (armel) */
++
++struct _user {
+       char id;
+       int active;
+       int disabled;
+@@ -73,7 +76,7 @@
+ #endif
+ };
+-extern struct user users[USERS];
++extern struct _user users[USERS];
+ int init_users(in_addr_t, int);
+ const char* users_get_first_ip();
index 16bc9285e06c9b5c2a322e7adcbd887a946e8112..6927ef907185f3256c2af40f828ef6aefa6af917 100644 (file)
@@ -2,3 +2,4 @@
 spelling.patch
 manpage.patch
 uninitialized-inside_topdomain.patch
+armel_ftbfs.patch