emacs-compat / compat

COMPATibility Library for Emacs Lisp
https://elpa.gnu.org/packages/compat.html
GNU General Public License v3.0
69 stars 12 forks source link

LOW PRIORITY : Unbreak test for 32-bit architectures #16

Closed aagon closed 1 year ago

aagon commented 1 year ago

Hello Daniel,

This is a low priority fix, and needs not be tackled immediately. I'm just posting it now so that I don't forget about it later.

Debian's CI system caught a test failure on 32-bit architectures : the failing test expects the argument 100000000000 to be out of range. However, on 32-bit architectures, since 100000000000 is not a fixnum, but a bignum, the error produced is wrong-type-argument : in the primitive string-search, the correct type is tested before the correct range.

There are several ways to solve this :

I'm also forwarding you this fix since github's CI could not possibly have caught this failure, for it runs the tests on only one architecture, x86_64.

Best,

Aymeric