Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 744946 - net-libs/mbedtls-2.24.0 uses -Werror for build
Summary: net-libs/mbedtls-2.24.0 uses -Werror for build
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Anthony Basile
URL:
Whiteboard:
Keywords: PATCH
Depends on:
Blocks: werror gcc-11
  Show dependency tree
 
Reported: 2020-09-27 13:23 UTC by Agostino Sarubbo
Modified: 2021-09-19 07:12 UTC (History)
2 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Agostino Sarubbo gentoo-dev 2020-09-27 13:23:38 UTC
https://blogs.gentoo.org/ago/2020/07/04/gentoo-tinderbox/

Issue: net-libs/mbedtls-2.24.0 uses -Werror for build.
Discovered on: amd64 (internal ref: ci)

-Werror is not recommended for releases and should always be disabled when encountered in build-logs, because there are numerous cases where this breaks without purpose.
See also: https://devmanual.gentoo.org/ebuild-writing/common-mistakes/index.html

NOTE: If you want to change the summary, to avoid auto-filed duplicates, please at least keep the word '-Werror'
Comment 1 Sergei Trofimovich (RETIRED) gentoo-dev 2020-10-13 21:56:29 UTC
The -Werror also happens to break the build on gcc-11: https://github.com/ARMmbed/mbedtls/issues/3782
Comment 2 Sergei Trofimovich (RETIRED) gentoo-dev 2020-11-16 08:05:07 UTC
CMakeLists.txt already exposes a mechanism to disable -Werror. The following seems to work:

--- a/net-libs/mbedtls/mbedtls-2.24.0.ebuild
+++ b/net-libs/mbedtls/mbedtls-2.24.0.ebuild
@@ -53,6 +53,7 @@ multilib_src_configure() {
                -DUSE_SHARED_MBEDTLS_LIBRARY=ON
                -DINSTALL_MBEDTLS_HEADERS=ON
                -DLIB_INSTALL_DIR="${EPREFIX}/usr/$(get_libdir)"
+               -DMBEDTLS_FATAL_WARNINGS=OFF
        )

        cmake_src_configure
Comment 4 Larry the Git Cow gentoo-dev 2021-09-19 07:12:51 UTC
The bug has been closed via the following commit(s):

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

commit 93f6e77da684f70ea10256e40f88cab0d45573d9
Author:     Andreas K. Hüttel <dilfridge@gentoo.org>
AuthorDate: 2021-09-19 06:23:03 +0000
Commit:     Andreas K. Hüttel <dilfridge@gentoo.org>
CommitDate: 2021-09-19 07:12:34 +0000

    net-libs/mbedtls: Remove old
    
    Closes: https://bugs.gentoo.org/744946
    Package-Manager: Portage-3.0.20, Repoman-3.0.3
    Signed-off-by: Andreas K. Huettel <dilfridge@gentoo.org>

 net-libs/mbedtls/Manifest               |  2 -
 net-libs/mbedtls/mbedtls-2.16.10.ebuild | 93 ---------------------------------
 net-libs/mbedtls/mbedtls-2.16.11.ebuild | 93 ---------------------------------
 3 files changed, 188 deletions(-)