linbox-team / fflas-ffpack

FFLAS-FFPACK - Finite Field Linear Algebra Subroutines / Package
http://linbox-team.github.io/fflas-ffpack/
GNU Lesser General Public License v2.1
56 stars 23 forks source link

Tarball for release 2.4.3 does not include all commits of the corresponding tag #340

Closed jpoiret closed 3 years ago

jpoiret commented 3 years ago

The tarball leaves out commit f1f9ee78971b0e560a762a5e78d0dc8bc04fa682, which breaks compilation for macaulay2 with GCC 10.

jpoiret commented 3 years ago

After looking at this more closely, it seems that tag v2.4.3 was one year after tag 2.4.3, and that the release v2.4.3 on github links to a tarball of 2.4.3.

ClementPernet commented 3 years ago

This is very bad indeed.

As for f1f9ee7, breaking compilation of macaulay2 with gcc10, I guess it deserves its own issue.

All apologies, as I am very likely the main culprit for this mess.

ClementPernet commented 3 years ago

Ok, I just updated the 2.4.3 release to point to the tag 2.4.3 and no longer v2.4.3. After a second thought, I will not create a 2.4.4 at the position of v2.4.3 to avoid unnecessary complications, since we will release 2.5.0, very soon.

tornaria commented 3 years ago

@ClementPernet: Note also that the tar.bz2 are named fflas_ffpack instead of fflas-ffpack. If you will keep doing bz2, maybe you can fix that to be fflas-ffpack as well.

ClementPernet commented 3 years ago

Well, no, this bz2 archive is produced for integration into sagemath, which requires this naming convention.