From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from lists.gentoo.org (pigeon.gentoo.org [208.92.234.80]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits)) (No client certificate requested) by finch.gentoo.org (Postfix) with ESMTPS id 562CA15A7D9 for ; Tue, 21 Mar 2023 20:31:25 +0000 (UTC) Received: from pigeon.gentoo.org (localhost [127.0.0.1]) by pigeon.gentoo.org (Postfix) with SMTP id 78762E08F3; Tue, 21 Mar 2023 20:31:23 +0000 (UTC) Received: from smtp.gentoo.org (woodpecker.gentoo.org [140.211.166.183]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits)) (No client certificate requested) by pigeon.gentoo.org (Postfix) with ESMTPS id 495AEE08F3 for ; Tue, 21 Mar 2023 20:31:23 +0000 (UTC) Received: from oystercatcher.gentoo.org (oystercatcher.gentoo.org [148.251.78.52]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits)) (No client certificate requested) by smtp.gentoo.org (Postfix) with ESMTPS id 68A4B34116E for ; Tue, 21 Mar 2023 20:31:22 +0000 (UTC) Received: from localhost.localdomain (localhost [IPv6:::1]) by oystercatcher.gentoo.org (Postfix) with ESMTP id D52978EE for ; Tue, 21 Mar 2023 20:31:20 +0000 (UTC) From: "Michał Górny" To: gentoo-commits@lists.gentoo.org Content-Transfer-Encoding: 8bit Content-type: text/plain; charset=UTF-8 Reply-To: gentoo-dev@lists.gentoo.org, "Michał Górny" Message-ID: <1679430677.b33d8f72988e5dd16750351a113ab374b9c3526f.mgorny@gentoo> Subject: [gentoo-commits] repo/gentoo:master commit in: sys-devel/clang-common/ X-VCS-Repository: repo/gentoo X-VCS-Files: sys-devel/clang-common/clang-common-15.0.7-r4.ebuild sys-devel/clang-common/clang-common-15.0.7-r5.ebuild sys-devel/clang-common/clang-common-16.0.0-r1.ebuild sys-devel/clang-common/clang-common-16.0.0.9999.ebuild sys-devel/clang-common/clang-common-16.0.0.ebuild sys-devel/clang-common/clang-common-17.0.0.9999.ebuild sys-devel/clang-common/clang-common-17.0.0_pre20230314-r1.ebuild sys-devel/clang-common/clang-common-17.0.0_pre20230314.ebuild X-VCS-Directories: sys-devel/clang-common/ X-VCS-Committer: mgorny X-VCS-Committer-Name: Michał Górny X-VCS-Revision: b33d8f72988e5dd16750351a113ab374b9c3526f X-VCS-Branch: master Date: Tue, 21 Mar 2023 20:31:20 +0000 (UTC) Precedence: bulk List-Post: List-Help: List-Unsubscribe: List-Subscribe: List-Id: Gentoo Linux mail X-BeenThere: gentoo-commits@lists.gentoo.org X-Auto-Response-Suppress: DR, RN, NRN, OOF, AutoReply X-Archives-Salt: 22ab3dc4-540c-4037-9c3a-0d8871e29c8d X-Archives-Hash: e1ed52d9d165624d63a5af0d49a7216f commit: b33d8f72988e5dd16750351a113ab374b9c3526f Author: Gavin D. Howard gavinhoward com> AuthorDate: Tue Mar 21 03:55:54 2023 +0000 Commit: Michał Górny gentoo org> CommitDate: Tue Mar 21 20:31:17 2023 +0000 URL: https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=b33d8f72 sys-devel/clang-common: Fix a warning when using -Weverything This warning is caused by the fortify.h header that has a system-reserved macro name. This commit simply adds a pragma to tell Clang that the header is a system header. I surrounded the pragma with `#ifdef __clang__` and such in case this header is used for GCC or other compilers, but according to a Gentoo maintainer, this is not strictly necessary. I incremented the revision number on 15.0.7-4, 16.0.0, and 17.0.0_pre20230314 but I did not increment them on 16.0.0.999 and 17.0.0.999. I may be wrong, but it doesn't seem like revisions should be incremented on ebuilds that follow the latest commits of projects. I thought I had read about that somewhere, but the only source I could find was [1], and that one did not mention anything about 9999-type ebuilds and revision numbers. That said, [2] did have an example where a _pre* ebuild had a revision number, so I specifically added a revision number to 17.0.0_pre20230314. Also, [1] said to not add a revision number if the fix is trivial and would not be worth it with the compile times. However, this package is trivial to "compile" since it only installs certain files, so I thought it worth it to increment revisions. If the Gentoo authors disagree, I don't mind at all, though. [1]: https://devmanual.gentoo.org/general-concepts/ebuild-revisions/index.html [2]: https://devmanual.gentoo.org/ebuild-writing/file-format/index.html#file-naming-rules Signed-off-by: Gavin D. Howard gavinhoward.com> Closes: https://github.com/gentoo/gentoo/pull/30276 Signed-off-by: Michał Górny gentoo.org> .../{clang-common-15.0.7-r4.ebuild => clang-common-15.0.7-r5.ebuild} | 3 +++ .../{clang-common-16.0.0.ebuild => clang-common-16.0.0-r1.ebuild} | 3 +++ sys-devel/clang-common/clang-common-16.0.0.9999.ebuild | 3 +++ sys-devel/clang-common/clang-common-17.0.0.9999.ebuild | 3 +++ ....0_pre20230314.ebuild => clang-common-17.0.0_pre20230314-r1.ebuild} | 3 +++ 5 files changed, 15 insertions(+) diff --git a/sys-devel/clang-common/clang-common-15.0.7-r4.ebuild b/sys-devel/clang-common/clang-common-15.0.7-r5.ebuild similarity index 98% rename from sys-devel/clang-common/clang-common-15.0.7-r4.ebuild rename to sys-devel/clang-common/clang-common-15.0.7-r5.ebuild index c7af956a5dd7..8d526d297a03 100644 --- a/sys-devel/clang-common/clang-common-15.0.7-r4.ebuild +++ b/sys-devel/clang-common/clang-common-15.0.7-r5.ebuild @@ -107,6 +107,9 @@ src_install() { # without optimization and that would at the very least be very noisy # during builds and at worst trigger many -Werror builds. cat >> "${ED}/usr/include/gentoo/fortify.h" <<- EOF || die + #ifdef __clang__ + # pragma clang system_header + #endif #ifndef _FORTIFY_SOURCE # if defined(__has_feature) # define __GENTOO_HAS_FEATURE(x) __has_feature(x) diff --git a/sys-devel/clang-common/clang-common-16.0.0.ebuild b/sys-devel/clang-common/clang-common-16.0.0-r1.ebuild similarity index 98% rename from sys-devel/clang-common/clang-common-16.0.0.ebuild rename to sys-devel/clang-common/clang-common-16.0.0-r1.ebuild index 784fa3ae7a88..4acd801b683d 100644 --- a/sys-devel/clang-common/clang-common-16.0.0.ebuild +++ b/sys-devel/clang-common/clang-common-16.0.0-r1.ebuild @@ -106,6 +106,9 @@ src_install() { # without optimization and that would at the very least be very noisy # during builds and at worst trigger many -Werror builds. cat >> "${ED}/usr/include/gentoo/fortify.h" <<- EOF || die + #ifdef __clang__ + # pragma clang system_header + #endif #ifndef _FORTIFY_SOURCE # if defined(__has_feature) # define __GENTOO_HAS_FEATURE(x) __has_feature(x) diff --git a/sys-devel/clang-common/clang-common-16.0.0.9999.ebuild b/sys-devel/clang-common/clang-common-16.0.0.9999.ebuild index 808c1e1a0ae1..5b4a6647f3ca 100644 --- a/sys-devel/clang-common/clang-common-16.0.0.9999.ebuild +++ b/sys-devel/clang-common/clang-common-16.0.0.9999.ebuild @@ -106,6 +106,9 @@ src_install() { # without optimization and that would at the very least be very noisy # during builds and at worst trigger many -Werror builds. cat >> "${ED}/usr/include/gentoo/fortify.h" <<- EOF || die + #ifdef __clang__ + # pragma clang system_header + #endif #ifndef _FORTIFY_SOURCE # if defined(__has_feature) # define __GENTOO_HAS_FEATURE(x) __has_feature(x) diff --git a/sys-devel/clang-common/clang-common-17.0.0.9999.ebuild b/sys-devel/clang-common/clang-common-17.0.0.9999.ebuild index 808c1e1a0ae1..5b4a6647f3ca 100644 --- a/sys-devel/clang-common/clang-common-17.0.0.9999.ebuild +++ b/sys-devel/clang-common/clang-common-17.0.0.9999.ebuild @@ -106,6 +106,9 @@ src_install() { # without optimization and that would at the very least be very noisy # during builds and at worst trigger many -Werror builds. cat >> "${ED}/usr/include/gentoo/fortify.h" <<- EOF || die + #ifdef __clang__ + # pragma clang system_header + #endif #ifndef _FORTIFY_SOURCE # if defined(__has_feature) # define __GENTOO_HAS_FEATURE(x) __has_feature(x) diff --git a/sys-devel/clang-common/clang-common-17.0.0_pre20230314.ebuild b/sys-devel/clang-common/clang-common-17.0.0_pre20230314-r1.ebuild similarity index 98% rename from sys-devel/clang-common/clang-common-17.0.0_pre20230314.ebuild rename to sys-devel/clang-common/clang-common-17.0.0_pre20230314-r1.ebuild index 808c1e1a0ae1..5b4a6647f3ca 100644 --- a/sys-devel/clang-common/clang-common-17.0.0_pre20230314.ebuild +++ b/sys-devel/clang-common/clang-common-17.0.0_pre20230314-r1.ebuild @@ -106,6 +106,9 @@ src_install() { # without optimization and that would at the very least be very noisy # during builds and at worst trigger many -Werror builds. cat >> "${ED}/usr/include/gentoo/fortify.h" <<- EOF || die + #ifdef __clang__ + # pragma clang system_header + #endif #ifndef _FORTIFY_SOURCE # if defined(__has_feature) # define __GENTOO_HAS_FEATURE(x) __has_feature(x)