[PATCH] ctdb-scripts: Fix regression when cleaning up routing table IDs (bug 12516)

Martin Schwenke martin at meltin.net
Tue Jan 17 01:01:07 UTC 2017


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

Plus an extra commit containing a test...

Please review and maybe push...

peace & happiness,
martin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: ctdb.patch
Type: text/x-patch
Size: 3159 bytes
Desc: not available
URL: <http://lists.samba.org/pipermail/samba-technical/attachments/20170117/17aeb7a9/ctdb.bin>


More information about the samba-technical mailing list