[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] gnu: femtolisp: Remove support for mips and armhf.
From: |
Ludovic Courtès |
Subject: |
Re: [PATCH] gnu: femtolisp: Remove support for mips and armhf. |
Date: |
Mon, 26 Sep 2016 12:38:37 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.1 (gnu/linux) |
ng0 <address@hidden> skribis:
> Ludovic Courtès <address@hidden> writes:
>
>> Andreas Enge <address@hidden> skribis:
>>
>>> femtolisp fails on mips and arm:
>>> http://hydra.gnu.org:3000/build/1472987/nixlog/2/tail-reload
>>> with the message
>>> In file included from llt/llt.h:6:0,
>>> from string.c:16:
>>> llt/utils.h:27:4: error: #error "unknown architecture"
>>> # error "unknown architecture"
>>>
>>> Should it be disabled there?
>>
>> Definitely. It would be worth checking in that file the set of
>> supported architectures, and putting them in ‘supported-platforms’.
>
> I'm not exactly sure what's supported, I found win32, osx, freebsd,
> openbsd, and that's it.
Sorry, I really meant supported architectures, which is what utils.h
seems to be about (although I don’t have the source here to check).
Could you check the #ifdefs in there?
> + ;; armhf and mips64el fail to build, it has been reported upstream:
> + ;; https://github.com/JeffBezanson/femtolisp/issues/25
> + (supported-systems
> + (and
> + (delete "armhf-linux" %supported-systems)
> + (delete "mips64el-linux" %supported-systems)
> + #t))
‘supported-systems’ must be a list of strings, so:
(fold delete %supported-systems
'("armhf-linux" "mips64el-linux"))
Thanks,
Ludo’.
- Re: [PATCH 1/2] gnu: Add femtolisp., (continued)
- 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., 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 <=
- 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
- Re: [PATCH 1/2] gnu: Add femtolisp., ng0, 2016/09/18
- Re: [PATCH 1/2] gnu: Add femtolisp., ng0, 2016/09/18