]> git.ipfire.org Git - thirdparty/iptables.git/commit
Revert "ebtables: use extrapositioned negation consistently"
authorPhil Sutter <phil@nwl.cc>
Tue, 5 Feb 2019 17:18:02 +0000 (18:18 +0100)
committerFlorian Westphal <fw@strlen.de>
Tue, 5 Feb 2019 17:19:59 +0000 (18:19 +0100)
commit35b22e82fa62e10950d8e0fa53a755d4abadf346
tree9eae6b69c634d607d71d9cf5d9caeb02797cf604
parent148131f20421046fea028e638581e938ec985783
Revert "ebtables: use extrapositioned negation consistently"

This reverts commit 5f508b76a0cebaf91965ffa678089222e2d47964.

While attempts at unifying syntax between arp-, eb- and iptables-nft
increase the opportunity for more code-sharing, they are problematic
when it comes to compatibility. Accepting the old syntax on input helps,
but due to the fact that neither arptables nor ebtables support --check
command we must expect for users to test existence of a rule by
comparing input with output. If that happens in a script, deviating from
the old syntax in output has a high chance of breaking it.

Therefore revert Florian's patch changing inversion character position
in output and review the old code for consistency - the only thing
changed on top of the actual revert is ebtables' own copy of
print_iface() to make it adhere to the intrapositioned negation scheme
used throughout ebtables.

Added extension tests by the reverted commit have been kept.

Signed-off-by: Phil Sutter <phil@nwl.cc>
Signed-off-by: Florian Westphal <fw@strlen.de>
17 files changed:
extensions/libebt_802_3.c
extensions/libebt_802_3.t
extensions/libebt_arp.c
extensions/libebt_arp.t
extensions/libebt_ip.c
extensions/libebt_ip.t
extensions/libebt_ip6.c
extensions/libebt_ip6.t
extensions/libebt_mark_m.c
extensions/libebt_mark_m.t
extensions/libebt_pkttype.c
extensions/libebt_pkttype.t
extensions/libebt_standard.t
extensions/libebt_stp.c
extensions/libebt_vlan.c
extensions/libebt_vlan.t
iptables/nft-bridge.c