José Guilherme Vanz created MESOS-3268: ------------------------------------------
Summary: Non ASCII char crashes apply-review.sh Key: MESOS-3268 URL: https://issues.apache.org/jira/browse/MESOS-3268 Project: Mesos Issue Type: Bug Components: reviewbot Reporter: José Guilherme Vanz Priority: Minor There is an issue in the apply-review script when user name field has some non ascii char. E.g.: Bad patch! Reviews applied: [37468] Failed command: ./support/apply-review.sh -n -r 37468 Error: 2015-08-14 04:22:30 URL:https://reviews.apache.org/r/37468/diff/raw/ [23334/23334] -> "37468.patch" [1] Traceback (most recent call last): File "./support/jsonurl.py", line 25, in <module> print data UnicodeEncodeError: 'ascii' codec can't encode character u'\xe9' in position 3: ordinal not in range(128) Successfully applied: Removed allocation types to mesos::master namespace The allocation-related types was moved to mesos::master namespace. MESOS-2516 Review: https://reviews.apache.org/r/37468 fatal: empty ident name (for <guilherme....@gmail.com>) not allowed Failed to commit patch In this example, the problem was caused because the full name of the user has the "é" char. For simulate the problem you can run the following shell script: `AUTHOR_NAME=$(./support/jsonurl.py https://reviews.apache.org/api/users/jvanz/ user fullname)` (This is my user, I removed the non ascii char to send more patches) The problem is when the result of the python script is kept in the variable. If you call the python script without store the result in a variable everything works fine. -- This message was sent by Atlassian JIRA (v6.3.4#6332)