ctdb-scripts: Fix regression when cleaning up routing table IDs
authorMartin Schwenke <martin@meltin.net>
Sun, 15 Jan 2017 20:24:15 +0000 (07:24 +1100)
committerAmitay Isaacs <amitay@samba.org>
Tue, 17 Jan 2017 01:20:16 +0000 (02:20 +0100)
commitf9368f8e129cb32ee30cb6501a6fe728db37e1d5
tree238347eb23e7904d1b884bfad3d577ce10a0bc5c
parent5e00a6b346325f52e35b9785eaffd72239aebcf5
ctdb-scripts: Fix regression when cleaning up routing table IDs

Commit 0ca00267cd2620a14968961738bcd2a69b597e95 removed explicit
continuations in strings for awk programs.  In one case this causes a
disconnect between condition and action, where an implicit
continuation does not work.  This results in duplicate lines in the
rt_tables file.

Move the opening brace for the action to make the implicit
continuation work as expected.

An alternative would be to revert the removal of the explicit
continuations and add shellcheck tags.  However, that doesn't mean
that an author of future code will necessarily use explicit
continuations, so the same mistake might still be make in the future.

BUG: https://bugzilla.samba.org/show_bug.cgi?id=12516

Reported-by: Barry Evans <bevans@pixitmedia.com>
Signed-off-by: Martin Schwenke <martin@meltin.net>
Reviewed-by: Amitay Isaacs <amitay@gmail.com>
ctdb/config/events.d/13.per_ip_routing