[Bug 1514274] Review Request: twitter-twemoji-fonts - Twitter Emoji for everyone

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



https://bugzilla.redhat.com/show_bug.cgi?id=1514274



--- Comment #29 from Peter Oliver <mavit@xxxxxxxxxxxx> ---
(In reply to Neal Gompa from comment #28)
> @Peter, can we drop the whole Noto Emoji source requirement from this font
> package now? It looks like the only file you use from it is a single tmpl
> file.

This would be true if https://github.com/googlei18n/noto-emoji/issues/9 was
fixed, but I don’t think anyone is working on that.

For now, also required from Noto Emoji are the Makefile and many of the scripts
the Makefile calls.  I haven’t looked into it in detail myself, but Jeremy
Bicha of Ubuntu has recently done the equivalent integration work for the Emoji
Two font, and the patch touches 32 files.  It hasn’t yet been accepted by Emoji
Two, because of, as I understand it, its complexity
(https://github.com/EmojiTwo/emojitwo/pull/167).

> Can you instead submit a version of that to the Twemoji GitHub repository
> and use that locally in this package?

We could, yes, but I suspect that Twitter may be reluctant to accept it, since
it would require maintaining this fork of the build system long term.  It seems
like too much code to copy and paste from one project to another, to me.  Let
me know what you think.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
package-review mailing list -- package-review@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx




[Index of Archives]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite Conditions]     [KDE Users]

  Powered by Linux