New issue
Advanced search Search tips

Issue 807678 link

Starred by 1 user

Issue metadata

Status: Duplicate
Merged: issue 827502
Owner:
Closed: Apr 2018
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

[WPT Import] Notifier cannot handle commit subjects with non-ASCII characters

Project Member Reported by robertma@chromium.org, Jan 31 2018

Issue description

e.g. 
2c0c21f87e15333a54a9ef94d07ccee44d303f0e More red backgrounds, as suggested bt Gérard Talbot. Typo in rel=author link.

Traceback (most recent call last):
  File "/mnt/data/b/rr/tmpl5yITh/w/src/third_party/WebKit/Tools/Scripts/wpt-import", line 24, in <module>
    main()
  File "/mnt/data/b/rr/tmpl5yITh/w/src/third_party/WebKit/Tools/Scripts/wpt-import", line 17, in main
    host.exit(importer.main())
  File "/mnt/data/b/rr/tmpl5yITh/w/src/third_party/WebKit/Tools/Scripts/webkitpy/w3c/test_importer.py", line 169, in main
    if not self.send_notifications(local_wpt, options.auto_file_bugs, options.monorail_auth_json):
  File "/mnt/data/b/rr/tmpl5yITh/w/src/third_party/WebKit/Tools/Scripts/webkitpy/w3c/test_importer.py", line 633, in send_notifications
    dry_run=not auto_file_bugs, service_account_key_json=monorail_auth_json)
  File "/mnt/data/b/rr/tmpl5yITh/w/src/third_party/WebKit/Tools/Scripts/webkitpy/w3c/import_notifier.py", line 71, in main
    bugs = self.create_bugs_from_new_failures(wpt_revision_start, wpt_revision_end, gerrit_url)
  File "/mnt/data/b/rr/tmpl5yITh/w/src/third_party/WebKit/Tools/Scripts/webkitpy/w3c/import_notifier.py", line 187, in create_bugs_from_new_failures
    commit_list = self.format_commit_list(imported_commits, full_directory)
  File "/mnt/data/b/rr/tmpl5yITh/w/src/third_party/WebKit/Tools/Scripts/webkitpy/w3c/import_notifier.py", line 218, in format_commit_list
    line = '{}: {}'.format(subject, GITHUB_COMMIT_PREFIX + sha)
UnicodeEncodeError: 'ascii' codec can't encode character u'\u2026' in position 69: ordinal not in range(128)
step returned non-zero exit code: 1
 

Comment 1 by ajuma@chromium.org, Apr 3 2018

Just checking for an update, since this is marked P2.
Mergedinto: 827502
Status: Duplicate (was: Assigned)

Sign in to add a comment