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

Update libfirewall.tcl #570

Merged
merged 1 commit into from Mar 4, 2019
Merged

Update libfirewall.tcl #570

merged 1 commit into from Mar 4, 2019

Conversation

libertyx82
Copy link
Contributor

Correting icmpv6 Type to solve

ip6tables v1.6.2: unknown option "--icmp-type" Try ip6tables -h' or 'ip6tables --help' for more information.`

Correting icmpv6 Type to solve

ip6tables v1.6.2: unknown option "--icmp-type" Try `ip6tables -h' or 'ip6tables --help' for more information.
@jens-maus
Copy link
Owner

Danke für den PR. Allerdings beinhaltet dieser unnötige newline/whitespace anpassungen. Bitte sowas vermeiden/unterlassen damit der Patch so gering wie möglich ausfällt und damit besser für eQ3 einsehbar ist. Daher bitte entsprechend noch einmal anpassen. Danke!

@libertyx82
Copy link
Contributor Author

Ist mir aufgefallen, darum hab ich alles nochmal gelöscht und nochmal nur die eine Zeile geändert aber jedesmal wenn ich auf PR senden Klicke werden die anderen Zeilen mit geändert.

@hoedlmoser
Copy link
Contributor

aufgrund des patch-names gehe ich davon aus, daß das im web-editor gemacht wurde. da kommt aber der warnhinweis "We’ve detected the file encoding as ISO-8859-1. When you commit changes we will transcode it to UTF-8." das erklärt dann imho auch die vielen änderungen, die immer bei umlauten auftreten. da wird man nur über einen echten git commit die änderungen eingeschmissen bekommen.

@jens-maus jens-maus added 🐛 bug-report Something isn't working 🌱 minor This is a issue/ticket which can be easily fixed labels Mar 2, 2019
@jens-maus jens-maus added this to the next release milestone Mar 2, 2019
jens-maus added a commit that referenced this pull request Mar 4, 2019
@jens-maus jens-maus mentioned this pull request Mar 4, 2019
@jens-maus jens-maus merged commit 55a90a0 into jens-maus:master Mar 4, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 bug-report Something isn't working 🌱 minor This is a issue/ticket which can be easily fixed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants