-
Notifications
You must be signed in to change notification settings - Fork 1.7k
Compile error debian 10 #2364
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
See this issue. |
Hi @mhf-ir, You can either update the version of your libmaxmind or disable it with: --with-maxmind=no as ./configure option. That was an issue on MaxMind that wasn't treating MMDB_s as a const as it needs to be. |
Using compile latest version of https://github.com/maxmind/libmaxminddb in debian 10,
|
Just two quick idea: check the |
make clean
Problem not solve |
@mhf-ir, I had no idea, so I started a VM and tried to reproduce your issue (the details were fine). There were few interesting lines in your output (and of course, in mine too), eg.:
So I also run into same result as you. I think the problem is in Debian's Whit this workaround I could compile the library on the up-to-date Debian 10 system:
Could you try this? (Please note, don't forget to run |
Hi @mhf-ir, Given the output of the configure scripts, it is likely that you have both versions installed. In that scenario the build script will pick the distribution one. You may want to force configure to use the version that you have installed by using Make sure that the libraries are in your LD_LIB_PATH otherwise - during load time - the main process will pick the wrong library again. You can double-check by using |
/usr/include -I/usr/include/libxml2 -DWITH_LIBXML2 -g -O2 -MT utils/libmodsecurity_la-geo_lookup.lo -MD -MP -MF utils/.deps/libmodsecurity_la-geo_lookup.Tpo -c utils/geo_lookup.cc -fPIC -DPIC -o utils/.libs/libmodsecurity_la-geo_lookup.o Thank you very much, I did the following and everything worked in the file geo_lookup.cc line 134 r = MMDB_lookup_string(&mmdb, target.c_str(), &gai_error, &mmdb_error); replaced by r = MMDB_lookup_string(const_cast<MMDB_s*>(&mmdb), target.c_str(), &gai_error, &mmdb_error); and HURRAY!!!! Thanks a lot! |
Describe the bug
Compile using debian 10 docker image
Following packages are installed:
Logs and dumps
Expected behavior
Compile success in debian 10
The text was updated successfully, but these errors were encountered: