[
https://issues.apache.org/jira/browse/ZOOKEEPER-2049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14156100#comment-14156100
]
Hadoop QA commented on ZOOKEEPER-2049:
--------------------------------------
-1 overall. Here are the results of testing the latest attachment
http://issues.apache.org/jira/secure/attachment/12671900/ZOOKEEPER-2049.patch
against trunk revision 1628224.
+1 @author. The patch does not contain any @author tags.
-1 tests included. The patch doesn't appear to include any new or modified
tests.
Please justify why no new tests are needed for this
patch.
Also please list what manual steps were performed to
verify this patch.
+1 javadoc. The javadoc tool did not generate any warning messages.
+1 javac. The applied patch does not increase the total number of javac
compiler warnings.
+1 findbugs. The patch does not introduce any new Findbugs (version 2.0.3)
warnings.
-1 release audit. The applied patch generated 2 release audit warnings
(more than the trunk's current 0 warnings).
-1 core tests. The patch failed core unit tests.
+1 contrib tests. The patch passed contrib unit tests.
Test results:
https://builds.apache.org/job/PreCommit-ZOOKEEPER-Build/2364//testReport/
Release audit warnings:
https://builds.apache.org/job/PreCommit-ZOOKEEPER-Build/2364//artifact/trunk/patchprocess/patchReleaseAuditProblems.txt
Findbugs warnings:
https://builds.apache.org/job/PreCommit-ZOOKEEPER-Build/2364//artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html
Console output:
https://builds.apache.org/job/PreCommit-ZOOKEEPER-Build/2364//console
This message is automatically generated.
> Yosemite build failure: htonll conflict
> ---------------------------------------
>
> Key: ZOOKEEPER-2049
> URL: https://issues.apache.org/jira/browse/ZOOKEEPER-2049
> Project: ZooKeeper
> Issue Type: Bug
> Affects Versions: 3.4.5, 3.4.6, 3.5.0
> Environment: OSX 10.10 (BETA3), Apple LLVM version 6.0
> (clang-600.0.51) (based on LLVM 3.5svn)
> Reporter: Till Toenshoff
> Assignee: Till Toenshoff
> Fix For: 3.4.7, 3.5.1, 3.6.0
>
> Attachments: ZOOKEEPER-2049.patch
>
>
> recordio.h defines {{htonll}} which conflicts with Apple's equally named
> implementation from within arpa/inet.h.
> {noformat}
> gcc -DHAVE_CONFIG_H -I. -I. -I. -I./include -I./tests -I./generated -Wall
> -Werror -g -O2 -D_GNU_SOURCE -MT zk_log.lo -MD -MP -MF .deps/zk_log.Tpo -c
> src/zk_log.c -fno-common -DPIC -o zk_log.o
> In file included from src/recordio.c:19:
> ./include/recordio.h:76:9: error: expected ')'
> int64_t htonll(int64_t v);
> ^
> /usr/include/sys/_endian.h:141:25: note: expanded from macro 'htonll'
> #define htonll(x) __DARWIN_OSSwapInt64(x)
> ^
> /usr/include/libkern/_OSByteOrder.h:78:30: note: expanded from macro
> '__DARWIN_OSSwapInt64'
> (__builtin_constant_p(x) ? __DARWIN_OSSwapConstInt64(x) : _OSSwapInt64(x))
> ^
> ./include/recordio.h:76:9: note: to match this '('
> /usr/include/sys/_endian.h:141:25: note: expanded from macro 'htonll'
> #define htonll(x) __DARWIN_OSSwapInt64(x)
> ^
> /usr/include/libkern/_OSByteOrder.h:78:5: note: expanded from macro
> '__DARWIN_OSSwapInt64'
> (__builtin_constant_p(x) ? __DARWIN_OSSwapConstInt64(x) : _OSSwapInt64(x))
> ^
> In file included from src/recordio.c:19:
> ./include/recordio.h:76:9: error: conflicting types for '__builtin_constant_p'
> int64_t htonll(int64_t v);
> ^
> /usr/include/sys/_endian.h:141:25: note: expanded from macro 'htonll'
> #define htonll(x) __DARWIN_OSSwapInt64(x)
> ^
> /usr/include/libkern/_OSByteOrder.h:78:6: note: expanded from macro
> '__DARWIN_OSSwapInt64'
> (__builtin_constant_p(x) ? __DARWIN_OSSwapConstInt64(x) : _OSSwapInt64(x))
> ^
> ./include/recordio.h:76:9: note: '__builtin_constant_p' is a builtin with
> type 'int ()'
> /usr/include/sys/_endian.h:141:25: note: expanded from macro 'htonll'
> #define htonll(x) __DARWIN_OSSwapInt64(x)
> ^
> /usr/include/libkern/_OSByteOrder.h:78:6: note: expanded from macro
> '__DARWIN_OSSwapInt64'
> (__builtin_constant_p(x) ? __DARWIN_OSSwapConstInt64(x) : _OSSwapInt64(x))
> ^
> In file included from generated/zookeeper.jute.c:20:
> In file included from ./generated/zookeeper.jute.h:21:
> ./include/recordio.h:76:9: error: expected ')'
> int64_t htonll(int64_t v);
> ^
> /usr/include/sys/_endian.h:141:25: note: expanded from macro 'htonll'
> #define htonll(x) __DARWIN_OSSwapInt64(x)
> ^
> /usr/include/libkern/_OSByteOrder.h:78:30: note: expanded from macro
> '__DARWIN_OSSwapInt64'
> (__builtin_constant_p(x) ? __DARWIN_OSSwapConstInt64(x) : _OSSwapInt64(x))
> ^
> ./include/recordio.h:76:9: note: to match this '('
> /usr/include/sys/_endian.h:141:25: note: expanded from macro 'htonll'
> #define htonll(x) __DARWIN_OSSwapInt64(x)
> ^
> /usr/include/libkern/_OSByteOrder.h:78:5: note: expanded from macro
> '__DARWIN_OSSwapInt64'
> (__builtin_constant_p(x) ? __DARWIN_OSSwapConstInt64(x) : _OSSwapInt64(x))
> ^
> In file included from generated/zookeeper.jute.c:20:
> In file included from ./generated/zookeeper.jute.h:21:
> ./include/recordio.h:76:9: error: conflicting types for '__builtin_constant_p'
> int64_t htonll(int64_t v);
> ^
> /usr/include/sys/_endian.h:141:25: note: expanded from macro 'htonll'
> #define htonll(x) __DARWIN_OSSwapInt64(x)
> ^
> /usr/include/libkern/_OSByteOrder.h:78:6: note: expanded from macro
> '__DARWIN_OSSwapInt64'
> (__builtin_constant_p(x) ? __DARWIN_OSSwapConstInt64(x) : _OSSwapInt64(x))
> ^
> ./include/recordio.h:76:9: note: '__builtin_constant_p' is a builtin with
> type 'int ()'
> /usr/include/sys/_endian.h:141:25: note: expanded from macro 'htonll'
> #define htonll(x) __DARWIN_OSSwapInt64(x)
> ^
> /usr/include/libkern/_OSByteOrder.h:78:6: note: expanded from macro
> '__DARWIN_OSSwapInt64'
> (__builtin_constant_p(x) ? __DARWIN_OSSwapConstInt64(x) : _OSSwapInt64(x))
> ^
> In file included from src/zookeeper.c:27:
> In file included from ./include/zookeeper.h:34:
> ./include/recordio.h:76:9: error: expected ')'
> int64_t htonll(int64_t v);
> ^
> /usr/include/sys/_endian.h:141:25: note: expanded from macro 'htonll'
> #define htonll(x) __DARWIN_OSSwapInt64(x)
> ^
> /usr/include/libkern/_OSByteOrder.h:78:30: note: expanded from macro
> '__DARWIN_OSSwapInt64'
> (__builtin_constant_p(x) ? __DARWIN_OSSwapConstInt64(x) : _OSSwapInt64(x))
> ^
> ./include/recordio.h:76:9: note: to match this '('
> /usr/include/sys/_endian.h:141:25: note: expanded from macro 'htonll'
> #define htonll(x) __DARWIN_OSSwapInt64(x)
> ^
> /usr/include/libkern/_OSByteOrder.h:78:5: note: expanded from macro
> '__DARWIN_OSSwapInt64'
> (__builtin_constant_p(x) ? __DARWIN_OSSwapConstInt64(x) : _OSSwapInt64(x))
> ^
> In file included from src/zookeeper.c:27:
> In file included from ./include/zookeeper.h:34:
> ./include/recordio.h:76:9: error: conflicting types for '__builtin_constant_p'
> int64_t htonll(int64_t v);
> ^
> /usr/include/sys/_endian.h:141:25: note: expanded from macro 'htonll'
> #define htonll(x) __DARWIN_OSSwapInt64(x)
> ^
> /usr/include/libkern/_OSByteOrder.h:78:6: note: expanded from macro
> '__DARWIN_OSSwapInt64'
> (__builtin_constant_p(x) ? __DARWIN_OSSwapConstInt64(x) : _OSSwapInt64(x))
> ^
> ./include/recordio.h:76:9: note: '__builtin_constant_p' is a builtin with
> type 'int ()'
> /usr/include/sys/_endian.h:141:25: note: expanded from macro 'htonll'
> #define htonll(x) __DARWIN_OSSwapInt64(x)
> ^
> /usr/include/libkern/_OSByteOrder.h:78:6: note: expanded from macro
> '__DARWIN_OSSwapInt64'
> (__builtin_constant_p(x) ? __DARWIN_OSSwapConstInt64(x) : _OSSwapInt64(x))
> ^
> In file included from src/zk_hashtable.c:19:
> In file included from src/zk_hashtable.h:22:
> In file included from ./include/zookeeper.h:34:
> ./include/recordio.h:76:9: error: expected ')'
> int64_t htonll(int64_t v);
> ^
> /usr/include/sys/_endian.h:141:25: note: expanded from macro 'htonll'
> #define htonll(x) __DARWIN_OSSwapInt64(x)
> ^
> /usr/include/libkern/_OSByteOrder.h:78:30: note: expanded from macro
> '__DARWIN_OSSwapInt64'
> (__builtin_constant_p(x) ? __DARWIN_OSSwapConstInt64(x) : _OSSwapInt64(x))
> ^
> ./include/recordio.h:76:9: note: to match this '('
> /usr/include/sys/_endian.h:141:25: note: expanded from macro 'htonll'
> #define htonll(x) __DARWIN_OSSwapInt64(x)
> ^
> /usr/include/libkern/_OSByteOrder.h:78:5: note: expanded from macro
> '__DARWIN_OSSwapInt64'
> (__builtin_constant_p(x) ? __DARWIN_OSSwapConstInt64(x) : _OSSwapInt64(x))
> ^
> In file included from src/zk_hashtable.c:19:
> In file included from src/zk_hashtable.h:22:
> In file included from ./include/zookeeper.h:34:
> ./include/recordio.h:76:9: error: conflicting types for '__builtin_constant_p'
> int64_t htonll(int64_t v);
> ^
> /usr/include/sys/_endian.h:141:25: note: expanded from macro 'htonll'
> #define htonll(x) __DARWIN_OSSwapInt64(x)
> ^
> /usr/include/libkern/_OSByteOrder.h:78:6: note: expanded from macro
> '__DARWIN_OSSwapInt64'
> (__builtin_constant_p(x) ? __DARWIN_OSSwapConstInt64(x) : _OSSwapInt64(x))
> ^
> ./include/recordio.h:76:9: note: '__builtin_constant_p' is a builtin with
> type 'int ()'
> /usr/include/sys/_endian.h:141:25: note: expanded from macro 'htonll'
> #define htonll(x) __DARWIN_OSSwapInt64(x)
> ^
> /usr/include/libkern/_OSByteOrder.h:78:6: note: expanded from macro
> '__DARWIN_OSSwapInt64'
> (__builtin_constant_p(x) ? __DARWIN_OSSwapConstInt64(x) : _OSSwapInt64(x))
> ^
> src/recordio.c:83:9: error: expected ')'
> int64_t htonll(int64_t v)
> ^
> /usr/include/sys/_endian.h:141:25: note: expanded from macro 'htonll'
> #define htonll(x) __DARWIN_OSSwapInt64(x)
> ^
> /usr/include/libkern/_OSByteOrder.h:78:30: note: expanded from macro
> '__DARWIN_OSSwapInt64'
> (__builtin_constant_p(x) ? __DARWIN_OSSwapConstInt64(x) : _OSSwapInt64(x))
> ^
> src/recordio.c:83:9: note: to match this '('
> /usr/include/sys/_endian.h:141:25: note: expanded from macro 'htonll'
> #define htonll(x) __DARWIN_OSSwapInt64(x)
> ^
> /usr/include/libkern/_OSByteOrder.h:78:5: note: expanded from macro
> '__DARWIN_OSSwapInt64'
> (__builtin_constant_p(x) ? __DARWIN_OSSwapConstInt64(x) : _OSSwapInt64(x))
> ^
> src/recordio.c:83:9: error: conflicting types for '__builtin_constant_p'
> int64_t htonll(int64_t v)
> ^
> /usr/include/sys/_endian.h:141:25: note: expanded from macro 'htonll'
> #define htonll(x) __DARWIN_OSSwapInt64(x)
> ^
> /usr/include/libkern/_OSByteOrder.h:78:6: note: expanded from macro
> '__DARWIN_OSSwapInt64'
> (__builtin_constant_p(x) ? __DARWIN_OSSwapConstInt64(x) : _OSSwapInt64(x))
> ^
> ./include/recordio.h:76:9: note: '__builtin_constant_p' is a builtin with
> type 'int ()'
> int64_t htonll(int64_t v);
> ^
> /usr/include/sys/_endian.h:141:25: note: expanded from macro 'htonll'
> #define htonll(x) __DARWIN_OSSwapInt64(x)
> ^
> /usr/include/libkern/_OSByteOrder.h:78:6: note: expanded from macro
> '__DARWIN_OSSwapInt64'
> (__builtin_constant_p(x) ? __DARWIN_OSSwapConstInt64(x) : _OSSwapInt64(x))
> ^
> src/recordio.c:83:9: error: definition of builtin function
> '__builtin_constant_p'
> int64_t htonll(int64_t v)
> ^
> /usr/include/sys/_endian.h:141:25: note: expanded from macro 'htonll'
> #define htonll(x) __DARWIN_OSSwapInt64(x)
> ^
> /usr/include/libkern/_OSByteOrder.h:78:6: note: expanded from macro
> '__DARWIN_OSSwapInt64'
> (__builtin_constant_p(x) ? __DARWIN_OSSwapConstInt64(x) : _OSSwapInt64(x))
> ^
> In file included from src/zk_log.c:23:
> In file included from ./include/zookeeper_log.h:22:
> In file included from ./include/zookeeper.h:34:
> ./include/recordio.h:76:9: error: expected ')'
> int64_t htonll(int64_t v);
> ^
> /usr/include/sys/_endian.h:141:25: note: expanded from macro 'htonll'
> #define htonll(x) __DARWIN_OSSwapInt64(x)
> ^
> /usr/include/libkern/_OSByteOrder.h:78:30: note: expanded from macro
> '__DARWIN_OSSwapInt64'
> (__builtin_constant_p(x) ? __DARWIN_OSSwapConstInt64(x) : _OSSwapInt64(x))
> ^
> ./include/recordio.h:76:9: note: to match this '('
> /usr/include/sys/_endian.h:141:25: note: expanded from macro 'htonll'
> #define htonll(x) __DARWIN_OSSwapInt64(x)
> ^
> /usr/include/libkern/_OSByteOrder.h:78:5: note: expanded from macro
> '__DARWIN_OSSwapInt64'
> (__builtin_constant_p(x) ? __DARWIN_OSSwapConstInt64(x) : _OSSwapInt64(x))
> ^
> In file included from src/zk_log.c:23:
> In file included from ./include/zookeeper_log.h:22:
> In file included from ./include/zookeeper.h:34:
> ./include/recordio.h:76:9: error: conflicting types for '__builtin_constant_p'
> int64_t htonll(int64_t v);
> ^
> /usr/include/sys/_endian.h:141:25: note: expanded from macro 'htonll'
> #define htonll(x) __DARWIN_OSSwapInt64(x)
> ^
> /usr/include/libkern/_OSByteOrder.h:78:6: note: expanded from macro
> '__DARWIN_OSSwapInt64'
> (__builtin_constant_p(x) ? __DARWIN_OSSwapConstInt64(x) : _OSSwapInt64(x))
> ^
> ./include/recordio.h:76:9: note: '__builtin_constant_p' is a builtin with
> type 'int ()'
> /usr/include/sys/_endian.h:141:25: note: expanded from macro 'htonll'
> #define htonll(x) __DARWIN_OSSwapInt64(x)
> ^
> /usr/include/libkern/_OSByteOrder.h:78:6: note: expanded from macro
> '__DARWIN_OSSwapInt64'
> (__builtin_constant_p(x) ? __DARWIN_OSSwapConstInt64(x) : _OSSwapInt64(x))
> ^
> 2 errors generated.
> 5 errors generated.
> 2 errors generated.
> make[5]: *** [recordio.lo] Error 1
> make[5]: *** Waiting for unfinished jobs....
> 2 errors generated.
> make[5]: *** [zookeeper.jute.lo] Error 1
> make[5]: *** [zk_hashtable.lo] Error 1
> make[5]: *** [zk_log.lo] Error 1
> 2 errors generated.
> make[5]: *** [zookeeper.lo] Error 1
> make[4]: *** [all] Error 2
> make[3]: *** [zookeeper-3.4.5/src/c/libzookeeper_mt.la] Error 2
> make[3]: *** Waiting for unfinished jobs....
> ln -fs libleveldb.dylib.1.4 libleveldb.dylib
> ln -fs libleveldb.dylib.1.4 libleveldb.dylib.1
> make[2]: *** [all-recursive] Error 1
> make[1]: *** [all] Error 2
> make: *** [all-recursive] Error 1
> {noformat}
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)