bio package star

bio package star

[Efraim Flashner]

[-- Attachment #1: Type: text/plain, Size: 501 bytes --]

http://hydra.gnu.org/build/1295673/log/raw Star currently fails to build
on i686. Looking at the logs, it freaks out at an unsigned long int,
__uint128_t, which as far as I know isn't supported on 32-bit systems,
so it looks to me like we should disable support for i686 and armhf.


-- 
Efraim Flashner   <efraim@flashner.co.il>   אפרים פלשנר
GPG key = A28B F40C 3E55 1372 662D  14F7 41AA E7DC CA3D 8351
Confidentiality cannot be guaranteed on emails sent or received unencrypted

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

Re: bio package star

[Ben Woodcroft]

On 07/07/16 15:52, Efraim Flashner wrote:
> http://hydra.gnu.org/build/1295673/log/raw Star currently fails to build
> on i686. Looking at the logs, it freaks out at an unsigned long int,
> __uint128_t, which as far as I know isn't supported on 32-bit systems,
> so it looks to me like we should disable support for i686 and armhf.

Agreed, it seems likely that this bionformatics tool never worked on 
these systems. I will disable builds for these platforms.

Thanks for the pick-up.
ben

Re: bio package star

[Ricardo Wurmus]

Ben Woodcroft <b.woodcroft@uq.edu.au> writes:

> On 07/07/16 15:52, Efraim Flashner wrote:
>> http://hydra.gnu.org/build/1295673/log/raw Star currently fails to build
>> on i686. Looking at the logs, it freaks out at an unsigned long int,
>> __uint128_t, which as far as I know isn't supported on 32-bit systems,
>> so it looks to me like we should disable support for i686 and armhf.
>
> Agreed, it seems likely that this bionformatics tool never worked on 
> these systems. I will disable builds for these platforms.

Thank you.  This seems to be the right approach for now.  Maybe we
should also file a bug report upstream?  At least we’ll know if they
would be willing to support systems other than x86_64.

~~ Ricardo

Re: bio package star

[Ben Woodcroft]

On 07/07/16 17:07, Ricardo Wurmus wrote:
> Ben Woodcroft <b.woodcroft@uq.edu.au> writes:
>
>> On 07/07/16 15:52, Efraim Flashner wrote:
>>> http://hydra.gnu.org/build/1295673/log/raw Star currently fails to build
>>> on i686. Looking at the logs, it freaks out at an unsigned long int,
>>> __uint128_t, which as far as I know isn't supported on 32-bit systems,
>>> so it looks to me like we should disable support for i686 and armhf.
>> Agreed, it seems likely that this bionformatics tool never worked on
>> these systems. I will disable builds for these platforms.
> Thank you.  This seems to be the right approach for now.  Maybe we
> should also file a bug report upstream?  At least we’ll know if they
> would be willing to support systems other than x86_64.

Pushed as 'c11f79a'.

I decided not to file a bug report since I noted that the README 
specifies that a 64 bit system is required.

Thanks,
ben

Re: bio package star

[Ricardo Wurmus]

Ben Woodcroft <b.woodcroft@uq.edu.au> writes:

> On 07/07/16 17:07, Ricardo Wurmus wrote:
>> Ben Woodcroft <b.woodcroft@uq.edu.au> writes:
>>
>>> On 07/07/16 15:52, Efraim Flashner wrote:
>>>> http://hydra.gnu.org/build/1295673/log/raw Star currently fails to build
>>>> on i686. Looking at the logs, it freaks out at an unsigned long int,
>>>> __uint128_t, which as far as I know isn't supported on 32-bit systems,
>>>> so it looks to me like we should disable support for i686 and armhf.
>>> Agreed, it seems likely that this bionformatics tool never worked on
>>> these systems. I will disable builds for these platforms.
>> Thank you.  This seems to be the right approach for now.  Maybe we
>> should also file a bug report upstream?  At least we’ll know if they
>> would be willing to support systems other than x86_64.
>
> Pushed as 'c11f79a'.

Thanks!

> I decided not to file a bug report since I noted that the README 
> specifies that a 64 bit system is required.

Seems reasonable to me.  It’s a little sad that in bioinformatics x86_64
is used almost exclusively and very few tools work with any other
architecture.

~~ Ricardo