public inbox for gentoo-commits@lists.gentoo.org
 help / color / mirror / Atom feed
From: "Miroslav Šulc" <fordfrog@gentoo.org>
To: gentoo-commits@lists.gentoo.org
Subject: [gentoo-commits] repo/gentoo:master commit in: app-cdr/brasero/files/, app-cdr/brasero/
Date: Sun,  7 Jul 2024 08:56:36 +0000 (UTC)	[thread overview]
Message-ID: <1720342590.42f46ee5b25ca13a3fb9f9780fc07414df2ce624.fordfrog@gentoo> (raw)

commit:     42f46ee5b25ca13a3fb9f9780fc07414df2ce624
Author:     Miroslav Šulc <fordfrog <AT> gentoo <DOT> org>
AuthorDate: Sun Jul  7 08:56:16 2024 +0000
Commit:     Miroslav Šulc <fordfrog <AT> gentoo <DOT> org>
CommitDate: Sun Jul  7 08:56:30 2024 +0000
URL:        https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=42f46ee5

app-cdr/brasero: fixed incompatible pointers with a better patch

Bug: https://bugs.gentoo.org/process_bug.cgi
Signed-off-by: Miroslav Šulc <fordfrog <AT> gentoo.org>

 ...sero-3.12.3.ebuild => brasero-3.12.3-r1.ebuild} |  0
 .../brasero-3.12.3-incompatible-pointers.patch     | 56 +++++++++++++++++++---
 2 files changed, 49 insertions(+), 7 deletions(-)

diff --git a/app-cdr/brasero/brasero-3.12.3.ebuild b/app-cdr/brasero/brasero-3.12.3-r1.ebuild
similarity index 100%
rename from app-cdr/brasero/brasero-3.12.3.ebuild
rename to app-cdr/brasero/brasero-3.12.3-r1.ebuild

diff --git a/app-cdr/brasero/files/brasero-3.12.3-incompatible-pointers.patch b/app-cdr/brasero/files/brasero-3.12.3-incompatible-pointers.patch
index edfdb8786cd9..e8c6dd58a480 100644
--- a/app-cdr/brasero/files/brasero-3.12.3-incompatible-pointers.patch
+++ b/app-cdr/brasero/files/brasero-3.12.3-incompatible-pointers.patch
@@ -1,14 +1,42 @@
+From 5cdefa8c76ddb797bce8b67a3f5767678bd36a5a Mon Sep 17 00:00:00 2001
+From: sid <sidtosh4@gmail.com>
+Date: Mon, 3 Jun 2024 18:51:08 +0100
+Subject: [PATCH] Fix gcc 14.x build failure (due to
+ -Wincompatible-pointer-types)
+
+The changes for 'brasero-drive-properties.c' are kept inline with
+'brasero-burn-options.c' (public API) for the sake of consistency.
+
+Fixes: https://gitlab.gnome.org/GNOME/brasero/-/issues/370
 --- a/libbrasero-burn/brasero-drive-properties.c
 +++ b/libbrasero-burn/brasero-drive-properties.c
-@@ -844,7 +844,7 @@ brasero_drive_properties_set_property (GObject *object,
- 		/* NOTE: no need to unref a potential previous session since
- 		 * it's only set at construct time */
- 		session = g_value_get_object (value);
+@@ -835,23 +835,19 @@ brasero_drive_properties_set_property (GObject *object,
+ 				       GParamSpec *pspec)
+ {
+ 	BraseroDrivePropertiesPrivate *priv;
+-	BraseroBurnSession *session;
+ 
+ 	priv = BRASERO_DRIVE_PROPERTIES_PRIVATE (object);
+ 
+ 	switch (property_id) {
+ 	case PROP_SESSION: /* Readable and only writable at creation time */
+-		/* NOTE: no need to unref a potential previous session since
+-		 * it's only set at construct time */
+-		session = g_value_get_object (value);
 -		priv->session = g_object_ref (session);
-+		priv->session = (BraseroSessionCfg *)g_object_ref (session);
++		priv->session = g_object_ref (g_value_get_object (value));
  
  		brasero_drive_properties_update (BRASERO_DRIVE_PROPERTIES (object));
- 		priv->valid_sig = g_signal_connect (session,
+-		priv->valid_sig = g_signal_connect (session,
++		priv->valid_sig = g_signal_connect (priv->session,
+ 						    "is-valid",
+ 						    G_CALLBACK (brasero_drive_properties_is_valid_cb),
+ 						    object);
+-		priv->output_sig = g_signal_connect (session,
++		priv->output_sig = g_signal_connect (priv->session,
+ 						     "output-changed",
+ 						     G_CALLBACK (brasero_drive_properties_output_changed_cb),
+ 						     object);
 --- a/libbrasero-utils/brasero-metadata.c
 +++ b/libbrasero-utils/brasero-metadata.c
 @@ -665,7 +665,7 @@ brasero_metadata_install_missing_plugins (BraseroMetadata *self)
@@ -16,7 +44,21 @@
  	context = gst_install_plugins_context_new ();
  	gst_install_plugins_context_set_xid (context, brasero_metadata_get_xid (self));
 -	status = gst_install_plugins_async ((gchar **) details->pdata,
-+	status = gst_install_plugins_async ((const gchar * const *) details->pdata,
++	status = gst_install_plugins_async ((const gchar* const*) details->pdata,
  					    context,
  					    brasero_metadata_install_plugins_result,
  					    downloads);
+--- a/libbrasero-utils/brasero-pk.c
++++ b/libbrasero-utils/brasero-pk.c
+@@ -230,7 +230,7 @@ brasero_pk_install_gstreamer_plugin (BraseroPK *package,
+ 
+ 	context = gst_install_plugins_context_new ();
+ 	gst_install_plugins_context_set_xid (context, xid);
+-	status = gst_install_plugins_async ((gchar **) gst_plugins->pdata,
++	status = gst_install_plugins_async ((const gchar* const*) gst_plugins->pdata,
+ 	                                    context,
+ 	                                    brasero_pk_install_gst_plugin_result,
+ 	                                    package);
+-- 
+GitLab
+


             reply	other threads:[~2024-07-07  8:56 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-07-07  8:56 Miroslav Šulc [this message]
  -- strict thread matches above, loose matches on Subject: below --
2017-12-20 20:23 [gentoo-commits] repo/gentoo:master commit in: app-cdr/brasero/files/, app-cdr/brasero/ Mart Raudsepp

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1720342590.42f46ee5b25ca13a3fb9f9780fc07414df2ce624.fordfrog@gentoo \
    --to=fordfrog@gentoo.org \
    --cc=gentoo-commits@lists.gentoo.org \
    --cc=gentoo-dev@lists.gentoo.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox