UTF-8 support

[Ratatosk42]

Something don't work in UTF-8 :

git-game:40:in `split': invalid byte sequence in UTF-8 (ArgumentError)

[debugger22]

Got this in SymPy repo:

/usr/bin/git-game:40:in `split': invalid byte sequence in UTF-8 (ArgumentError)
    from /usr/bin/git-game:40:in `<main>'

[myoffe]

I'm getting this in my private repo:

/usr/local/bin/git-game:40:in `split': invalid byte sequence in US-ASCII (ArgumentError)
    from /usr/local/bin/git-game:40:in `<main>'

[mo]

I hacked around this using:

diff --git git-game git-game
index fcde219..be22b74 100755
--- git-game
+++ git-game
@@ -37,7 +37,9 @@ FIELD_DELIMITER = "|||"

commit_format = ["%an", "%ar", "%B"].join(FIELD_DELIMITER)

-raw_commits = git log --no-merges --pretty="#{COMMIT_DELIMITER}#{commit_format}" #{input if input}.split("#{COMMIT_DELIMITER}")
+raw_commits = git log --no-merges --pretty="#{COMMIT_DELIMITER}#{commit_format}" #{input if input}
+raw_commits = raw_commits.encode("utf-8", "binary", :undef => :replace)
+raw_commits = raw_commits.split("#{COMMIT_DELIMITER}")
commits = []
raw_commits.each do |c|
next if c.strip.empty?

...too ugly for PR though.

[Ratatosk42]

It's working for me !

[pesterhazy]

mo's patch works for me (although the backticks were stripped because of github's markdown)

[mreishus]

you can just chain it:

raw_commits = `git log --no-merges --pretty="#{COMMIT_DELIMITER}#{commit_format}" #{input if input}`.encode("utf-8", "binary", :undef => :replace).split("#{COMMIT_DELIMITER}")

[jsomers]

@gzugmeyer @pesterhazy @mo Want to give @mreishus's patch a try? If that works and if someone wants to submit a pull I will happily merge it

[Ratatosk42]

It's done : #28