On Wed, Jun 12, 2013 at 02:52:08PM +0300, Timo Aaltonen wrote:
On 12.06.2013 14:39, Lukas Slebodnik wrote:
> ehlo,
>
> It seems that some linkers have problem with wrong order of libraries.
> This commit only change order.
>
> Attached patch should fix next issue:
> Library libsss_crypt.so is used in linking process, but there are
> undefined references.
>
> It seems that I changed order of libraries after rewriting link_all_deplibs
> patch to use automake variable SSSD_INTERNAL_LTLIBS. I only tested
> new patch with patched version of libtool on fedora.
>
> libtool: link: gcc -Wall -Wshadow -Wstrict-prototypes -Wpointer-arith
-Wcast-qual -Wcast-align -Wwrite-strings -Werror-implicit-function-declaration
-fno-strict-aliasing -std=gnu99 -g -O2 -fstack-protector --param=ssp-buffer-size=4
-Wformat -Werror=format-security -I/usr/include/samba-4.0 -D_FILE_OFFSET_BITS=64
-D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -Wl,-Bsymbolic-functions -Wl,-z -Wl,relro -o
.libs/sss_ssh_authorizedkeys src/sss_client/sss_ssh_authorizedkeys-common.o
src/sss_client/ssh/sss_ssh_authorizedkeys-sss_ssh_client.o
src/sss_client/ssh/sss_ssh_authorizedkeys-sss_ssh_authorizedkeys.o -lpthread -ltalloc
/usr/lib/x86_64-linux-gnu/libpopt.so ./.libs/libsss_crypt.so ./.libs/libsss_debug.so
./.libs/libsss_child.so ./.libs/libsss_util.so -Wl,-rpath
-Wl,/usr/lib/x86_64-linux-gnu/sssd
> ./.libs/libsss_util.so: undefined reference to `sss_base64_decode'
> ./.libs/libsss_util.so: undefined reference to `s3crypt_gen_salt'
> ./.libs/libsss_util.so: undefined reference to `sss_base64_encode'
> ./.libs/libsss_util.so: undefined reference to `s3crypt_sha512'
> collect2: error: ld returned 1 exit status
ACK
fixed the build issue I had, thanks!
That's bizzare. But obviously works for Timo and still works in Fedora
as well.
Ack++, pushed to master.