From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: <gentoo-commits+bounces-1401644-garchives=archives.gentoo.org@lists.gentoo.org> 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 8A152158090 for <garchives@archives.gentoo.org>; Sat, 28 May 2022 02:17:36 +0000 (UTC) Received: from pigeon.gentoo.org (localhost [127.0.0.1]) by pigeon.gentoo.org (Postfix) with SMTP id BB56BE089A; Sat, 28 May 2022 02:17:35 +0000 (UTC) Received: from smtp.gentoo.org (smtp.gentoo.org [IPv6:2001:470:ea4a:1:5054:ff:fec7:86e4]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) by pigeon.gentoo.org (Postfix) with ESMTPS id 9E80CE089A for <gentoo-commits@lists.gentoo.org>; Sat, 28 May 2022 02:17:35 +0000 (UTC) Received: from oystercatcher.gentoo.org (unknown [IPv6:2a01:4f8:202:4333:225:90ff:fed9:fc84]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) by smtp.gentoo.org (Postfix) with ESMTPS id 75AF73414AA for <gentoo-commits@lists.gentoo.org>; Sat, 28 May 2022 02:17:33 +0000 (UTC) Received: from localhost.localdomain (localhost [IPv6:::1]) by oystercatcher.gentoo.org (Postfix) with ESMTP id C517AD3 for <gentoo-commits@lists.gentoo.org>; Sat, 28 May 2022 02:17:31 +0000 (UTC) From: "Matt Turner" <mattst88@gentoo.org> To: gentoo-commits@lists.gentoo.org Content-Transfer-Encoding: 8bit Content-type: text/plain; charset=UTF-8 Reply-To: gentoo-dev@lists.gentoo.org, "Matt Turner" <mattst88@gentoo.org> Message-ID: <1653704245.d8566b90d226a1c086bfb359fbdf458b0fc196dd.mattst88@gentoo> Subject: [gentoo-commits] repo/gentoo:master commit in: dev-libs/gjs/files/ X-VCS-Repository: repo/gentoo X-VCS-Files: dev-libs/gjs/files/gjs-1.64.5-gcc-11-support.patch X-VCS-Directories: dev-libs/gjs/files/ X-VCS-Committer: mattst88 X-VCS-Committer-Name: Matt Turner X-VCS-Revision: d8566b90d226a1c086bfb359fbdf458b0fc196dd X-VCS-Branch: master Date: Sat, 28 May 2022 02:17:31 +0000 (UTC) Precedence: bulk List-Post: <mailto:gentoo-commits@lists.gentoo.org> List-Help: <mailto:gentoo-commits+help@lists.gentoo.org> List-Unsubscribe: <mailto:gentoo-commits+unsubscribe@lists.gentoo.org> List-Subscribe: <mailto:gentoo-commits+subscribe@lists.gentoo.org> List-Id: Gentoo Linux mail <gentoo-commits.gentoo.org> X-BeenThere: gentoo-commits@lists.gentoo.org X-Auto-Response-Suppress: DR, RN, NRN, OOF, AutoReply X-Archives-Salt: 78da2caa-75a7-4bdd-8131-d22e02fa9dd1 X-Archives-Hash: af2264cf67e00c97d908f39f902608cd commit: d8566b90d226a1c086bfb359fbdf458b0fc196dd Author: Matt Turner <mattst88 <AT> gentoo <DOT> org> AuthorDate: Sat May 28 02:14:20 2022 +0000 Commit: Matt Turner <mattst88 <AT> gentoo <DOT> org> CommitDate: Sat May 28 02:17:25 2022 +0000 URL: https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=d8566b90 dev-libs/gjs: Add missing GCC-11 support patch Signed-off-by: Matt Turner <mattst88 <AT> gentoo.org> dev-libs/gjs/files/gjs-1.64.5-gcc-11-support.patch | 129 +++++++++++++++++++++ 1 file changed, 129 insertions(+) diff --git a/dev-libs/gjs/files/gjs-1.64.5-gcc-11-support.patch b/dev-libs/gjs/files/gjs-1.64.5-gcc-11-support.patch new file mode 100644 index 000000000000..3808b6d7862b --- /dev/null +++ b/dev-libs/gjs/files/gjs-1.64.5-gcc-11-support.patch @@ -0,0 +1,129 @@ +From 757d42d87e8a3b52a0782bc9cbd33c788ecc34e4 Mon Sep 17 00:00:00 2001 +From: Philip Chimento <philip.chimento@gmail.com> +Date: Mon, 15 Feb 2021 20:40:43 -0800 +Subject: [PATCH 1/2] GjsPrivate: Remove volatile from g_once_init_enter flag + +On platforms where g_once_init_enter() is defined to use C11 atomic +builtins, passing a pointer to a volatile value will generate a warning +in GCC 11 and later. + +More info about the GCC change: +https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95378 +https://gcc.gnu.org/pipermail/gcc-patches/2020-June/548283.html + +More info about changes made in GLib: +https://gitlab.gnome.org/GNOME/glib/-/issues/600 + +See: #376 +(cherry picked from commit 989ac9ac723dc1c8b6b8961292f236c558f5c0f0) +--- + libgjs-private/gjs-util.c | 8 ++++---- + 1 file changed, 4 insertions(+), 4 deletions(-) + +diff --git a/libgjs-private/gjs-util.c b/libgjs-private/gjs-util.c +index 20a732d5..f7f3189a 100644 +--- a/libgjs-private/gjs-util.c ++++ b/libgjs-private/gjs-util.c +@@ -57,8 +57,8 @@ gjs_format_int_alternative_output(int n) + GType + gjs_locale_category_get_type(void) + { +- static volatile size_t g_define_type_id__volatile = 0; +- if (g_once_init_enter(&g_define_type_id__volatile)) { ++ static size_t gjs_locale_category_get_type = 0; ++ if (g_once_init_enter(&gjs_locale_category_get_type)) { + static const GEnumValue v[] = { + { GJS_LOCALE_CATEGORY_ALL, "GJS_LOCALE_CATEGORY_ALL", "all" }, + { GJS_LOCALE_CATEGORY_COLLATE, "GJS_LOCALE_CATEGORY_COLLATE", "collate" }, +@@ -72,9 +72,9 @@ gjs_locale_category_get_type(void) + GType g_define_type_id = + g_enum_register_static(g_intern_static_string("GjsLocaleCategory"), v); + +- g_once_init_leave(&g_define_type_id__volatile, g_define_type_id); ++ g_once_init_leave(&gjs_locale_category_get_type, g_define_type_id); + } +- return g_define_type_id__volatile; ++ return gjs_locale_category_get_type; + } + + /** +-- +2.35.1 + +From dade6ee66e07a1865dc380060abe921eaeeae763 Mon Sep 17 00:00:00 2001 +From: Philip Chimento <philip.chimento@gmail.com> +Date: Sun, 14 Feb 2021 12:20:09 -0800 +Subject: [PATCH 2/2] maint: Avoid g_once_init_enter error in GCC 11 + +On platforms where g_once_init_enter() is defined to use C11 atomic +builtins, passing a pointer to a volatile value is an error in GCC 11 and +later, in C++. + +More info about the GCC change: +https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95378 +https://gcc.gnu.org/pipermail/gcc-patches/2020-June/548283.html + +However, it's my understanding that in modern C++ there is no longer a +need to guard the initialization of these variables. Since C++11, static +local variables in a function are guaranteed to be initialized only once, +the first time control passes through that function. So we can just remove +the g_once_init_enter guard. + +More info: +https://en.cppreference.com/w/cpp/language/storage_duration#Static_local_variables + +Stack Overflow answers with quotations from the C++ standard: +https://stackoverflow.com/a/58804/172999 +https://stackoverflow.com/a/8102145/172999 + +Closes: #376 +(cherry picked from commit f02eaf3a9d3465915eb849428c2d9615e2184a4c) +--- + gjs/error-types.cpp | 32 +++++++++++++------------------- + 1 file changed, 13 insertions(+), 19 deletions(-) + +diff --git a/gjs/error-types.cpp b/gjs/error-types.cpp +index 86cb8789..5eba61b2 100644 +--- a/gjs/error-types.cpp ++++ b/gjs/error-types.cpp +@@ -31,24 +31,18 @@ G_DEFINE_QUARK(gjs-js-error-quark, gjs_js_error) + // clang-format on + + GType gjs_js_error_get_type(void) { +- static volatile GType g_type_id; +- +- if (g_once_init_enter(&g_type_id)) { +- static GEnumValue errors[] = { +- { GJS_JS_ERROR_ERROR, "Error", "error" }, +- { GJS_JS_ERROR_EVAL_ERROR, "EvalError", "eval-error" }, +- { GJS_JS_ERROR_INTERNAL_ERROR, "InternalError", "internal-error" }, +- { GJS_JS_ERROR_RANGE_ERROR, "RangeError", "range-error" }, +- { GJS_JS_ERROR_REFERENCE_ERROR, "ReferenceError", "reference-error" }, +- { GJS_JS_ERROR_STOP_ITERATION, "StopIteration", "stop-iteration" }, +- { GJS_JS_ERROR_SYNTAX_ERROR, "SyntaxError", "syntax-error" }, +- { GJS_JS_ERROR_TYPE_ERROR, "TypeError", "type-error" }, +- { GJS_JS_ERROR_URI_ERROR, "URIError", "uri-error" }, +- { 0, nullptr, nullptr } +- }; +- +- g_type_id = g_enum_register_static("GjsJSError", errors); +- } +- ++ static const GEnumValue errors[] = { ++ {GJS_JS_ERROR_ERROR, "Error", "error"}, ++ {GJS_JS_ERROR_EVAL_ERROR, "EvalError", "eval-error"}, ++ {GJS_JS_ERROR_INTERNAL_ERROR, "InternalError", "internal-error"}, ++ {GJS_JS_ERROR_RANGE_ERROR, "RangeError", "range-error"}, ++ {GJS_JS_ERROR_REFERENCE_ERROR, "ReferenceError", "reference-error"}, ++ {GJS_JS_ERROR_STOP_ITERATION, "StopIteration", "stop-iteration"}, ++ {GJS_JS_ERROR_SYNTAX_ERROR, "SyntaxError", "syntax-error"}, ++ {GJS_JS_ERROR_TYPE_ERROR, "TypeError", "type-error"}, ++ {GJS_JS_ERROR_URI_ERROR, "URIError", "uri-error"}, ++ {0, nullptr, nullptr}}; ++ // Initialization of static local variable guaranteed only once in C++11 ++ static GType g_type_id = g_enum_register_static("GjsJSError", errors); + return g_type_id; + } +-- +2.35.1 +