Bug 560046

Summary:	net-ftp/vsftpd - sysdeputil.o: In function `vsf_sysdep_check_auth': sysdeputil.c:(.text+0xe8): undefined reference to `crypt'
Product:	Gentoo Linux	Reporter:	Ivan S. Titov <iohann.s.titov>
Component:	Current packages	Assignee:	No maintainer - Look at https://wiki.gentoo.org/wiki/Project:Proxy_Maintainers if you want to take care of it <maintainer-needed>
Status:	RESOLVED TEST-REQUEST
Severity:	normal	CC:	bugs, CasperVector, gentoo_bugs, hwoarang, idarktemplar, mail, net-ftp, reot+gentoo, sam, wired
Priority:	Normal	Keywords:	PATCH
Version:	unspecified
Hardware:	All
OS:	Linux
Whiteboard:
Package list:		Runtime testing required:	---
Attachments:	build.log emerge --info output emerge-info.txt emerge-history.txt environment net-ftp:vsftpd-3.0.3:20160807-082724.log Build log Output from emerge --info Proposed patch build.log emerge-info.txt

Description Ivan S. Titov 2015-09-09 14:26:00 UTC

net-ftp/vsftpd fails to build on armv6j and armv7a systems complaining on "undefined reference to `crypt'"
Affected version are 3.0.2-r2 to 3.0.2-r4 and 3.0.3.


Reproducible: Always

Comment 1 Ivan S. Titov 2015-09-09 18:30:34 UTC

Created attachment 411450 [details]
build.log

Comment 2 Ivan S. Titov 2015-09-09 18:32:18 UTC

Created attachment 411452 [details]
emerge --info output

Comment 3 Ivan S. Titov 2015-09-09 18:43:23 UTC

It comes from the solution of bug #508192:

# Override LIBS variable. Bug #508192
LIBS=
use caps && LIBS+=" -lcap"
use pam && LIBS+=" -lpam"
use tcpd && LIBS+=" -lwrap"
use ssl && LIBS+=" -lssl -lcrypto"

Comment 4 Martin Filo 2015-12-20 19:37:53 UTC

This work for me on amd64
--- /usr/portage/net-ftp/vsftpd/vsftpd-3.0.3.ebuild     2015-08-29 12:31:02.0000
00000 +0200
+++ vsftpd-3.0.3.ebuild 2015-12-20 20:31:12.518972984 +0100
@@ -67,6 +67,7 @@
        use pam && LIBS+=" -lpam"
        use tcpd && LIBS+=" -lwrap"
        use ssl && LIBS+=" -lssl -lcrypto"
+       LIBS+=" -lcrypt"
        CFLAGS="${CFLAGS}" \
        CC="$(tc-getCC)" \

Comment 5 Ivan S. Titov 2016-01-05 16:12:20 UTC

This works for me on both arm v6 and 7...

Comment 6 Toralf Förster gentoo-dev

2016-08-07 08:45:33 UTC

happened at tinderbox image amd64-13.0-systemd-unstable_20160731-161603 too

Comment 7 Toralf Förster gentoo-dev

2016-08-07 08:46:13 UTC

Created attachment 442706 [details]
emerge-info.txt

same at a tinderbox image

Comment 8 Toralf Förster gentoo-dev

2016-08-07 08:46:18 UTC

Created attachment 442708 [details]
emerge-history.txt

Comment 9 Toralf Förster gentoo-dev

2016-08-07 08:46:21 UTC

Created attachment 442710 [details]
environment

Comment 10 Toralf Förster gentoo-dev

2016-08-07 08:46:24 UTC

Created attachment 442712 [details]
net-ftp:vsftpd-3.0.3:20160807-082724.log

Comment 11 Rolf Eike Beer archtester

2018-06-27 20:54:06 UTC

Same with 3.0.3-r2 on sparc.

