From: "Tomáš Čech" <sleep_walker@suse.cz>
To: guix-devel@gnu.org
Subject: Re: [PATCH 4/5] gnu: Add emotion-generic-players
Date: Thu, 26 Feb 2015 19:09:08 +0100 [thread overview]
Message-ID: <20150226180908.GC10407@venom> (raw)
In-Reply-To: <874mq8hmav.fsf@gmail.com>
[-- Attachment #1: Type: text/plain, Size: 2451 bytes --]
On Thu, Feb 26, 2015 at 08:47:20PM +0800, 宋文武 wrote:
>Tomáš Čech <sleep_walker@suse.cz> writes:
>
>> Hello,
>>
>> thanks for review.
>>
>> On Thu, Feb 26, 2015 at 11:23:27AM +0100, Andreas Enge wrote:
>>>Hello,
>>>
>>>just a quick comment on the descriptions. It would be good to write full
>>>sentences, for instance:
>>>
>>>On Thu, Feb 26, 2015 at 01:31:12AM +0100, Tomáš Čech wrote:
>>>> + "Set of generic media players allowing to open video files through emotion. Useful only for emotion library based applications.")
>>>
>>>"Emotion Generic Players provides/consists of/is a set of generic ...
>>>It is useful only for ...".
>>
>> I agree that sentences may be better, I only suck at it. I'll try
>> to improvide it or I can accept any recommendations in the meantime.
>>
>>>I think it would also be good to expand a bit where possible, and to introduce
>>>the word "enlightenment" in each of these packages (by replacing "EFL" with
>>>"Enlightenment Foundation Libraries",
>>
>> I do agree that use of "Enlightenment Foundation Libraries" instead of
>> "EFL" could make it more enjoyable to read, but we could expand also
>> Gimp toolkit, Tool Command Language, ...
>>
>> Those who would need such these packages will know and it won't help
>> to others anyway.
>>
>> Synopsis of efl package will show you "Enlightenment Foundation Libraries".
>>
>>>for instance, or adding a sentence like
>>>"It is part of the Enlightenment desktop environment").
>>
>> Enlightenment is using the same libraries (they were developed for
>> Enlightenment) but the libraries are much more generic. It's like
>> relationship between Qt and KDE, only with reversed history order.
>>
>>>Maybe call the file enlightenment.scm?
>>
>> I believe efl.scm is more accurate as the packages there would be EFL
>> based but not necessarily Enlightenment related.
>Sorry, I didn't notice this line. (for my comment about terminology).
>
>Yes, this make sense. But I think it's more clear to make packages
>from enlightenment.org go into enlightenment.scm, while packages not
>from enlightenment.org but based on EFL into seperated files.
I'm afraid it is still quite optimistic POV, there is not that big
number of EFL based applications worth of packaging.
http://edevel.tumblr.com/post/98156942204
But it seems that the filename is really important to you all so I'll
change the name.
S_W
[-- Attachment #2: Type: application/pgp-signature, Size: 181 bytes --]
next prev parent reply other threads:[~2015-02-26 18:09 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-02-26 0:31 [PATCH 1/5] gnu: Add efl Tomáš Čech
2015-02-26 0:31 ` [PATCH 2/5] gnu: Add elementary Tomáš Čech
2015-02-27 17:01 ` Ludovic Courtès
2015-02-26 0:31 ` [PATCH 3/5] gnu: Add evas-generic-loaders Tomáš Čech
2015-02-27 17:02 ` Ludovic Courtès
2015-02-26 0:31 ` [PATCH 4/5] gnu: Add emotion-generic-players Tomáš Čech
2015-02-26 10:23 ` Andreas Enge
2015-02-26 11:14 ` Tomáš Čech
2015-02-26 12:03 ` Andreas Enge
2015-02-26 12:47 ` 宋文武
2015-02-26 18:09 ` Tomáš Čech [this message]
2015-02-26 18:16 ` Andreas Enge
2015-02-27 16:58 ` Ludovic Courtès
2015-02-26 0:31 ` [PATCH 5/5] gnu: Add terminology Tomáš Čech
2015-02-26 12:30 ` 宋文武
2015-02-27 17:04 ` Ludovic Courtès
2015-02-26 20:58 ` [PATCH 1/5] gnu: Add efl Andreas Enge
2015-02-26 21:05 ` Andreas Enge
2015-02-27 16:56 ` Ludovic Courtès
-- strict thread matches above, loose matches on Subject: below --
2015-02-28 22:11 EFL, elementary, loaders, players and terminology - round 2 Tomáš Čech
2015-02-28 22:11 ` [PATCH 4/5] gnu: Add emotion-generic-players Tomáš Čech
2015-03-01 20:35 ` Ludovic Courtès
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
List information: https://guix.gnu.org/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20150226180908.GC10407@venom \
--to=sleep_walker@suse.cz \
--cc=guix-devel@gnu.org \
/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 public inbox
https://git.savannah.gnu.org/cgit/guix.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).