[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] Update enlightenment.scm: Add Rage Player
From: |
Ricardo Wurmus |
Subject: |
Re: [PATCH] Update enlightenment.scm: Add Rage Player |
Date: |
Wed, 05 Aug 2015 16:11:07 +0200 |
Hi Daniel,
unfortunately, the commit message is still not correct. It’s all on one
line and “gnu: enlightenment:” should be “gnu:” instead. If you use
“git commit” without “-m” you can edit the message in your default
editor (configurable by setting the EDITOR variable).
> +(define-public rage
> + (package
> + (name "rage")
> + (version "0.1.4")
> + (source (origin
> + (method url-fetch)
> + (uri
> + (string-append
> + "http://download.enlightenment.org/rel/apps/rage/rage-"
> + version ".tar.gz"))
> + (sha256
> + (base32
> "10j3n8crk16jzqz2hn5djx6vms5f6x83qyiaphhqx94h9dgv2mgg"))))
> + (build-system gnu-build-system)
> + (native-inputs
> + `(("pkg-config" ,pkg-config)))
> + (inputs
> + `(("efl" ,efl)
> + ("elementary" ,elementary)))
> + (home-page "http://www.enlightenment.org")
The home page for “rage” actually seems to be
https://www.enlightenment.org/about-rage
> + (synopsis "Powerful video play based on EFL")
“video play” –> “video player”. Not sure if “Powerful” is a good word
for a synopsis. What makes it powerful? If this cannot be expressed
within the limited space of a synopsis it should be omitted.
> + (description
> + "Rage is a video and audio player written with Enlightenment Foundation
> Libraries
> +with some extra bells and whistles.")
This line looks a little too long. When in Emacs please just use M-q to
break the long line according to our limits.
~~ Ricardo
- [PATCH] Update enlightenment.scm: Add Rage Player, Daniel Pimentel, 2015/08/05
- Re: [PATCH] Update enlightenment.scm: Add Rage Player, Thompson, David, 2015/08/05
- Re: [PATCH] Update enlightenment.scm: Add Rage Player,
Ricardo Wurmus <=
- Re: [PATCH] Update enlightenment.scm: Add Rage Player, Daniel Pimentel, 2015/08/05
- Re: [PATCH] Update enlightenment.scm: Add Rage Player, Ricardo Wurmus, 2015/08/05
- Re: [PATCH] Update enlightenment.scm: Add Rage Player, Daniel Pimentel, 2015/08/05
- Re: [PATCH] Update enlightenment.scm: Add Rage Player, Daniel Pimentel, 2015/08/06
- Re: [PATCH] Update enlightenment.scm: Add Rage Player, Ricardo Wurmus, 2015/08/06
- Re: [PATCH] Update enlightenment.scm: Add Rage Player, Daniel Pimentel, 2015/08/06
- Re: [PATCH] Update enlightenment.scm: Add Rage Player, Ludovic Courtès, 2015/08/18
- Re: [PATCH] Update enlightenment.scm: Add Rage Player, Daniel Pimentel, 2015/08/18