[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 1/2] gnu: Add femtolisp.
From: |
ng0 |
Subject: |
Re: [PATCH 1/2] gnu: Add femtolisp. |
Date: |
Wed, 14 Sep 2016 11:17:00 +0000 |
Ricardo Wurmus <address@hidden> writes:
> 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:
Ah, you are right.
> 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
>
>
I will send an updated patch in one or two hours, thanks.
--
ng0
- [PATCH 2/2] gnu: lispf4: Remove unused inputs key., (continued)
- [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, 2016/09/14
- Re: [PATCH 1/2] gnu: Add femtolisp.,
ng0 <=
- 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
- Re: [PATCH] gnu: femtolisp: Remove support for mips and armhf., Leo Famulari, 2016/09/27