diff options
author | Jakov Smolić <jsmolic@gentoo.org> | 2022-04-27 23:09:17 +0200 |
---|---|---|
committer | Jakov Smolić <jsmolic@gentoo.org> | 2022-04-27 23:09:28 +0200 |
commit | 32fd2d327a841841a0aa1440994fd072eb7973b1 (patch) | |
tree | 6d9db7b58405f8564663d100ea749cb4b059bb65 /sys-devel/bmake | |
parent | net-misc/gallery-dl: drop old 1.20.5 (diff) | |
download | gentoo-32fd2d327a841841a0aa1440994fd072eb7973b1.tar.gz gentoo-32fd2d327a841841a0aa1440994fd072eb7973b1.tar.bz2 gentoo-32fd2d327a841841a0aa1440994fd072eb7973b1.zip |
sys-devel/bmake: Fix compilation warnings
Closes: https://bugs.gentoo.org/841362
Signed-off-by: Jakov Smolić <jsmolic@gentoo.org>
Diffstat (limited to 'sys-devel/bmake')
-rw-r--r-- | sys-devel/bmake/bmake-20220418.ebuild | 5 | ||||
-rw-r--r-- | sys-devel/bmake/files/bmake-20220418-warnings.patch | 38 |
2 files changed, 42 insertions, 1 deletions
diff --git a/sys-devel/bmake/bmake-20220418.ebuild b/sys-devel/bmake/bmake-20220418.ebuild index 7b11aa7cebf4..5505e007c65a 100644 --- a/sys-devel/bmake/bmake-20220418.ebuild +++ b/sys-devel/bmake/bmake-20220418.ebuild @@ -17,7 +17,10 @@ SLOT="0" KEYWORDS="~amd64 ~ppc ~x86" # Skip failing test (sandbox and csh) -PATCHES=( "${FILESDIR}"/${PN}-20210206-tests.patch ) +PATCHES=( + "${FILESDIR}"/${PN}-20210206-tests.patch + "${FILESDIR}"/${PN}-20220418-warnings.patch +) src_prepare() { default diff --git a/sys-devel/bmake/files/bmake-20220418-warnings.patch b/sys-devel/bmake/files/bmake-20220418-warnings.patch new file mode 100644 index 000000000000..1fccb91e95a3 --- /dev/null +++ b/sys-devel/bmake/files/bmake-20220418-warnings.patch @@ -0,0 +1,38 @@ +# https://bugs.gentoo.org/841362 +# +# Fix -Wimplicit-function-declaration and -Wunused-result warnings +# +# meta.c: In function ‘meta_name’: +# meta.c:262:21: warning: implicit declaration of function ‘strlcpy’; did you mean ‘strncpy’? [-Wimplicit-function-declaration] +# 262 | strlcpy(rp, tname_base, sizeof buf - (size_t)(rp - buf)); +# | ^~~~~~~ +# | strncpy +# +# meta.c: In function ‘meta_job_error’: +# meta.c:774:5: warning: ignoring return value of ‘getcwd’ declared with attribute ‘warn_unused_result’ [-Wunused-result] +# 774 | getcwd(cwd, sizeof cwd); +# | ^~~~~~~~~~~~~~~~~~~~~~~ + +--- a/meta.c ++++ b/meta.c +@@ -106,6 +106,10 @@ + char * stresep(char **, const char *, int); + #endif + ++#if !defined(HAVE_STRLCPY) ++size_t strlcpy(char *, const char *, size_t); ++#endif ++ + /* + * Filemon is a kernel module which snoops certain syscalls. + * +@@ -769,7 +773,8 @@ + } + if (gn != NULL) + Global_Set(".ERROR_TARGET", GNode_Path(gn)); +- getcwd(cwd, sizeof cwd); ++ if (getcwd(cwd, sizeof cwd) == NULL) ++ err(1, "Could not get current working directory"); + Global_Set(".ERROR_CWD", cwd); + if (pbm->meta_fname[0] != '\0') { + Global_Set(".ERROR_META_FILE", pbm->meta_fname); |