From: Leo Prikler <leo.prikler@student.tugraz.at>
To: Ricardo Wurmus <rekado@elephly.net>
Cc: 39086@debbugs.gnu.org
Subject: [bug#39086] [PATCH 2/5] gnu: Add unicode-emoji.
Date: Thu, 16 Jan 2020 23:49:37 +0100 [thread overview]
Message-ID: <24b49d2a2cef963826d44db2229ed120f4a40362.camel@student.tugraz.at> (raw)
In-Reply-To: <87k15rxgcz.fsf@elephly.net>
Hi Ricardo,
Am Donnerstag, den 16.01.2020, 22:04 +0100 schrieb Ricardo Wurmus:
> Hi Leo,
>
> > * gnu/packages/ibus.scm (unicode-emoji): New package.
>
> Thank you for these patches!
>
> > +(define-public unicode-emoji
> > + (package
> > + (name "unicode-emoji")
> > + (version "12.0")
> > + (source #f)
>
> It’s pretty unusual to not provide a source. This will also make
> “guix
> build -S unicode-emoji” turn up nothing. It may be a good idea to
> elevate at least one of the inputs to “source” status.
>
> Ideally, all of the inputs would be considered sources like we’re
> doing
> it for texlive-* packages. It’s a pity that we can’t express
> multiple
> URLs for the url-fetch method.
I'm somewhat confused by this comment. While some of the texlive
packages do have actual sources, the big ones (texlive, texlive-union,
...) don't. I know that computed-origin is a thing that exists, but
this would really have been overkill for me.
> > + (build-system trivial-build-system)
> > + (arguments
> > + `(#:modules ((guix build utils))
> > + #:builder
> > + (let ((out (string-append %output "/share/unicode/emoji")))
> > + (use-modules (guix build utils))
> > + (mkdir-p out)
> > + (for-each
> > + (lambda (input)
> > + (copy-file
> > + (cdr input)
> > + (string-append out "/"
> > + (substring (car input) 8) ; strip
> > "unicode-"
> > + ".txt")))
>
> We don’t like to use CAR and CDR, because they feel so 80s… :) We
> prefer
> to use MATCH instead, because that feels more like the 90s. You
> could
> also use MATCH-LAMBDA.
Will do.
> Using a magical 8 also doesn’t feel right. You only need to strip
> the
> “unicode-” prefix because you chose to use it in the labels of these
> inputs. I don’t know… I feel that there’s a prettier way to express
> this, but perhaps there’s no *shorter* way.
The thing is, I kinda do need those prefixes, because otherwise those
files will be stored as gnu/store/<hash>-emoji-<whatever>.txt, which
feels wrong. AFAIK Guile has no string-remove-prefix.
> > + (inputs
> > + `(("unicode-emoji-data"
> > + ,(origin
> > + (method url-fetch)
> > + (uri "
> > https://www.unicode.org/Public/emoji/12.0/emoji-data.txt")
> > + (sha256
> > + (base32
> > "03sf7h1d6kb9m5s02lif88jsi5kjszpkfvcymaqxj8ds70ar9pgv"))))
> > + ("unicode-emoji-sequences"
> > + ,(origin
> > + (method url-fetch)
> > + (uri "
> > https://www.unicode.org/Public/emoji/12.0/emoji-sequences.txt")
> > + (sha256
> > + (base32
> > "1hghki2rn3n7m4lwpwi2a5wrsf2nij4bxga9ldabx4g0g2k23svs"))))
> > + ("unicode-emoji-test"
> > + ,(origin
> > + (method url-fetch)
> > + (uri "
> > https://www.unicode.org/Public/emoji/12.0/emoji-test.txt")
> > + (sha256
> > + (base32
> > "1dqd0fh999mh6naj816ni113m9dimfy3ih9nffjq2lrv9mmlgdck"))))
> > + ("unicode-emoji-variation-sequences"
> > + ,(origin
> > + (method url-fetch)
> > + (uri "
> > https://www.unicode.org/Public/emoji/12.0/emoji-variation-sequences.txt
> > ")
> > + (sha256
> > + (base32
> > "1cccwx5bl79w4c19vi5dhjqxrph92s8hihp9y8s2cqvdzmgbln7a"))))
> > + ("unicode-emoji-zwj-sequences"
> > + ,(origin
> > + (method url-fetch)
> > + (uri "
> > https://www.unicode.org/Public/emoji/12.0/emoji-zwj-sequences.txt")
> > + (sha256
> > + (base32
> > "1l791nbijmmhwa7kmvfn8gp26ban512l6mgqpz1mnbq3xm19181n"))))))
>
> Please use the version string from above in the URLs here. It’s just
> a
> little less to update when upgrading the package in the future.
Will do.
> > + (description
> > + "Data files for the Unicode Technological Standard #51
> > (Emoji).")
>
> Please use a complete sentence here.
I doubt the nature of this package will allow me to be more
descriptive, but I'll try.
Regards, Leo
next prev parent reply other threads:[~2020-01-16 22:50 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-01-11 11:58 [bug#39086] [PATCH] Enable Emoji support in IBus Leo Prikler
2020-01-11 12:08 ` [bug#39086] [PATCH 1/5] licenses: Add Unicode license Leo Prikler
2020-01-11 12:08 ` [bug#39086] [PATCH 2/5] gnu: Add unicode-emoji Leo Prikler
2020-01-16 21:04 ` Ricardo Wurmus
2020-01-16 22:49 ` Leo Prikler [this message]
2020-01-11 12:08 ` [bug#39086] [PATCH 3/5] gnu: Add unicode-cldr-common Leo Prikler
2020-01-16 21:08 ` Ricardo Wurmus
2020-01-16 22:53 ` Leo Prikler
2020-01-17 7:41 ` Ricardo Wurmus
2020-01-17 10:41 ` Leo Prikler
2020-02-03 9:49 ` Leo Prikler
2020-02-03 15:33 ` bug#39086: " Ricardo Wurmus
2020-02-03 15:46 ` [bug#39086] " Leo Prikler
2020-02-03 16:06 ` Ricardo Wurmus
2020-01-11 12:08 ` [bug#39086] [PATCH 4/5] gnu: ibus: Build with emoji support Leo Prikler
2020-01-16 21:10 ` Ricardo Wurmus
2020-01-11 12:08 ` [bug#39086] [PATCH 5/5] gnu: ibus: Disable parallel build Leo Prikler
2020-01-16 21:13 ` Ricardo Wurmus
2020-01-17 0:19 ` [bug#39086] [PATCH 2/5] gnu: Add unicode-emoji Leo Prikler
2020-01-17 0:19 ` [bug#39086] [PATCH 3/5] gnu: Add unicode-cldr-common Leo Prikler
2020-01-17 0:19 ` [bug#39086] [PATCH 4/5] gnu: ibus: Build with emoji support Leo Prikler
2020-01-17 0:19 ` [bug#39086] [PATCH 5/5] gnu: ibus: Disable parallel build Leo Prikler
2020-01-17 1:17 ` [bug#39086] [PATCH 2/5] gnu: Add unicode-emoji Leo Prikler
2020-01-17 10:37 ` Leo Prikler
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=24b49d2a2cef963826d44db2229ed120f4a40362.camel@student.tugraz.at \
--to=leo.prikler@student.tugraz.at \
--cc=39086@debbugs.gnu.org \
--cc=rekado@elephly.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/guix.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.