https://bugzilla.wikimedia.org/show_bug.cgi?id=47633

       Web browser: ---
            Bug ID: 47633
           Summary: git review dies if non-utf8 text as commit subject
           Product: Wikimedia
           Version: unspecified
          Hardware: All
                OS: All
            Status: NEW
          Severity: major
          Priority: Unprioritized
         Component: Git/Gerrit
          Assignee: [email protected]
          Reporter: [email protected]
                CC: [email protected], [email protected],
                    [email protected], [email protected]
    Classification: Unclassified
   Mobile Platform: ---

twn:/resources/nike/translatewiki/puppet/modules/nginx/files/sites
(2013/puppet)$ git review
Traceback (most recent call last):
  File "/usr/local/bin/git-review", line 1168, in <module>
    main()
  File "/usr/local/bin/git-review", line 1123, in main
    assert_one_change(remote, branch, yes, have_hook)
  File "/usr/local/bin/git-review", line 545, in assert_one_change
    (status, output) = run_command_status(cmd)
  File "/usr/local/bin/git-review", line 122, in run_command_status
    out = out.decode('utf-8')
  File "/usr/lib/python2.6/encodings/utf_8.py", line 16, in decode
    return codecs.utf_8_decode(input, errors, True)
UnicodeDecodeError: 'utf8' codec can't decode bytes in position 209-212:
invalid data

In git log (truncated from both ends):
Author: Siebrand Mazeland <[email protected]>
Date:   Sun Apr 21 18:36:26 2013 +0200

    Fix encoding

    Change-Id: I7e59a42be59f5df6965b8e83ec2534f3736d7cb3

commit 270d0644c61fd4f56e0acbd2442400227972647c
Author: Siebrand Mazeland <[email protected]>
Date:   Sun Apr 21 18:27:03 2013 +0200

    Fix paths for Vicuna Uploader

    Change-Id: Ib523d6aa368801ca7049d0b01fb8c2fa769fd664

commit a08134502d182caec86aeaf6585c5b67d10ce357
Author: Siebrand Mazeland <[email protected]>
Date:   Sun Apr 21 18:04:57 2013 +0200

    Updates for supporting Vicu<F1>aUploader

    Change-Id: Ibf28241e31d6803f14c2dc1f581ebd46f064677e



Then I remember that we still and again have the bug where I have to write git
fetch origin or git review will try submit unrelated commits to gerrit.

Ideally someone should work with upstream to fix both of these issues asap.

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
_______________________________________________
Wikibugs-l mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l

Reply via email to