[PATCH v2 1/1] Fix uClibc build on 64bit platforms by including stdint.h

Jeremy Allison jra at samba.org
Thu Aug 23 23:41:48 UTC 2018


On Mon, Aug 20, 2018 at 10:21:12AM +0200, Volker Lendecke via samba-technical wrote:
> On Sat, Aug 18, 2018 at 09:43:00AM +0200, Bernd Kuhls via samba-technical wrote:
> > Fixes an error detected by buildroot autobuilders:
> > http://autobuild.buildroot.net/results/573/573e2268e205e10d1352fa81122d8f225fdb4575/build-end.log
> > 
> > /home/rclinux/rc-buildroot-test/scripts/instance-1/output/host/mips64el-buildroot-linux-uclibc/sysroot/usr/include/stdint.h:122:27:
> > error: conflicting types for 'uintptr_t'
> >  typedef unsigned long int uintptr_t;
> >                            ^
> > In file included from ../lib/ldb/tests/ldb_msg.c:17:0:
> > ../third_party/cmocka/cmocka.h:126:28: note: previous declaration of 'uintptr_t' was here
> >        typedef unsigned int uintptr_t;
> > 
> > The define __WORDSIZE is missing when cmocka.h decides how to
> > define uintptr_t, this patch includes stdint.h when needed.
> > 
> > Patch sent upstream:
> > https://lists.samba.org/archive/samba-technical/2018-January/125306.html
> 
> I still consider this to be a cmocka bug, but as cmocka seems to not
> cover this case, RB+. Another team reviewer?

LGTM. RB+ and pushed.



More information about the samba-technical mailing list