You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@mesos.apache.org by "Adam B (JIRA)" <ji...@apache.org> on 2015/10/07 19:02:26 UTC
[jira] [Commented] (MESOS-3268) apply-review.sh crashes with non
ASCII char
[ https://issues.apache.org/jira/browse/MESOS-3268?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14947217#comment-14947217 ]
Adam B commented on MESOS-3268:
-------------------------------
https://reviews.apache.org/r/39087/
> apply-review.sh crashes with non ASCII char
> -------------------------------------------
>
> 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 <gu...@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)