Bug 211614

Summary: [PATCH] games/cowsay: cowsay doesn't handle UTF8 properly
Product: Ports & Packages Reporter: Aleksander Alekseev <afiskon>
Component: Individual Port(s)Assignee: Carlos J. Puga Medina <cpm>
Status: Closed FIXED    
Severity: Affects Many People CC: afiskon, amdmi3, cpm, junovitch, lifanov, w.schwarzenfeld
Priority: --- Keywords: patch, patch-ready
Version: LatestFlags: bugzilla: maintainer-feedback? (lifanov)
Hardware: Any   
OS: Any   
Attachments:
Description Flags
Fixed files/patch-cowsay
none
diff -u of old and new patch-cowsay
none
patch-cowsay lifanov: maintainer-approval+

Description Aleksander Alekseev 2016-08-05 22:37:20 UTC
Created attachment 173345 [details]
Fixed files/patch-cowsay

Fixed files/patch-cowsay is attached.

Before:

```
$ echo 'Привет!' | cowsay
 _______________ 
< Привет! >
 --------------- 
        \   ^__^
         \  (oo)\_______
            (__)\       )\/\
                ||----w |
                ||     ||
```

After:

```
$ echo 'Привет!' | cowsay
 _________ 
< Привет! >
 --------- 
        \   ^__^
         \  (oo)\_______
            (__)\       )\/\
                ||----w |
                ||     ||
```
Comment 1 Aleksander Alekseev 2016-08-05 22:39:46 UTC
Created attachment 173346 [details]
diff -u of old and new patch-cowsay
Comment 2 Carlos J. Puga Medina freebsd_committer 2016-08-06 03:11:20 UTC
Created attachment 173347 [details]
patch-cowsay

Bump PORTREVISION
Comment 3 Nikolai Lifanov 2016-08-06 12:33:44 UTC
Comment on attachment 173347 [details]
patch-cowsay

Approved! This works as you described. Thank you for the fix!
Comment 4 Walter Schwarzenfeld freebsd_triage 2016-08-09 09:29:39 UTC
It is not really important, but there are still problems with UTF-8 with the eye string and the tongue string.
Comment 5 Carlos J. Puga Medina freebsd_committer 2016-08-09 22:46:39 UTC
Waiting for mentor approval.
Comment 6 Jason Unovitch freebsd_committer 2016-08-10 02:26:12 UTC
Mentor approved.
Comment 7 commit-hook freebsd_committer 2016-08-10 10:11:57 UTC
A commit references this bug:

Author: cpm
Date: Wed Aug 10 10:11:50 UTC 2016
New revision: 420004
URL: https://svnweb.freebsd.org/changeset/ports/420004

Log:
  - Partially fix cowsay UTF-8 handling
  - Bump PORTREVISION

  PR:	211614
  Submitted by:	Aleksander Alekseev <afiskon@devzen.ru>
  Approved by:	Nikolai Lifanov (maintainer), junovitch (mentor)

Changes:
  head/games/cowsay/Makefile
  head/games/cowsay/files/patch-cowsay
Comment 8 Carlos J. Puga Medina freebsd_committer 2016-08-10 10:12:36 UTC
Committed!

Thanks