[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: [help] porting brian on MIPS and POWER, or dropping support



Hi Michael,

Michael Crusoe, on 2021-01-30 16:51:27 +0100:
> On Sat, 30 Jan 2021 at 14:33, Étienne Mollier <etienne.mollier@mailoo.org>
> wrote:
> The package brian[1] has a mechanism based on the GSL which does
> > some sort of compilation just-in-time.  The default set of build
> > flags works rather well on all flavors of amd64 architecture
> > with or without extensions, however not all of the specified
> > options by default are supported on all architectures.
> >
> 
> Hmm.. this sounds like a recipe for a tough time :-D
> 
> Personally I put no extra effort for mips :-)

I thought so.  Surprizingly, the mipsel 32 bits variant turned
out to work normally, contrary to the 64 bits ISA (at least in
Qemu context).  I opened Bug#981397[2] to proceed to the removal
to buy us some time.

[2] https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=981397

> > [1] https://salsa.debian.org/med-team/brian/-/tree/master
> 
> https://salsa.debian.org/med-team/brian/-/blob/master/debian/patches/gsl-compiler-arg.patch#L29
> 
> -march=native is not allowed as it violates the architecture baseline,
> unless there is an alternative binary or library built without it.

I would generally agree, but in this case I understood the code
was compiled (in the background) on the end user's machine[3].
I don't exclude that I could have misunderstood the mechanism,
so am triple checking at the moment.  I probably should have put
the link in the patch header, by the way.

[3] https://brian2.readthedocs.io/en/stable/user/computation.html

In any way, I guess putting more regular compiler flags at first
should give a far safer outcome than current architecture
dependent selection.

Thanks for your review!  :)
-- 
Étienne Mollier <etienne.mollier@mailoo.org>
Fingerprint:  8f91 b227 c7d6 f2b1 948c  8236 793c f67e 8f0d 11da
Sent from /dev/pts/2, please excuse my verbosity.

Attachment: signature.asc
Description: PGP signature


Reply to: