Openembedded Core Discussions
 help / color / mirror / Atom feed
* [PATCH] atk: make sure that introspection is enabled
@ 2018-04-06  4:35 Anuj Mittal
  2018-04-06  5:02 ` ✗ patchtest: failure for " Patchwork
  2018-04-06  7:50 ` [PATCH] " Alexander Kanavin
  0 siblings, 2 replies; 3+ messages in thread
From: Anuj Mittal @ 2018-04-06  4:35 UTC (permalink / raw)
  To: openembedded-core

Fix the meson flags to make sure that introspection files are built
when it is enabled.

Signed-off-by: Anuj Mittal <anuj.mittal@intel.com>
---
 ...ld-enable-introspection-for-cross-compile.patch | 28 ++++++++++++++++++++++
 meta/recipes-support/atk/atk_2.28.1.bb             |  3 +++
 2 files changed, 31 insertions(+)
 create mode 100644 meta/recipes-support/atk/atk/0001-meson.build-enable-introspection-for-cross-compile.patch

diff --git a/meta/recipes-support/atk/atk/0001-meson.build-enable-introspection-for-cross-compile.patch b/meta/recipes-support/atk/atk/0001-meson.build-enable-introspection-for-cross-compile.patch
new file mode 100644
index 0000000..c604a98
--- /dev/null
+++ b/meta/recipes-support/atk/atk/0001-meson.build-enable-introspection-for-cross-compile.patch
@@ -0,0 +1,28 @@
+From 3838757d29590cc1ef99c97f7268377322fc17e9 Mon Sep 17 00:00:00 2001
+From: Anuj Mittal <anuj.mittal@intel.com>
+Date: Fri, 6 Apr 2018 12:04:00 +0800
+Subject: [PATCH] meson.build: enable introspection for cross-compile
+
+It works fine in OE-core and doesn't need to be disabled. Let the user decide
+if it should be disabled or not.
+
+Upstream-Status: Pending
+
+Signed-off-by: Anuj Mittal <anuj.mittal@intel.com>
+---
+ atk/meson.build | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/atk/meson.build b/atk/meson.build
+index 7b5a683..855e28f 100644
+--- a/atk/meson.build
++++ b/atk/meson.build
+@@ -139,7 +139,7 @@ libatk_dep = declare_dependency(link_with: libatk,
+ 
+ disable_introspection = get_option('disable_introspection')
+ 
+-if not meson.is_cross_build() and not disable_introspection
++if not disable_introspection
+   gnome.generate_gir(libatk,
+                      sources: atk_sources + atk_headers + [ atk_enum_h ] + [ atk_version_h ],
+                      namespace: 'Atk',
diff --git a/meta/recipes-support/atk/atk_2.28.1.bb b/meta/recipes-support/atk/atk_2.28.1.bb
index bb47916..8107df9 100644
--- a/meta/recipes-support/atk/atk_2.28.1.bb
+++ b/meta/recipes-support/atk/atk_2.28.1.bb
@@ -13,6 +13,9 @@ DEPENDS = "glib-2.0"
 GNOMEBASEBUILDCLASS = "meson"
 inherit gnomebase gtk-doc gettext upstream-version-is-even gobject-introspection
 
+SRC_URI_append = " \
+                   file://0001-meson.build-enable-introspection-for-cross-compile.patch \
+                   "
 SRC_URI[archive.md5sum] = "dfb5e7474220afa3f4ca7e45af9f3a11"
 SRC_URI[archive.sha256sum] = "cd3a1ea6ecc268a2497f0cd018e970860de24a6d42086919d6bf6c8e8d53f4fc"
 
-- 
2.7.4



^ permalink raw reply related	[flat|nested] 3+ messages in thread

* ✗ patchtest: failure for atk: make sure that introspection is enabled
  2018-04-06  4:35 [PATCH] atk: make sure that introspection is enabled Anuj Mittal
@ 2018-04-06  5:02 ` Patchwork
  2018-04-06  7:50 ` [PATCH] " Alexander Kanavin
  1 sibling, 0 replies; 3+ messages in thread
From: Patchwork @ 2018-04-06  5:02 UTC (permalink / raw)
  To: Anuj Mittal; +Cc: openembedded-core

== Series Details ==

Series: atk: make sure that introspection is enabled
Revision: 1
URL   : https://patchwork.openembedded.org/series/11694/
State : failure

== Summary ==


Thank you for submitting this patch series to OpenEmbedded Core. This is
an automated response. Several tests have been executed on the proposed
series by patchtest resulting in the following failures:



* Issue             Series does not apply on top of target branch [test_series_merge_on_head] 
  Suggested fix    Rebase your series on top of targeted branch
  Targeted branch  master (currently at dc52f9cf34)



If you believe any of these test results are incorrect, please reply to the
mailing list (openembedded-core@lists.openembedded.org) raising your concerns.
Otherwise we would appreciate you correcting the issues and submitting a new
version of the patchset if applicable. Please ensure you add/increment the
version number when sending the new version (i.e. [PATCH] -> [PATCH v2] ->
[PATCH v3] -> ...).

---
Guidelines:     https://www.openembedded.org/wiki/Commit_Patch_Message_Guidelines
Test framework: http://git.yoctoproject.org/cgit/cgit.cgi/patchtest
Test suite:     http://git.yoctoproject.org/cgit/cgit.cgi/patchtest-oe



^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] atk: make sure that introspection is enabled
  2018-04-06  4:35 [PATCH] atk: make sure that introspection is enabled Anuj Mittal
  2018-04-06  5:02 ` ✗ patchtest: failure for " Patchwork
@ 2018-04-06  7:50 ` Alexander Kanavin
  1 sibling, 0 replies; 3+ messages in thread
From: Alexander Kanavin @ 2018-04-06  7:50 UTC (permalink / raw)
  To: Anuj Mittal, openembedded-core

On 04/06/2018 07:35 AM, Anuj Mittal wrote:
> Fix the meson flags to make sure that introspection files are built
> when it is enabled.

This does fix the gtk/gtk3 issues observed on AB in master-next, thanks!

Alex


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2018-04-06  7:56 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-04-06  4:35 [PATCH] atk: make sure that introspection is enabled Anuj Mittal
2018-04-06  5:02 ` ✗ patchtest: failure for " Patchwork
2018-04-06  7:50 ` [PATCH] " Alexander Kanavin

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox