Re: [PATCH] gnu: femtolisp: Remove support for mips and armhf.

[ludo@gnu.org (Ludovic Courtès)]

ng0 <ngillmann@runbox.com> skribis:

> Ludovic Courtès <ludo@gnu.org> writes:
>
>> Andreas Enge <andreas@enge.fr> 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’.