From: Yann E. MORIN <yann.morin.1998@free.fr>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH v2 2/2] libgtk3: bump version to 3.14.5
Date: Tue, 11 Nov 2014 12:44:31 +0100 [thread overview]
Message-ID: <20141111114431.GE4240@free.fr> (raw)
In-Reply-To: <1415704522-14201-3-git-send-email-Vincent.Riera@imgtec.com>
Vicente, All,
On 2014-11-11 11:15 +0000, Vicente Olivert Riera spake thusly:
> - Bump version to 3.14.5
> - Add a hash file
> - Add a new patch to fix a regression
>
> Signed-off-by: Vicente Olivert Riera <Vincent.Riera@imgtec.com>
Manually downloaded, and manually verified the hash.
Patch indeed exactly as from upstream.
Reviewed-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Regards,
Yann E. MORIN.
> ---
> ...lug-is-conditionally-supported-on-X11-onl.patch | 57 ++++++++++++++++++++
> package/libgtk3/libgtk3.hash | 2 +
> package/libgtk3/libgtk3.mk | 2 +-
> 3 files changed, 60 insertions(+), 1 deletions(-)
> create mode 100644 package/libgtk3/0006-window-GtkPlug-is-conditionally-supported-on-X11-onl.patch
> create mode 100644 package/libgtk3/libgtk3.hash
>
> diff --git a/package/libgtk3/0006-window-GtkPlug-is-conditionally-supported-on-X11-onl.patch b/package/libgtk3/0006-window-GtkPlug-is-conditionally-supported-on-X11-onl.patch
> new file mode 100644
> index 0000000..081cd1e
> --- /dev/null
> +++ b/package/libgtk3/0006-window-GtkPlug-is-conditionally-supported-on-X11-onl.patch
> @@ -0,0 +1,57 @@
> +window: GtkPlug is conditionally supported on X11 only
> +
> +Apply an upstream patch to fix a build regression in 3.14.5. This patch
> +will be included in the next 3.14.6 release, so we can remove it in the
> +next version bump.
> +
> +Error message:
> +
> +./.libs/libgtk-3.so: undefined reference to `GTK_IS_PLUG'
> +
> +Bug report:
> + https://bugzilla.gnome.org/show_bug.cgi?id=739885
> +
> +Upstream commit:
> + https://git.gnome.org/browse/gtk+/commit/?id=36fedb0c4b373272afc2d76a7c790d696ed48034
> +
> +Signed-off-by: Vicente Olivert Riera <Vincent.Riera@imgtec.com>
> +
> +From 36fedb0c4b373272afc2d76a7c790d696ed48034 Mon Sep 17 00:00:00 2001
> +From: Emmanuele Bassi <ebassi@gnome.org>
> +Date: Mon, 10 Nov 2014 14:08:20 +0000
> +Subject: [PATCH] window: GtkPlug is conditionally supported on X11 only
> +
> +We cannot do a type check on it if we don't have the X11 backend
> +compiled in.
> +
> +https://bugzilla.gnome.org/show_bug.cgi?id=739885
> +---
> + gtk/gtkwindow.c | 9 ++++++++-
> + 1 files changed, 8 insertions(+), 1 deletions(-)
> +
> +diff --git a/gtk/gtkwindow.c b/gtk/gtkwindow.c
> +index 0d0fb88..6cb355b 100644
> +--- a/gtk/gtkwindow.c
> ++++ b/gtk/gtkwindow.c
> +@@ -1610,10 +1610,17 @@ gtk_window_constructed (GObject *object)
> + {
> + GtkWindow *window = GTK_WINDOW (object);
> + GtkWindowPrivate *priv = window->priv;
> ++ gboolean is_plug;
> +
> + G_OBJECT_CLASS (gtk_window_parent_class)->constructed (object);
> +
> +- if (priv->type == GTK_WINDOW_TOPLEVEL && !GTK_IS_PLUG (window))
> ++#ifdef GDK_WINDOWING_X11
> ++ is_plug = GTK_IS_PLUG (window);
> ++#else
> ++ is_plug = FALSE;
> ++#endif
> ++
> ++ if (priv->type == GTK_WINDOW_TOPLEVEL && !is_plug)
> + {
> + priv->multipress_gesture = gtk_gesture_multi_press_new (GTK_WIDGET (object));
> + gtk_gesture_single_set_button (GTK_GESTURE_SINGLE (priv->multipress_gesture), 0);
> +--
> +1.7.1
> +
> diff --git a/package/libgtk3/libgtk3.hash b/package/libgtk3/libgtk3.hash
> new file mode 100644
> index 0000000..dc5cf26
> --- /dev/null
> +++ b/package/libgtk3/libgtk3.hash
> @@ -0,0 +1,2 @@
> +# From http://ftp.acc.umu.se/pub/gnome/sources/gtk+/3.14/gtk+-3.14.5.sha256sum
> +sha256 ba70f5ccde6646c6d8aa5a6398794b7bcf23fc45af22580a215d258f392dbbe2 gtk+-3.14.5.tar.xz
> diff --git a/package/libgtk3/libgtk3.mk b/package/libgtk3/libgtk3.mk
> index 540f08d..b406c09 100644
> --- a/package/libgtk3/libgtk3.mk
> +++ b/package/libgtk3/libgtk3.mk
> @@ -5,7 +5,7 @@
> ################################################################################
>
> LIBGTK3_VERSION_MAJOR = 3.14
> -LIBGTK3_VERSION = $(LIBGTK3_VERSION_MAJOR).4
> +LIBGTK3_VERSION = $(LIBGTK3_VERSION_MAJOR).5
> LIBGTK3_SOURCE = gtk+-$(LIBGTK3_VERSION).tar.xz
> LIBGTK3_SITE = http://ftp.gnome.org/pub/gnome/sources/gtk+/$(LIBGTK3_VERSION_MAJOR)
> LIBGTK3_LICENSE = LGPLv2+
> --
> 1.7.1
>
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
| +33 223 225 172 `------------.-------: X AGAINST | \e/ There is no |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
'------------------------------^-------^------------------^--------------------'
next prev parent reply other threads:[~2014-11-11 11:44 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-11-11 11:15 [Buildroot] [PATCH v2 0/2] libgtk3: version bump Vicente Olivert Riera
2014-11-11 11:15 ` [Buildroot] [PATCH v2 1/2] libgtk3: rename patches to follow the new name structure Vicente Olivert Riera
2014-11-11 11:41 ` Yann E. MORIN
2014-11-11 11:15 ` [Buildroot] [PATCH v2 2/2] libgtk3: bump version to 3.14.5 Vicente Olivert Riera
2014-11-11 11:44 ` Yann E. MORIN [this message]
2014-11-11 14:39 ` [Buildroot] [PATCH v2 0/2] libgtk3: version bump Thomas Petazzoni
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=20141111114431.GE4240@free.fr \
--to=yann.morin.1998@free.fr \
--cc=buildroot@busybox.net \
/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