Hello community, here is the log from the commit of package acl for openSUSE:Factory checked in at 2012-12-28 14:55:02 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Comparing /work/SRC/openSUSE:Factory/acl (Old) and /work/SRC/openSUSE:Factory/.acl.new (New) ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "acl", Maintainer is "[email protected]" Changes: -------- --- /work/SRC/openSUSE:Factory/acl/acl.changes 2012-09-17 13:47:29.000000000 +0200 +++ /work/SRC/openSUSE:Factory/.acl.new/acl.changes 2012-12-28 14:55:05.000000000 +0100 @@ -1,0 +2,7 @@ +Wed Dec 26 18:22:59 UTC 2012 - [email protected] + +- Use OS byteswapping routines, application already Includes + "endian.h" but then goes ahead defining ad-hoc equivalent + functionality (0001-Use-OS-byteswapping-macros.patch) + +------------------------------------------------------------------- New: ---- 0001-Use-OS-byteswapping-macros.patch ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Other differences: ------------------ ++++++ acl.spec ++++++ --- /var/tmp/diff_new_pack.yu7I1L/_old 2012-12-28 14:55:07.000000000 +0100 +++ /var/tmp/diff_new_pack.yu7I1L/_new 2012-12-28 14:55:07.000000000 +0100 @@ -20,7 +20,7 @@ %define lname libacl1 BuildRequires: libattr-devel Summary: Commands for Manipulating POSIX Access Control Lists -License: GPL-2.0+ ; LGPL-2.1+ +License: GPL-2.0+ and LGPL-2.1+ Group: System/Filesystems Version: 2.2.51 Release: 0 @@ -28,6 +28,7 @@ Source2: baselibs.conf Patch0: builddefs.in.diff Patch1: acl-fiximplicit.patch +Patch2: 0001-Use-OS-byteswapping-macros.patch Url: http://download.savannah.gnu.org/releases-noredirect/acl/ BuildRoot: %{_tmppath}/%{name}-%{version}-build @@ -74,7 +75,7 @@ %setup -q -n acl-%version %patch0 -p1 %patch1 -p1 - +%patch2 -p1 %build export OPTIMIZER="$RPM_OPT_FLAGS -fPIC" export DEBUG=-DNDEBUG ++++++ 0001-Use-OS-byteswapping-macros.patch ++++++ >From 100baaf89cdcb01a1c8d4b05782d7fba56813f96 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Cristian=20Rodr=C3=ADguez?= <[email protected]> Date: Wed, 26 Dec 2012 15:16:40 -0300 Subject: [PATCH] Use OS byteswapping macros while the code already includes "endian.h" it goes ahead redefining byteswapping routines, thus defeating the very purpose of the header. --- libacl/byteorder.h | 25 ++++++++----------------- 1 file changed, 8 insertions(+), 17 deletions(-) diff --git a/libacl/byteorder.h b/libacl/byteorder.h index 05f5d87..65c87f3 100644 --- a/libacl/byteorder.h +++ b/libacl/byteorder.h @@ -17,21 +17,12 @@ #include <endian.h> -#if __BYTE_ORDER == __BIG_ENDIAN -# define cpu_to_le16(w16) le16_to_cpu(w16) -# define le16_to_cpu(w16) ((u_int16_t)((u_int16_t)(w16) >> 8) | \ - (u_int16_t)((u_int16_t)(w16) << 8)) -# define cpu_to_le32(w32) le32_to_cpu(w32) -# define le32_to_cpu(w32) ((u_int32_t)( (u_int32_t)(w32) >>24) | \ - (u_int32_t)(((u_int32_t)(w32) >> 8) & 0xFF00) | \ - (u_int32_t)(((u_int32_t)(w32) << 8) & 0xFF0000) | \ - (u_int32_t)( (u_int32_t)(w32) <<24)) -#elif __BYTE_ORDER == __LITTLE_ENDIAN -# define cpu_to_le16(w16) ((u_int16_t)(w16)) -# define le16_to_cpu(w16) ((u_int16_t)(w16)) -# define cpu_to_le32(w32) ((u_int32_t)(w32)) -# define le32_to_cpu(w32) ((u_int32_t)(w32)) -#else -# error unknown endianess? -#endif +# define cpu_to_le16(w16) htole16(w16) + +# define le16_to_cpu(w16) le16toh(w16) + +# define cpu_to_le32(w32) htole32(w32) + +# define le32_to_cpu(w32) le32toh(w32) + -- 1.8.0.2 -- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
