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

ENH: fix to use upstream --remove-rules #412

Merged
merged 3 commits into from Nov 6, 2013
Merged

ENH: fix to use upstream --remove-rules #412

merged 3 commits into from Nov 6, 2013

Conversation

grooverdan
Copy link
Contributor

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 8ac6081 on grooverdan:firewalld into 93de46a on fail2ban:master.

@yarikoptic
Copy link
Member

so this is with the hope of patched firewalld being soon released, right?
would it be worth providing older line also in a comment may be for older installations? or the firewalld is new enough?

@grooverdan
Copy link
Contributor Author

Looking at https://git.fedorahosted.org/cgit/firewalld.git/log/ it looks like is about on the two week mark when there should be a release.

Debian:

http://packages.debian.org/search?keywords=firewalld&searchon=names&suite=all&section=all

Fedora, The main to upstream devs are the package maintainers for Fedora

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 5eddd5d on grooverdan:firewalld into 93de46a on fail2ban:master.

@grooverdan
Copy link
Contributor Author

upstream and fedora have released 0.3.8 so require that as a version rather than ugly scripting hacks. http://pkgs.fedoraproject.org/cgit/firewalld.git/

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 87f68d7 on grooverdan:firewalld into 93de46a on fail2ban:master.

grooverdan added a commit that referenced this pull request Nov 6, 2013
ENH: enhance firewall-cmd to use firewall-0.8.3's --remove-rules
@grooverdan grooverdan merged commit ac1f45d into fail2ban:master Nov 6, 2013
@grooverdan grooverdan deleted the firewalld branch November 6, 2013 00:46
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.

None yet

3 participants