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

Rename ipset → libcorkipset #49

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Rename ipset → libcorkipset #49

wants to merge 1 commit into from

Conversation

dcreager
Copy link
Owner

ipset is pretty generic, and already taken. Arch and Debian both renamed this to libcorkipset, so that's what we'll do too!

I've cleaned up the header files a bit; the main one is now installed as <libcork/ipset.h> instead of <libcorkipset/ipset.h>, which is what's used in the current Arch and Debian packages. I've kept a copy available at the old location, with a deprecation warning telling you about the new location.

`ipset` is pretty generic, and already taken.  Arch and Debian both
renamed this to libcorkipset, so that's what we'll do too!

I've cleaned up the header files a bit; the main one is now installed as
<libcork/ipset.h> instead of <libcorkipset/ipset.h>, which is what's
used in the current Arch and Debian packages.  I've kept a copy
available at the old location, with a deprecation warning telling you
about the new location.
@dcreager
Copy link
Owner Author

@rogers0, please let me know if I missed anything here that was covered by your Debian patches.

@rogers0
Copy link
Contributor

rogers0 commented Aug 24, 2021

@dcreager
Sorry for the late reply.
Yes, this patch is perfect. Please kindly help to merge to mainline and make a new release.
Thank you!

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

Successfully merging this pull request may close these issues.

2 participants