Pull request on

27 views
Skip to first unread message

Jeehoon Kang

unread,
May 13, 2019, 3:05:01 AM5/13/19
to Hafnium
(I don't know where to send a pull request, so I'm sending one here.  If there is a proper way to send a pull request, please kindly inform me.)

`./kokoro/ubuntu/test.sh` requires python 2, but a simple tweak makes it work also for python 3:

```
diff --git a/test/hftest/hftest.py b/test/hftest/hftest.py
index 7834c4d..bf66bb2 100755
--- a/test/hftest/hftest.py
+++ b/test/hftest/hftest.py
@@ -258,7 +258,7 @@ def Main():
             suite_xml.set("failures", str(failures_from_suite))
     suites_xml.set("tests", str(tests_run))
     suites_xml.set("failures", str(failures))
-    with open(os.path.join(log, "sponge_log.xml"), "w") as f:
+    with open(os.path.join(log, "sponge_log.xml"), "wb") as f:
         ET.ElementTree(suites_xml).write(
             f, encoding='utf-8', xml_declaration=True)
     # If none were run, this is probably a mistake.
```

diff

Andrew Scull

unread,
May 13, 2019, 5:26:32 AM5/13/19
to Jeehoon Kang, Hafnium
Hi Jeehoon, thanks for the suggestion. We use Gerrit for code review. You should be able to push your CL for review with the following command:

    git push origin HEAD:refs/for/master

You will need to have signed the Google CLA before being able to upload CLs.

--
You received this message because you are subscribed to the Google Groups "Hafnium" group.
To unsubscribe from this group and stop receiving emails from it, send an email to hafnium-discu...@googlegroups.com.
To post to this group, send email to hafnium...@googlegroups.com.
To view this discussion on the web, visit https://groups.google.com/d/msgid/hafnium-discuss/2aa47d3a-588c-4238-bd68-83e1ae48ef21%40googlegroups.com.
For more options, visit https://groups.google.com/d/optout.
Reply all
Reply to author
Forward
0 new messages