[ovs-git] [openvswitch/ovs] a9b874: tests: Fix sed usage in pmd test.

GitHub noreply at github.com
Tue Apr 3 18:13:23 UTC 2018


  Branch: refs/heads/branch-2.9
  Home:   https://github.com/openvswitch/ovs
  Commit: a9b87459b20fffdf1202d1b1e4500aa47c78144f
      https://github.com/openvswitch/ovs/commit/a9b87459b20fffdf1202d1b1e4500aa47c78144f
  Author: Ben Pfaff <blp at ovn.org>
  Date:   2018-04-03 (Tue, 03 Apr 2018)

  Changed paths:
    M tests/pmd.at

  Log Message:
  -----------
  tests: Fix sed usage in pmd test.

SUSv7 2016 Edition says:

  [2addr] {editing command
  editing command
  ...
  }

    Execute a list of sed editing commands only when the pattern space is
    selected. The list of sed editing commands shall be surrounded by
    braces. The braces can be preceded or followed by <blank> characters.
    The <right-brace> shall be preceded by a <newline> or <semicolon>
    (before any optional <blank> characters preceding the <right-brace>).

This usage in pmd.at omitted the semicolon before the right brace.  This
commit fixes the problem, which was rejected by the sed utility on Alpine
Linux (which presumably comes from some version of busybox, but BusyBox
v1.22.1 (Debian 1:1.22.0-9+b1) on my system accepts the form without
semicolon).

Reported-by: Stuart Cardall <developer at it-offshore.co.uk>
Reported-at: https://mail.openvswitch.org/pipermail/ovs-discuss/2018-March/046460.html
Signed-off-by: Ben Pfaff <blp at ovn.org>
Tested-by: Stuart Cardall <developer at it-offshore.co.uk>




More information about the git mailing list