[PATCH] D68224: [LNT] Python 3 support: fix text/binary confusion in importreport

[Thomas Preud'homme via Phabricator via llvm-commits]

This revision was automatically updated to reflect the committed changes.
Closed by commit rL373861: [LNT] Python 3 support: fix text/binary confusion in importreport (authored by thopre, committed by ).
Herald added a project: LLVM.

Changed prior to commit:
  https://reviews.llvm.org/D68224?vs=222420&id=223433#toc

Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D68224/new/

https://reviews.llvm.org/D68224

Files:
  lnt/trunk/lnt/lnttool/import_report.py


Index: lnt/trunk/lnt/lnttool/import_report.py
===================================================================
--- lnt/trunk/lnt/lnttool/import_report.py
+++ lnt/trunk/lnt/lnttool/import_report.py
@@ -3,8 +3,8 @@
 
 @click.command("importreport", short_help="import simple space separated "
                "data into a report to submit.")
- at click.argument("input", type=click.File('rb'), default="-", required=False)
- at click.argument("output", type=click.File('wb'), default="report.json",
+ at click.argument("input", type=click.File('r'), default="-", required=False)
+ at click.argument("output", type=click.File('w'), default="report.json",
                 required=False)
 @click.option("--testsuite", "suite", default="nts", show_default=True,
               required=True, help="short name of the test suite to submit to")


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D68224.223433.patch
Type: text/x-patch
Size: 829 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20191006/697e30bc/attachment.bin>