[all-commits] [llvm/llvm-project] bc1773: [yamls2obj][test] - Make the Archives/regular.yaml...

Georgii Rymar via All-commits all-commits at lists.llvm.org
Wed Oct 28 07:05:11 PDT 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: bc1773f5b5e90e5fab3202768e516e145881666f
      https://github.com/llvm/llvm-project/commit/bc1773f5b5e90e5fab3202768e516e145881666f
  Author: Georgii Rymar <grimar at accesssoftek.com>
  Date:   2020-10-28 (Wed, 28 Oct 2020)

  Changed paths:
    M llvm/test/tools/yaml2obj/Archives/regular.yaml

  Log Message:
  -----------
  [yamls2obj][test] - Make the Archives/regular.yaml test stricter.

In rG6d656c9691d4 I had to relax the check from

`CONTENT: 21 3c 61 72 63 68 3e 0a 12{{$}}`

to

`CONTENT: 21 3c 61 72 63 68 3e 0a 12`

to fix the FreeBSD bot quickly: http://lab.llvm.org:8011/#/builders/28/builds/547

It turns out that "od" prints a trailing white space on FreeBSD, that is
why EOL mark ({{$}}) can't be used. But we still want to check the output size.

This patch adds a check of output size with "wc -c", similar to how it is done
below in the same test. This restores the original strictness.




More information about the All-commits mailing list