besser82 / libxcrypt

Extended crypt library for descrypt, md5crypt, bcrypt, and others
GNU Lesser General Public License v2.1
189 stars 52 forks source link

Fails to build with gcc 10 #103

Closed rfc1036 closed 4 years ago

rfc1036 commented 4 years ago

The package fails to build in a test rebuild on at least amd64 with gcc-10/g++-10, but succeeds to build with gcc-9/g++-9.

The full build log can be found at: http://people.debian.org/~doko/logs/gcc10-20200225/libxcrypt_4.4.10-10_unstable_gcc10.log

To build with GCC 10, either set CC=gcc-10 CXX=g++-10 explicitly, or install the gcc, g++, gfortran, ... packages from experimental.

apt-get -t=experimental install g++

Common build failures are new warnings resulting in build failures with -Werror turned on, or new/dropped symbols in Debian symbols files. For other C/C++ related build failures see the porting guide at http://gcc.gnu.org/gcc-10/porting_to.html

/usr/bin/ld: lib/.libs/libcrypt_la-crypt-des-obsolete.o:./build-deb1/../lib/crypt-common.h:27: multiple definition of `_crypt_ascii64'; lib/.libs/libcrypt_la-alg-des-tables.o:./build-deb1/../lib/crypt-common.h:27: first defined here
rfc1036 commented 4 years ago

This is Debian bug #957484.

besser82 commented 4 years ago

From the NEWS file:

Version 4.4.12
* Another fix for GCC v10.x, which occurs on s390 architectures only.

Version 4.4.11
* Fixes for GCC v10.x (issue #95).
* Change how the known-answer tests are parallelized.

If you really need to stick with v4.4.10, then you need to cherry-pick the following commits:

Anyways, I've seen v4.4.16 has already migrated to Debian testing, so this should not be an issue anymore?

rfc1036 commented 4 years ago

Indeed, sorry for the noise: I did not notice that an old version was rebuilt.

besser82 commented 4 years ago

No problem. =)

This issue can be closed then?