USE='-caps pam -ssl -tcpd -xinetd' succeeded for =net-ftp/vsftpd-3.0.3-r2
USE='caps pam -ssl -tcpd -xinetd' succeeded for =net-ftp/vsftpd-3.0.3-r2
USE='caps -pam ssl -tcpd -xinetd' failed for =net-ftp/vsftpd-3.0.3-r2
USE='caps pam -ssl tcpd -xinetd' succeeded for =net-ftp/vsftpd-3.0.3-r2
USE='-caps -pam ssl tcpd -xinetd' failed for =net-ftp/vsftpd-3.0.3-r2
USE='-caps pam ssl tcpd -xinetd' succeeded for =net-ftp/vsftpd-3.0.3-r2
USE='caps pam ssl tcpd -xinetd' succeeded for =net-ftp/vsftpd-3.0.3-r2
USE='-caps -pam -ssl -tcpd xinetd' failed for =net-ftp/vsftpd-3.0.3-r2
USE='-caps -pam ssl -tcpd xinetd' failed for =net-ftp/vsftpd-3.0.3-r2
USE='caps -pam ssl -tcpd xinetd' failed for =net-ftp/vsftpd-3.0.3-r2
USE='-caps pam -ssl tcpd xinetd' succeeded for =net-ftp/vsftpd-3.0.3-r2
USE='caps -pam ssl tcpd xinetd' failed for =net-ftp/vsftpd-3.0.3-r2

Comment 12 reot+gentoo 2018-10-09 10:19:04 UTC

This old bug is apparently still present, at least on amd64 building without caps, pam, and tcpd.  The patch proposed in Comment 3 works for me.  What is the procedure for somebody to commit this?

Comment 13 reot+gentoo 2018-10-09 10:20:03 UTC

Created attachment 550228 [details]
Build log

Comment 14 reot+gentoo 2018-10-09 10:20:25 UTC

Created attachment 550230 [details]
Output from emerge --info

Comment 15 reot+gentoo 2018-10-09 10:20:58 UTC

Created attachment 550232 [details, diff]
Proposed patch

Comment 16 reot+gentoo 2018-10-09 10:23:24 UTC

Was referring to the patch proposed in Comment 4 of course.  Proposed patch attached.

Comment 17 Jeroen Roovers (RETIRED) gentoo-dev

2019-01-28 15:26:35 UTC

*** Bug 676632 has been marked as a duplicate of this bug. ***

Comment 18 Davide Palma 2020-03-24 10:49:31 UTC

Bug is still there, patch in Comment 4 works

Comment 19 Sam James archtester

2021-06-08 09:16:47 UTC

(In reply to Davide Palma from comment #18)
> Bug is still there, patch in Comment 4 works

How about now with 3.0.4?

Comment 20 Ivan S. Titov 2021-06-08 17:00:27 UTC

Retested current net-ftp/vsftpd-3.0.3-r3 version.
Compiles on both arm and arm64.
Will make separate request to keyword it for arm64.

Comment 21 Martin Filo 2021-06-08 19:58:06 UTC

Still same problem undefined reference to 'crypt'.

Comment 22 Ivan S. Titov 2021-06-08 20:52:28 UTC

(In reply to Martin Filo from comment #21)
> Still same problem undefined reference to 'crypt'.

What's your gcc version? or just give emerge --info output, please.

Comment 23 Sam James archtester

2021-06-08 22:12:19 UTC

(In reply to Martin Filo from comment #21)
> Still same problem undefined reference to 'crypt'.

Full build.log and emerge —-info?

Comment 24 i.Dark_Templar 2021-06-09 05:59:41 UTC

Created attachment 714798 [details]
build.log

Still reproduces for me with net-ftp/vsftpd-3.0.4 and USE="-pam".

Command:

$ USE="-pam" PORTAGE_TMPDIR=/tmp/portage DISTDIR=/tmp/portage PORTAGE_USERNAME=$(id -u -n) PORTAGE_GRPNAME=$(id -g -n) ebuild /usr/portage/net-ftp/vsftpd/vsftpd-3.0.4.ebuild clean install

Comment 25 i.Dark_Templar 2021-06-09 06:00:38 UTC

Created attachment 714801 [details]
emerge-info.txt

Output of emerge --info.