Bug 664648 – dev-db/redis-5.0_rc4 USE=luajit fails to build: ld: geohash_helper.o: undefined reference to symbol 'sin@@GLIBC_2.2.5'

• Status: RESOLVED FIXED
• Product: Gentoo Linux

[Description Harris Landgarten 2018-08-26 13:28:14 UTC]

Created attachment 545112 [details]
build log

These are the errors:

redis-check-aof.c: In function 'consumeNewline.part.0':
redis-check-aof.c:37:20: warning: '%s' directive writing up to 1023 bytes into a region of size 1004 [-Wformat-overflow=]
     sprintf(error, "0x%16llx: %s", (long long)epos, __buf); \
                    ^~~~~~~~~~~~~~                   ~~~~~
redis-check-aof.c:45:9: note: in expansion of macro 'ERROR'
         ERROR("Expected \\r\\n, got: %02x%02x",buf[0],buf[1]);
         ^~~~~
In file included from /usr/include/stdio.h:862,
                 from rio.h:35,
                 from server.h:36,
                 from redis-check-aof.c:31:
/usr/include/bits/stdio2.h:33:10: note: '__builtin___sprintf_chk' output between 21 and 1044 bytes into a destination of size 1024
   return __builtin___sprintf_chk (__s, __USE_FORTIFY_LEVEL - 1,
          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
       __bos (__s), __fmt, __va_arg_pack ());
       ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
redis-check-aof.c: In function 'readLong':
redis-check-aof.c:37:20: warning: '%s' directive writing up to 1023 bytes into a region of size 1004 [-Wformat-overflow=]
     sprintf(error, "0x%16llx: %s", (long long)epos, __buf); \
                    ^~~~~~~~~~~~~~                   ~~~~~
redis-check-aof.c:58:9: note: in expansion of macro 'ERROR'
         ERROR("Expected prefix '%c', got: '%c'",prefix,buf[0]);
         ^~~~~
In file included from /usr/include/stdio.h:862,
                 from rio.h:35,
                 from server.h:36,
                 from redis-check-aof.c:31:
/usr/include/bits/stdio2.h:33:10: note: '__builtin___sprintf_chk' output between 21 and 1044 bytes into a destination of size 1024

[Comment 1 Tomáš Mózes 2018-08-26 17:07:44 UTC]

Please attach your emerge --info output.

[Comment 2 Harris Landgarten 2018-08-26 17:24:56 UTC]

Created attachment 545154 [details]
emerge info

[Comment 3 Tomáš Mózes 2018-08-27 05:34:17 UTC]

Seems like the reason is that I accidentally removed -lm from linking flags, can you please test with https://raw.githubusercontent.com/gentoo/gentoo/159e0c2bca9bc8b1ba4f6134097faaef7d3e5b25/dev-db/redis/files/redis-5.0-sharedlua.patch?

[Comment 4 Holger 2018-08-30 20:05:03 UTC]

Yes. The updated patch worked.

[Comment 5 Tomáš Mózes 2018-08-31 05:23:18 UTC]

Thanks for testing out.

[Comment 6 Larry the Git Cow 2018-09-05 21:20:02 UTC]

The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=9638f65d0442ee6ceb72fbf845e645bcffa7a1be

commit 9638f65d0442ee6ceb72fbf845e645bcffa7a1be
Author:     Tomas Mozes <[e-mail]>
AuthorDate: 2018-08-27 05:30:39 +0000
Commit:     Robin H. Johnson <[e-mail]>
CommitDate: 2018-09-05 21:19:53 +0000

    dev-db/redis: re-add -lm for luajit
    
    Closes: https://bugs.gentoo.org/664648
    Package-Manager: Portage-2.3.45, Repoman-2.3.10
    (cherry picked from commit 159e0c2bca9bc8b1ba4f6134097faaef7d3e5b25)
    Signed-off-by: Robin H. Johnson <[e-mail]>
    Fixes: https://github.com/gentoo/gentoo/pull/9706

 dev-db/redis/files/redis-5.0-sharedlua.patch | 15 +++++++--------
 1 file changed, 7 insertions(+), 8 deletions(-)