[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] Add Rage
From: |
Ricardo Wurmus |
Subject: |
Re: [PATCH] Add Rage |
Date: |
Wed, 05 Aug 2015 14:51:18 +0200 |
Hi Daniel,
as I wrote in a previous email the commit message should follow the
format shown in other messages. Take a look at the output of “git log”.
Your patch adds a lot more than just “rage”. Also the copyright line
seems to be wrong. Did you really intend to submit a patch with copies
of “efl”, “elementary”, “evas-generic-loaders”, and
“emotion-generic-players” in addition to “rage”? If not, please do not
include them in your commit.
Please submit one separate patch for every single package definition you
add.
I also think that “rage” better not be in its own module. How about
adding it to “enlightenment.scm” or “audio.scm” or similar?
Please also run “guix lint rage” to let Guix tell you a couple of things
that can be improved about your package.
~~ Ricardo
- [PATCH] Add Rage, Daniel Pimentel, 2015/08/05
- Re: [PATCH] Add Rage,
Ricardo Wurmus <=