Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 879797

Summary: net-misc/turbovnc: possible configure breakage with Clang 16
Product: Gentoo Linux Reporter: Matt Jolly <kangie>
Component: Current packagesAssignee: Viorel Munteanu <ceamac>
Status: RESOLVED FIXED    
Severity: normal CC: sam
Priority: Normal Keywords: PullRequest
Version: unspecified   
Hardware: All   
OS: Linux   
See Also: https://github.com/gentoo/gentoo/pull/30965
Whiteboard:
Package list:
Runtime testing required: ---
Bug Depends on:    
Bug Blocks: 906027    
Attachments: clang-14 log
clang-15 log
clang.diff

Description Matt Jolly gentoo-dev 2022-11-04 23:10:04 UTC
Created attachment 827655 [details]
clang-14 log

This package has been identified via automation as potentially impacted by upcoming Clang 16 changes.
The following have been identified as potential issues:

[-Werror,-Wimplicit-function-declaration]

[-Werror,-Wimplicit-function-declaration]
error: call to undeclared library function 'strlcat' with type 'unsigned long (char *, const char *, unsigned long)'; ISO C99 and later do not support implicit function declarations [-Werror,-Wimplicit-function-declaration]
error: call to undeclared library function 'strlcpy' with type 'unsigned long (char *, const char *, unsigned long)'; ISO C99 and later do not support implicit function declarations [-Werror,-Wimplicit-function-declaration]
Comment 1 Matt Jolly gentoo-dev 2022-11-04 23:10:27 UTC
Created attachment 827657 [details]
clang-15 log
Comment 2 Matt Jolly gentoo-dev 2022-11-04 23:10:43 UTC
Created attachment 827659 [details, diff]
clang.diff
Comment 3 Viorel Munteanu gentoo-dev 2022-11-05 09:50:13 UTC
I looked at the diff.  It is true clang-14 gives a warning while clang-15 gives an error, but they both fail afterwards at link time (the functions in question do not exist by default, they are in <bsd/string.h> and -lbsd).  So in this case the functions are not implicitly declared, they are simply missing.

Also if the functions did exist, `check_symbol_exists` has an include file, so they would both succeed.
Comment 4 Larry the Git Cow gentoo-dev 2023-05-12 04:40:44 UTC
The bug has been closed via the following commit(s):

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

commit de291e9ee855a904ec9f56d1339d0be7a464a2a2
Author:     Viorel Munteanu <ceamac@gentoo.org>
AuthorDate: 2023-05-10 07:53:21 +0000
Commit:     Viorel Munteanu <ceamac@gentoo.org>
CommitDate: 2023-05-12 04:34:46 +0000

    net-misc/turbovnc: fix clang-16 warning QA check
    
    Ignore BSD-only functions.
    
    Closes: https://bugs.gentoo.org/879797
    Signed-off-by: Viorel Munteanu <ceamac@gentoo.org>

 net-misc/turbovnc/turbovnc-3.0.3.ebuild | 3 +++
 1 file changed, 3 insertions(+)