[oe] [PATCH 2/6] glade3: remove ancient version of glade

Khem Raj raj.khem at gmail.com
Mon Dec 16 02:40:13 UTC 2019


On Sun, Dec 15, 2019 at 5:21 PM Andreas Müller <schnitzeltony at gmail.com>
wrote:

> * This was suggested already [1] but I complained that there is a use-case.
>   There are several workarounds possible for this so we don't need to keep
>   it anymore.
> * glade3 popped into focus during preparations for python2 removal. So
> instead
>   of wasting time to keep unmaintained version just drop it
>
> [1]
> http://lists.openembedded.org/pipermail/openembedded-devel/2019-May/199523.html
>

I see forget my comment on previous patch then

>
> Signed-off-by: Andreas Müller <schnitzeltony at gmail.com>
> ---
>  ...NTF-on-functions-with-format-strings.patch | 62 -------------------
>  ...-doc-utils.make-sysrooted-pkg-config.patch | 51 ---------------
>  .../glade/glade3/0002-fix-gcc-6-build.patch   | 39 ------------
>  .../recipes-devtools/glade/glade3_3.8.5.bb    | 23 -------
>  4 files changed, 175 deletions(-)
>  delete mode 100644
> meta-xfce/recipes-devtools/glade/glade3/0001-Add-G_GNUC_PRINTF-on-functions-with-format-strings.patch
>  delete mode 100644
> meta-xfce/recipes-devtools/glade/glade3/0001-gnome-doc-utils.make-sysrooted-pkg-config.patch
>  delete mode 100644
> meta-xfce/recipes-devtools/glade/glade3/0002-fix-gcc-6-build.patch
>  delete mode 100644 meta-xfce/recipes-devtools/glade/glade3_3.8.5.bb
>
> diff --git
> a/meta-xfce/recipes-devtools/glade/glade3/0001-Add-G_GNUC_PRINTF-on-functions-with-format-strings.patch
> b/meta-xfce/recipes-devtools/glade/glade3/0001-Add-G_GNUC_PRINTF-on-functions-with-format-strings.patch
> deleted file mode 100644
> index 03cb762ce..000000000
> ---
> a/meta-xfce/recipes-devtools/glade/glade3/0001-Add-G_GNUC_PRINTF-on-functions-with-format-strings.patch
> +++ /dev/null
> @@ -1,62 +0,0 @@
> -From fcf29abe59607b5791f9de18ddb86b9ae3c9b7cc Mon Sep 17 00:00:00 2001
> -From: Khem Raj <raj.khem at gmail.com>
> -Date: Sat, 26 Aug 2017 23:50:05 -0700
> -Subject: [PATCH] Add G_GNUC_PRINTF on functions with format strings
> -
> -This allows compilation with clang without errors, even when
> --Wformat-nonliteral is active (as long as there are no real cases of
> -non literal formatting).
> -
> -Signed-off-by: Khem Raj <raj.khem at gmail.com>
> ----
> - gladeui/glade-command.c | 4 ++--
> - gladeui/glade-utils.c   | 4 ++--
> - 2 files changed, 4 insertions(+), 4 deletions(-)
> -
> -diff --git a/gladeui/glade-command.c b/gladeui/glade-command.c
> -index 9819766..4ac40ee 100644
> ---- a/gladeui/glade-command.c
> -+++ b/gladeui/glade-command.c
> -@@ -266,7 +266,7 @@ glade_command_collapse (GladeCommand  *command,
> -  *
> -  * Marks the begining of a group.
> -  */
> --void
> -+G_GNUC_PRINTF(1, 2) void
> - glade_command_push_group (const gchar *fmt, ...)
> - {
> -       va_list         args;
> -@@ -655,7 +655,7 @@ glade_command_set_properties_list (GladeProject
> *project, GList *props)
> -
> -       multiple = g_list_length (me->sdata) > 1;
> -       if (multiple)
> --              glade_command_push_group (cmd->description);
> -+              glade_command_push_group ("%s", cmd->description);
> -
> -
> -       glade_command_check_group (GLADE_COMMAND (me));
> -diff --git a/gladeui/glade-utils.c b/gladeui/glade-utils.c
> -index ae52501..c51ae59 100644
> ---- a/gladeui/glade-utils.c
> -+++ b/gladeui/glade-utils.c
> -@@ -197,7 +197,7 @@ glade_utils_get_pspec_from_funcname (const gchar
> *funcname)
> -  *          selected "OK", True if the @type was GLADE_UI_YES_OR_NO and
> -  *          the user selected "YES"; False otherwise.
> -  */
> --gint
> -+G_GNUC_PRINTF(4, 5) gint
> - glade_util_ui_message (GtkWidget           *parent,
> -                      GladeUIMessageType   type,
> -                      GtkWidget           *widget,
> -@@ -320,7 +320,7 @@ remove_message_timeout (FlashInfo * fi)
> -  *
> -  * Flash a temporary message on the statusbar.
> -  */
> --void
> -+G_GNUC_PRINTF(3, 4) void
> - glade_util_flash_message (GtkWidget *statusbar, guint context_id, gchar
> *format, ...)
> - {
> -       va_list args;
> ---
> -2.14.1
> -
> diff --git
> a/meta-xfce/recipes-devtools/glade/glade3/0001-gnome-doc-utils.make-sysrooted-pkg-config.patch
> b/meta-xfce/recipes-devtools/glade/glade3/0001-gnome-doc-utils.make-sysrooted-pkg-config.patch
> deleted file mode 100644
> index 1b24c39ce..000000000
> ---
> a/meta-xfce/recipes-devtools/glade/glade3/0001-gnome-doc-utils.make-sysrooted-pkg-config.patch
> +++ /dev/null
> @@ -1,51 +0,0 @@
> -From aed002cd9ff9e8f972120fbac33b4a65aba952e1 Mon Sep 17 00:00:00 2001
> -From: =?UTF-8?q?Andreas=20M=C3=BCller?= <schnitzeltony at googlemail.com>
> -Date: Tue, 25 Sep 2012 10:28:33 +0200
> -Subject: [PATCH] gnome-doc-utils.make: sysrooted pkg-config
> -MIME-Version: 1.0
> -Content-Type: text/plain; charset=UTF-8
> -Content-Transfer-Encoding: 8bit
> -
> -same approach as used used in gnome-disk-utility:
> -
> -In cross environment we have to prepend the sysroot to the path found by
> -pkgconfig since the path returned from pkgconfig does not have sysroot
> prefixed
> -it ends up using the files from host system. Now usually people have
> gnome installed
> -so the build succeeds but if you dont have gnome installed on build host
> then
> -it wont find the files on host system and packages using gnome-doc-utils
> wont
> -compile.
> -
> -This should work ok with non sysrooted builds too since in those cases
> PKG_CONFIG_SYSROOT_DIR
> -will be empty
> -
> -Upstream-Status: pending
> -
> -Signed-off-by: Andreas Müller <schnitzeltony at googlemail.com>
> ----
> - gnome-doc-utils.make |   10 +++++-----
> - 1 files changed, 5 insertions(+), 5 deletions(-)
> -
> -diff --git a/gnome-doc-utils.make b/gnome-doc-utils.make
> -index 42d9df3..f71bbfa 100644
> ---- a/gnome-doc-utils.make
> -+++ b/gnome-doc-utils.make
> -@@ -133,11 +133,11 @@ _DOC_ABS_SRCDIR = @abs_srcdir@
> - _xml2po ?= `which xml2po`
> - _xml2po_mode = $(if $(DOC_ID),mallard,docbook)
> -
> --_db2html ?= `$(PKG_CONFIG) --variable db2html gnome-doc-utils`
> --_db2omf  ?= `$(PKG_CONFIG) --variable db2omf gnome-doc-utils`
> --_chunks  ?= `$(PKG_CONFIG) --variable xmldir
> gnome-doc-utils`/gnome/xslt/docbook/utils/chunks.xsl
> --_credits ?= `$(PKG_CONFIG) --variable xmldir
> gnome-doc-utils`/gnome/xslt/docbook/utils/credits.xsl
> --_ids ?= $(shell $(PKG_CONFIG) --variable xmldir
> gnome-doc-utils)/gnome/xslt/docbook/utils/ids.xsl
> -+_db2html ?= ${PKG_CONFIG_SYSROOT_DIR}`$(PKG_CONFIG) --variable db2html
> gnome-doc-utils`
> -+_db2omf  ?= ${PKG_CONFIG_SYSROOT_DIR}`$(PKG_CONFIG) --variable db2omf
> gnome-doc-utils`
> -+_chunks  ?= ${PKG_CONFIG_SYSROOT_DIR}`$(PKG_CONFIG) --variable xmldir
> gnome-doc-utils`/gnome/xslt/docbook/utils/chunks.xsl
> -+_credits ?= ${PKG_CONFIG_SYSROOT_DIR}`$(PKG_CONFIG) --variable xmldir
> gnome-doc-utils`/gnome/xslt/docbook/utils/credits.xsl
> -+_ids ?= ${PKG_CONFIG_SYSROOT_DIR}`$(PKG_CONFIG) --variable xmldir
> gnome-doc-utils`/gnome/xslt/docbook/utils/ids.xsl
> -
> - if ENABLE_SK
> - _ENABLE_SK = true
> ---
> -1.7.6.5
> -
> diff --git
> a/meta-xfce/recipes-devtools/glade/glade3/0002-fix-gcc-6-build.patch
> b/meta-xfce/recipes-devtools/glade/glade3/0002-fix-gcc-6-build.patch
> deleted file mode 100644
> index f735ff819..000000000
> --- a/meta-xfce/recipes-devtools/glade/glade3/0002-fix-gcc-6-build.patch
> +++ /dev/null
> @@ -1,39 +0,0 @@
> -From 5aa3d2abb905fa8594f6c6572a87809da54c9342 Mon Sep 17 00:00:00 2001
> -From: =?UTF-8?q?Andreas=20M=C3=BCller?= <schnitzeltony at googlemail.com>
> -Date: Mon, 5 Sep 2016 11:25:27 +0200
> -Subject: [PATCH] fix gcc-6 build
> -MIME-Version: 1.0
> -Content-Type: text/plain; charset=UTF-8
> -Content-Transfer-Encoding: 8bit
> -
> -Upstream-Status: Pending
> -
> -Signed-off-by: Andreas Müller <schnitzeltony at googlemail.com>
> ----
> - gladeui/glade-editor-property.c | 3 +++
> - 1 file changed, 3 insertions(+)
> -
> -diff --git a/gladeui/glade-editor-property.c
> b/gladeui/glade-editor-property.c
> -index a0c1039..5e9ac38 100644
> ---- a/gladeui/glade-editor-property.c
> -+++ b/gladeui/glade-editor-property.c
> -@@ -2703,6 +2703,8 @@ glade_eprop_object_view (gboolean             radio)
> - }
> -
> -
> -+#pragma GCC diagnostic push
> -+#pragma GCC diagnostic ignored "-Wformat-nonliteral"
> - static gchar *
> - glade_eprop_object_dialog_title (GladeEditorProperty *eprop)
> - {
> -@@ -2731,6 +2733,7 @@ glade_eprop_object_dialog_title
> (GladeEditorProperty *eprop)
> -       return g_strdup_printf (format, g_type_name
> -                               (eprop->klass->pspec->value_type));
> - }
> -+#pragma GCC diagnostic pop
> -
> -
> - gboolean
> ---
> -2.5.5
> -
> diff --git a/meta-xfce/recipes-devtools/glade/glade3_3.8.5.bb
> b/meta-xfce/recipes-devtools/glade/glade3_3.8.5.bb
> deleted file mode 100644
> index 24d0d901f..000000000
> --- a/meta-xfce/recipes-devtools/glade/glade3_3.8.5.bb
> +++ /dev/null
> @@ -1,23 +0,0 @@
> -SUMMARY = "Glade - A User Interface Designer"
> -HOMEPAGE = "http://www.gnu.org/software/gnash"
> -LICENSE = "GPLv2 & LGPLv2"
> -LIC_FILES_CHKSUM = "file://COPYING;md5=aabe87591cb8ae0f3c68be6977bb5522 \
> -
> file://COPYING.GPL;md5=9ac2e7cff1ddaf48b6eab6028f23ef88 \
> -
> file://COPYING.LGPL;md5=252890d9eee26aab7b432e8b8a616475"
> -DEPENDS = "gtk+ gnome-doc-utils gnome-common libxml2 intltool-native"
> -
> -inherit features_check autotools pkgconfig pythonnative gtk-icon-cache
> -
> -REQUIRED_DISTRO_FEATURES = "x11"
> -
> -SRC_URI = "
> http://ftp.gnome.org/pub/GNOME/sources/glade3/3.8/glade3-${PV}.tar.xz \
> -           file://0001-gnome-doc-utils.make-sysrooted-pkg-config.patch \
> -           file://0002-fix-gcc-6-build.patch \
> -
>  file://0001-Add-G_GNUC_PRINTF-on-functions-with-format-strings.patch \
> -           "
> -SRC_URI[md5sum] = "4e4b4f5ee34a03e017e4cef97d796c1f"
> -SRC_URI[sha256sum] =
> "58a5f6e4df4028230ddecc74c564808b7ec4471b1925058e29304f778b6b2735"
> -
> -EXTRA_OECONF += "--disable-scrollkeeper"
> -
> -FILES_${PN} += "${datadir}/icons"
> --
> 2.21.0
>
> --
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel at lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-devel
>


More information about the Openembedded-devel mailing list