openldap-bugs

openldap-bugs@openldap.org

1 participants
20963 discussions

Re: (ITS#8830) build hardcodes gcc
by hyc＠symas.com 01 Apr '18

01 Apr '18

vcunat(a)gmail.com wrote: > On 04/01/2018 12:12 PM, Howard Chu wrote: >> The rules you patched are only for generating gcov-compatible binaries. >> They aren't even invoked by "make test" so I don't see why this has any >> impact on your build at all. >> > > The build looks like this: > ... > running tests > ar rs liblmdb.a mdb.o midl.o > gcc -pthread -O2 -g -W -Wall -Wno-unused-parameter -Wbad-function-cast > -Wuninitialized mdb_stat.o liblmdb.a -o mdb_stat > make: gcc: Command not found What's the actual command line you used to invoke this? -- -- Howard Chu CTO, Symas Corp. http://www.symas.com Director, Highland Sun http://highlandsun.com/hyc/ Chief Architect, OpenLDAP http://www.openldap.org/project/

1 0

Re: (ITS#8830) build hardcodes gcc
by vcunat＠gmail.com 01 Apr '18

01 Apr '18

On 04/01/2018 12:12 PM, Howard Chu wrote: > The rules you patched are only for generating gcov-compatible binaries. > They aren't even invoked by "make test" so I don't see why this has any > impact on your build at all. > The build looks like this: ... running tests ar rs liblmdb.a mdb.o midl.o gcc -pthread -O2 -g -W -Wall -Wno-unused-parameter -Wbad-function-cast -Wuninitialized mdb_stat.o liblmdb.a -o mdb_stat make: gcc: Command not found

1 0

Re: (ITS#8830) build hardcodes gcc
by hyc＠symas.com 01 Apr '18

01 Apr '18

vcunat(a)gmail.com wrote: > On 04/01/2018 11:53 AM, Howard Chu wrote: >> The coverage flags have only been tested with gcc. What compiler are you >> using that also provides code coverage with these flags? > > This is clang on osx, kind of. > > NixOS.org is packaging lmdb for various "platforms", including osx. And > we run `make test` to be more confident about the result. That's what > now broke when we attempt lmdb: 0.9.21 -> 0.9.22 - and this patch fixed > the build (though I didn't really check anything; I don't really have > any osx machine myself). > > My experience in general is that clang tries to be compatible with gcc. The rules you patched are only for generating gcov-compatible binaries. They aren't even invoked by "make test" so I don't see why this has any impact on your build at all. -- -- Howard Chu CTO, Symas Corp. http://www.symas.com Director, Highland Sun http://highlandsun.com/hyc/ Chief Architect, OpenLDAP http://www.openldap.org/project/

1 0

Re: (ITS#8830) build hardcodes gcc
by vcunat＠gmail.com 01 Apr '18

01 Apr '18

On 04/01/2018 11:53 AM, Howard Chu wrote: > The coverage flags have only been tested with gcc. What compiler are you > using that also provides code coverage with these flags? This is clang on osx, kind of. NixOS.org is packaging lmdb for various "platforms", including osx. And we run `make test` to be more confident about the result. That's what now broke when we attempt lmdb: 0.9.21 -> 0.9.22 - and this patch fixed the build (though I didn't really check anything; I don't really have any osx machine myself). My experience in general is that clang tries to be compatible with gcc.

1 0

Re: (ITS#8830) build hardcodes gcc
by hyc＠symas.com 01 Apr '18

01 Apr '18

vcunat(a)gmail.com wrote: > Full_Name: Vladim.r &#268;un.t > Version: 0.9.22 (lmdb) > OS: Linux (NixOS) > URL: ftp://ftp.openldap.org/incoming/ > Submission from: (NULL) (178.255.168.34) >=20 >=20 > Some parts of the LMDB Makefile hardcode gcc, seemingly unnecessarily. = Please > apply the following patch or some similar fix. The coverage flags have only been tested with gcc. What compiler are you = using=20 that also provides code coverage with these flags? >=20 > commit 029031a68873bc3784a8561bd8e049efbd34f9d0 (HEAD) > Author: Vladim=C3=ADr Čun=C3=A1t <vcunat(a)gmail.com> > Date: Sun Apr 1 11:05:31 2018 +0200 >=20 > make: gcc -> $(CC) >=20 > diff --git a/libraries/liblmdb/Makefile b/libraries/liblmdb/Makefile > index f254511..612484e 100644 > --- a/libraries/liblmdb/Makefile > +++ b/libraries/liblmdb/Makefile > @@ -102,13 +102,13 @@ COV_OBJS=3Dxmdb.o xmidl.o > =20 > coverage: xmtest > for i in mtest*.c [0-9]*.c; do j=3D`basename \$$i .c`; $(MAKE) $$j.o= ; \ > - gcc -o x$$j $$j.o $(COV_OBJS) -pthread $(COV_FLAGS); \ > + $(CC) -o x$$j $$j.o $(COV_OBJS) -pthread $(COV_FLAGS); \ > rm -rf testdb; mkdir testdb; ./x$$j; done > gcov xmdb.c > gcov xmidl.c > =20 > xmtest: mtest.o xmdb.o xmidl.o > - gcc -o xmtest mtest.o xmdb.o xmidl.o -pthread $(COV_FLAGS) > + $(CC) -o xmtest mtest.o xmdb.o xmidl.o -pthread $(COV_FLAGS) > =20 > xmdb.o: mdb.c lmdb.h midl.h > $(CC) $(CFLAGS) -fPIC $(CPPFLAGS) -O0 $(COV_FLAGS) -c mdb.c -o $@ >=20 >=20 --=20 -- Howard Chu CTO, Symas Corp. http://www.symas.com Director, Highland Sun http://highlandsun.com/hyc/ Chief Architect, OpenLDAP http://www.openldap.org/project/

1 0

Re: (ITS#8829) Right shift build warning VS2017 mdb.c
by hyc＠symas.com 01 Apr '18

01 Apr '18

robin.rowe(a)cinepaint.org wrote: > Full_Name: Robin Rowe > Version: git > OS: Windows 10 > URL: ftp://ftp.openldap.org/incoming/ > Submission from: (NULL) (23.242.20.216) > > > Warning from VS2017: > > mdb.c(7455): warning C4333: '>>': right shift by too large amount, data loss > > 7455: SETDSZ(node, nsize); > > 960: #define SETDSZ(node,size) do { \ > 961: (node)->mn_lo = (size) & 0xffff; (node)->mn_hi = (size) >> 16;} while(0) > > Intentional and irrelevant. Closing this ITS. -- -- Howard Chu CTO, Symas Corp. http://www.symas.com Director, Highland Sun http://highlandsun.com/hyc/ Chief Architect, OpenLDAP http://www.openldap.org/project/

1 0

(ITS#8830) build hardcodes gcc
by vcunat＠gmail.com 01 Apr '18

01 Apr '18

Full_Name: Vladim.r &#268;un.t Version: 0.9.22 (lmdb) OS: Linux (NixOS) URL: ftp://ftp.openldap.org/incoming/ Submission from: (NULL) (178.255.168.34) Some parts of the LMDB Makefile hardcode gcc, seemingly unnecessarily. Please apply the following patch or some similar fix. commit 029031a68873bc3784a8561bd8e049efbd34f9d0 (HEAD) Author: Vladimír Čunát <vcunat(a)gmail.com> Date: Sun Apr 1 11:05:31 2018 +0200 make: gcc -> $(CC) diff --git a/libraries/liblmdb/Makefile b/libraries/liblmdb/Makefile index f254511..612484e 100644 --- a/libraries/liblmdb/Makefile +++ b/libraries/liblmdb/Makefile @@ -102,13 +102,13 @@ COV_OBJS=xmdb.o xmidl.o coverage: xmtest for i in mtest*.c [0-9]*.c; do j=`basename \$$i .c`; $(MAKE) $$j.o; \ - gcc -o x$$j $$j.o $(COV_OBJS) -pthread $(COV_FLAGS); \ + $(CC) -o x$$j $$j.o $(COV_OBJS) -pthread $(COV_FLAGS); \ rm -rf testdb; mkdir testdb; ./x$$j; done gcov xmdb.c gcov xmidl.c xmtest: mtest.o xmdb.o xmidl.o - gcc -o xmtest mtest.o xmdb.o xmidl.o -pthread $(COV_FLAGS) + $(CC) -o xmtest mtest.o xmdb.o xmidl.o -pthread $(COV_FLAGS) xmdb.o: mdb.c lmdb.h midl.h $(CC) $(CFLAGS) -fPIC $(CPPFLAGS) -O0 $(COV_FLAGS) -c mdb.c -o $@

1 0

(ITS#8829) Right shift build warning VS2017 mdb.c
by robin.rowe＠cinepaint.org 31 Mar '18

31 Mar '18

Full_Name: Robin Rowe Version: git OS: Windows 10 URL: ftp://ftp.openldap.org/incoming/ Submission from: (NULL) (23.242.20.216) Warning from VS2017: mdb.c(7455): warning C4333: '>>': right shift by too large amount, data loss 7455: SETDSZ(node, nsize); 960: #define SETDSZ(node,size) do { \ 961: (node)->mn_lo = (size) & 0xffff; (node)->mn_hi = (size) >> 16;} while(0)

1 0

Re: (ITS#8827) lmdb from master fails to build with Visual Studio 2015.
by quanah＠symas.com 29 Mar '18

29 Mar '18

--On Thursday, March 29, 2018 9:20 PM +0000 rion4ik(a)gmail.com wrote: > Why do you use this mail-based crap? We won't be for much longer: <http://www.openldap.org/lists/openldap-devel/201801/msg00017.html> There are valid reasons to avoid Github. --Quanah -- Quanah Gibson-Mount Product Architect Symas Corporation Packaged, certified, and supported LDAP solutions powered by OpenLDAP: <http://www.symas.com>

1 0

Re: (ITS#8827) lmdb from master fails to build with Visual Studio 2015.
by rion4ik＠gmail.com 29 Mar '18

29 Mar '18

--000000000000ac244b056892dafb Content-Type: text/plain; charset="UTF-8" logs: c:\src\lmdb\libraries\liblmdb\mdb.c(55): error C2059: syntax error: '(' 1>c:\src\lmdb\libraries\liblmdb\mdb.c(61): error C2143: syntax error: missing '{' before '*' 1>c:\src\lmdb\libraries\liblmdb\mdb.c(68): error C2059: syntax error: '(' 1>c:\src\lmdb\libraries\liblmdb\mdb.c(75): error C2143: syntax error: missing '{' before '*' 1>c:\src\lmdb\libraries\liblmdb\mdb.c(77): error C2059: syntax error: '(' 1>c:\src\lmdb\libraries\liblmdb\mdb.c(79): error C2143: syntax error: missing '{' before '*' 1>c:\src\lmdb\libraries\liblmdb\mdb.c(4339): error C2063: 'NtCreateSection': not a function 1>c:\src\lmdb\libraries\liblmdb\mdb.c(4346): error C2063: 'NtMapViewOfSection': not a function 1>c:\src\lmdb\libraries\liblmdb\mdb.c(4350): error C2063: 'NtClose': not a function 1>c:\src\lmdb\libraries\liblmdb\mdb.c(4704): error C2065: 'NtCloseFunc': undeclared identifier 1>c:\src\lmdb\libraries\liblmdb\mdb.c(4704): error C2059: syntax error: ')' 1>c:\src\lmdb\libraries\liblmdb\mdb.c(4707): error C2065: 'NtMapViewOfSectionFunc': undeclared identifier 1>c:\src\lmdb\libraries\liblmdb\mdb.c(4707): error C2059: syntax error: ')' 1>c:\src\lmdb\libraries\liblmdb\mdb.c(4710): error C2065: 'NtCreateSectionFunc': undeclared identifier 1>c:\src\lmdb\libraries\liblmdb\mdb.c(4710): error C2059: syntax error: ')' 1>c:\src\lmdb\libraries\liblmdb\mdb.c(8314): warning C4333: '>>': right shift by too large amount, data los PS github's bug tracker is much more user-friendly, it even has command line interface for old-school guys. Why do you use this mail-based crap? Best Regards, Sergey --000000000000ac244b056892dafb Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable <div dir=3D"ltr">logs:<div><br></div><div><div>c:\src\lmdb\libraries\liblmd= b\mdb.c(55): error C2059: syntax error: '('</div><div>1>c:\src\l= mdb\libraries\liblmdb\mdb.c(61): error C2143: syntax error: missing '{&= #39; before '*'</div><div>1>c:\src\lmdb\libraries\liblmdb\mdb.c(= 68): error C2059: syntax error: '('</div><div>1>c:\src\lmdb\libr= aries\liblmdb\mdb.c(75): error C2143: syntax error: missing '{' bef= ore '*'</div><div>1>c:\src\lmdb\libraries\liblmdb\mdb.c(77): err= or C2059: syntax error: '('</div><div>1>c:\src\lmdb\libraries\li= blmdb\mdb.c(79): error C2143: syntax error: missing '{' before &#39= ;*'</div><div>1>c:\src\lmdb\libraries\liblmdb\mdb.c(4339): error C20= 63: 'NtCreateSection': not a function</div><div>1>c:\src\lmdb\li= braries\liblmdb\mdb.c(4346): error C2063: 'NtMapViewOfSection': not= a function</div><div>1>c:\src\lmdb\libraries\liblmdb\mdb.c(4350): error= C2063: 'NtClose': not a function</div><div>1>c:\src\lmdb\librar= ies\liblmdb\mdb.c(4704): error C2065: 'NtCloseFunc': undeclared ide= ntifier</div><div>1>c:\src\lmdb\libraries\liblmdb\mdb.c(4704): error C20= 59: syntax error: ')'</div><div>1>c:\src\lmdb\libraries\liblmdb\= mdb.c(4707): error C2065: 'NtMapViewOfSectionFunc': undeclared iden= tifier</div><div>1>c:\src\lmdb\libraries\liblmdb\mdb.c(4707): error C205= 9: syntax error: ')'</div><div>1>c:\src\lmdb\libraries\liblmdb\m= db.c(4710): error C2065: 'NtCreateSectionFunc': undeclared identifi= er</div><div>1>c:\src\lmdb\libraries\liblmdb\mdb.c(4710): error C2059: s= yntax error: ')'</div><div>1>c:\src\lmdb\libraries\liblmdb\mdb.c= (8314): warning C4333: '>>': right shift by too large amount,= data los</div></div><div class=3D"gmail_extra"><br></div><div class=3D"gma= il_extra">PS github's bug tracker is much more user-friendly, it even h= as command line interface for old-school guys. Why do you use this mail-bas= ed crap?</div><div class=3D"gmail_extra"><br clear=3D"all"><div><div class= =3D"gmail_signature" data-smartmail=3D"gmail_signature"><div dir=3D"ltr">Be= st Regards,<div>Sergey</div></div></div></div><br></div></div> --000000000000ac244b056892dafb--

1 0

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

openldap-bugs