Re: [Buildroot] [PATCH 1/1] package/armadillo: bump to version 10.1.0

[Thomas Petazzoni via buildroot <buildroot@buildroot.org>]

Hello Dario,

On Sat, 24 Aug 2024 17:15:11 +0200
Dario Binacchi <dario.binacchi@amarulasolutions.com> wrote:

> - C++11 is now the minimum required C++ standard
> - faster handling of compound expressions by trimatu() and trimatl()
> - faster sparse matrix addition, subtraction and element-wise multiplication
> - expanded sparse submatrix views to handle the non-contiguous form of X.cols(vector_of_column_indices)
> - expanded eigs_sym() and eigs_gen() with optional fine-grained parameters
> 
> Fixes:
> - http://autobuild.buildroot.org/results/d4e9e7b453960a4f62a199344b30b729a4f235bc
> 
> Signed-off-by: Dario Binacchi <dario.binacchi@amarulasolutions.com>
> ---
>  package/armadillo/armadillo.hash | 2 +-
>  package/armadillo/armadillo.mk   | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)

Same comment as for babeld: bumping to solve a download issue is not a
good solution. Indeed, if we have to bump because upstream dropped the
tarball, what tells us that they won't drop the new tarball again in
the future? This can only be resolved either by finding an alternate
upstream location or by talking with upstream so that they restore the
previous tarball and understand the issues caused by moving/removing
previous tarballs.

Also, if they made C++11 required, then I'm pretty sure we will need a
BR2_TOOLCHAIN_GCC_AT_LEAST_xyz dependency added, no?

Thanks!

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot