[Bug localedata/30649] New: [PATCH] Add transliteration of common emojis to smileys

[Bug localedata/30649] New: [PATCH] Add transliteration of common emojis to smileys

[colin at colino dot net]

https://sourceware.org/bugzilla/show_bug.cgi?id=30649

            Bug ID: 30649
           Summary: [PATCH] Add transliteration of common emojis to
                    smileys
           Product: glibc
           Version: unspecified
            Status: UNCONFIRMED
          Severity: normal
          Priority: P2
         Component: localedata
          Assignee: unassigned at sourceware dot org
          Reporter: colin at colino dot net
                CC: libc-locales at sourceware dot org
  Target Milestone: ---

Created attachment 14980
  --> https://sourceware.org/bugzilla/attachment.cgi?id=14980&action=edit
Patch against git

Hello,

This patch adds transliteration of common emojis (faces and hearts) to
old-school, ASCII-compliant smileys.

I hope you'll consider it for inclusion! 

Thanks,

Colin

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug localedata/30649] [PATCH] Add transliteration of common emojis to smileys

[colin at colino dot net]

https://sourceware.org/bugzilla/show_bug.cgi?id=30649

--- Comment #1 from Colin Leroy-Mira <colin at colino dot net> ---
Created attachment 14981
  --> https://sourceware.org/bugzilla/attachment.cgi?id=14981&action=edit
Example of transliteration

An example of the result

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug localedata/30649] [PATCH] Add transliteration of common emojis to smileys

[colin at colino dot net]

https://sourceware.org/bugzilla/show_bug.cgi?id=30649

Colin Leroy-Mira <colin at colino dot net> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Severity|normal                      |enhancement

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug localedata/30649] [PATCH] Add transliteration of common emojis to smileys

[colin at colino dot net]

https://sourceware.org/bugzilla/show_bug.cgi?id=30649

Colin Leroy-Mira <colin at colino dot net> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #14980|0                           |1
        is obsolete|                            |

--- Comment #2 from Colin Leroy-Mira <colin at colino dot net> ---
Created attachment 14982
  --> https://sourceware.org/bugzilla/attachment.cgi?id=14982&action=edit
Correct patch against git

Sorry, first patch was the wrong commit.

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug localedata/30649] [PATCH] Add transliteration of common emojis to smileys

[bruno at clisp dot org]

https://sourceware.org/bugzilla/show_bug.cgi?id=30649

Bruno Haible <bruno at clisp dot org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |bruno at clisp dot org

--- Comment #3 from Bruno Haible <bruno at clisp dot org> ---
Very nice!

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug localedata/30649] [PATCH] Add transliteration of common emojis to smileys

[carlos at redhat dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=30649

Carlos O'Donell <carlos at redhat dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Last reconfirmed|                            |2023-07-18
             Status|UNCONFIRMED                 |NEW
     Ever confirmed|0                           |1
                 CC|                            |carlos at redhat dot com

--- Comment #4 from Carlos O'Donell <carlos at redhat dot com> ---
(In reply to Colin Leroy-Mira from comment #2)
> Created attachment 14982 [details]
> Correct patch against git
> 
> Sorry, first patch was the wrong commit.

Thanks for all of the work on this.

Could you please post your patch to libc-alpha@sourceware.org for review?

The contribution checklist walks you through the process:
https://sourceware.org/glibc/wiki/Contribution%20checklist

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug localedata/30649] [PATCH] Add transliteration of common emojis to smileys

[colin at colino dot net]

https://sourceware.org/bugzilla/show_bug.cgi?id=30649

--- Comment #5 from Colin Leroy-Mira <colin at colino dot net> ---
Thank you for the pointer Carlos, I will do that.

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug localedata/30649] [PATCH] Add transliteration of common emojis to smileys

[sam at gentoo dot org]

https://sourceware.org/bugzilla/show_bug.cgi?id=30649

Sam James <sam at gentoo dot org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |sam at gentoo dot org

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug localedata/30649] [PATCH] Add transliteration of common emojis to smileys

[fweimer at redhat dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=30649

Florian Weimer <fweimer at redhat dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |fweimer at redhat dot com
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED
   Target Milestone|---                         |2.39

--- Comment #6 from Florian Weimer <fweimer at redhat dot com> ---
Fixed for 2.39 via:

commit dfe8c445883a50a55564b02b6957257bfc510db4
Author: Colin Leroy-Mira <colin@colino.net>
Date:   Tue Aug 29 08:02:50 2023 +0200

    localedata: Translit common emojis to smileys [BZ #30649]

    Add common emojis to the translit-able characters (mostly
    faces and hearts), and translit them to old-fashioned
    smileys.

    Signed-off-by: Colin Leroy-Mira <colin@colino.net>
    Reviewed-by: Florian Weimer <fweimer@redhat.com>

-- 
You are receiving this mail because:
You are on the CC list for the bug.