[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 1/2] gnu: Add femtolisp.
From: |
Ricardo Wurmus |
Subject: |
Re: [PATCH 1/2] gnu: Add femtolisp. |
Date: |
Wed, 14 Sep 2016 13:05:17 +0200 |
User-agent: |
mu4e 0.9.16; emacs 25.1.1 |
ng0 <address@hidden> writes:
> Ricardo Wurmus <address@hidden> writes:
>> In this case we should probably split these changes up as they are not
>> related to one another. Alternatively, the summary line should be
>> changed to apply to all of the changes in this patch.
>
> Can you give me an example how the summary line should look like then?
I just checked the git log but couldn’t find a similar package in which
changes of this magnitude were summarised in one commit.
I suggest splitting this up. Actually, though, I think the description
doesn’t need changing. I only suggested removing the sentence because I
assumed that the package didn’t yet exist. There’s no harm in keeping
it.
Without the description change it’s really just one thing that has been
changed:
gnu: femtolisp: Remove custom "patch-makefile" phase.
* gnu/packages/lisp.scm (femtolisp)[arguments]: Remove
"patch-makefile" phase; specify test target; build "release"
target instead of "default".
What do you think?
~~ Ricardo
- [PATCH 1/2] gnu: Add femtolisp., (continued)
- [PATCH 1/2] gnu: Add femtolisp., ng0, 2016/09/13
- [PATCH 2/2] gnu: lispf4: Remove unused inputs key., ng0, 2016/09/13
- Re: [PATCH 1/2] gnu: Add femtolisp., Ricardo Wurmus, 2016/09/13
- Re: [PATCH 1/2] gnu: Add femtolisp., ng0, 2016/09/13
- Re: [PATCH 1/2] gnu: Add femtolisp., ng0, 2016/09/13
- Re: [PATCH 1/2] gnu: Add femtolisp., Ricardo Wurmus, 2016/09/14
- Re: [PATCH 1/2] gnu: Add femtolisp., ng0, 2016/09/14
- Re: [PATCH 1/2] gnu: Add femtolisp., ng0, 2016/09/14
- Re: [PATCH 1/2] gnu: Add femtolisp., Ricardo Wurmus, 2016/09/14
- Re: [PATCH 1/2] gnu: Add femtolisp., ng0, 2016/09/14
- Re: [PATCH 1/2] gnu: Add femtolisp.,
Ricardo Wurmus <=
- Re: [PATCH 1/2] gnu: Add femtolisp., ng0, 2016/09/14
- Re: [PATCH 1/2] gnu: Add femtolisp., ng0, 2016/09/14
- Re: [PATCH 1/2] gnu: Add femtolisp., Andreas Enge, 2016/09/18
- Re: [PATCH 1/2] gnu: Add femtolisp., Ludovic Courtès, 2016/09/20
- Re: [PATCH 1/2] gnu: Add femtolisp., ng0, 2016/09/21
- Re: [PATCH 1/2] gnu: Add femtolisp., Ludovic Courtès, 2016/09/21
- Re: [PATCH 1/2] gnu: Add femtolisp., Ricardo Wurmus, 2016/09/21
- [PATCH] gnu: femtolisp: Remove support for mips and armhf., ng0, 2016/09/22
- Re: [PATCH] gnu: femtolisp: Remove support for mips and armhf., Ludovic Courtès, 2016/09/26
- Re: [PATCH] gnu: femtolisp: Remove support for mips and armhf., ng0, 2016/09/27