Skip to content
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

Incompatibility with ipset 6 #4

Open
MindHunter86 opened this issue Jun 21, 2016 · 4 comments
Open

Incompatibility with ipset 6 #4

MindHunter86 opened this issue Jun 21, 2016 · 4 comments

Comments

@MindHunter86
Copy link

Hi! We already have new version of IPSET from netfilter.

Can you upgrade this module? It's very useful fro nginx but is not compatible now =(

@MindHunter86
Copy link
Author

А если по-русски. Модуль ахерителен, но он сейчас вообще не совместим с новым модулем ipset.
Если ли какая-нить вероятность того, что модуль проагрейдят ?

@Vasfed
Copy link
Owner

Vasfed commented Jun 22, 2016

I'm not currently using this in production, so it will take some time to look into. Can you please provide more info on how it is incompatible?

Спасибо) Вероятность есть, но будет скорее всего небыстро. А какого вида там несовместимость - оно не компилится/падает?

@MindHunter86
Copy link
Author

Предлагаю по-русски только:
Да блин. У IPSET поменялась апиха. Версия там уже:

ipset --version
ipset v6.19, protocol version: 6

Следовательно все header файлы тоже.
Первое руганье на версию. Если вырубить в коде проверку, то после падает со сегфолтом.

А вообще модуль реально афигителен и очень юзабелен в проде. К сожалению, я так и не понял принцип программирования под него, да и вообще с C не дружу особо. Я все же админ)

@Vasfed Vasfed changed the title What about new version Incompatibility with ipset 6 Jun 23, 2016
@methuselah-0
Copy link

in the nginx source tree: nginx-1.12.1/objs/Makefile you can remove the -Werror flag as a workaround. It's only a warning and doesn't seem very important in this case. Then you can compile as normal.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants