[PATCH] D68224: [LNT] Python 3 support: fix text/binary confusion in importreport
Thomas Preud'homme via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sun Oct 6 11:32:36 PDT 2019
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>
More information about the llvm-commits
mailing list