Openembedded Core Discussions
 help / color / mirror / Atom feed
From: Markus Volk <f_l_k@t-online.de>
To: alexandre.belloni@bootlin.com
Cc: openembedded-core@lists.openembedded.org
Subject: Re: [oe-core][PATCH] gcr3: update 3.40.0 -> 3.41.1
Date: Thu, 05 Jan 2023 16:42:13 +0100	[thread overview]
Message-ID: <DMR0OR.Q1D80VIV2M3N1@t-online.de> (raw)
In-Reply-To: <Y7YD1ZgSe7b+UIBt@mail.local>

[-- Attachment #1: Type: text/plain, Size: 12390 bytes --]

I can't see much in the log. I'll take a closer look when I get a 
chance, but so far I can't reproduce the problem. But I think there is 
no pressure for the update

Am Mi, 4. Jan 2023 um 23:55:17 +0100 schrieb Alexandre Belloni via 
lists.openembedded.org 
<alexandre.belloni=bootlin.com@lists.openembedded.org>:
> This failed on the AB:
> 
> <https://autobuilder.yoctoproject.org/typhoon/#/builders/108/builds/3976/steps/11/logs/stdio>
> 
> On 01/01/2023 15:24:14+0100, Markus Volk wrote:
>>   - remove unneded backport patches
>>   - partly import the patch to fix ssh_agent option from gcr4 recipe
>> 
>>  gcr 3.41.1:
>>  - meson: Fix unknown kw argument in gnome.generate_gir 
>> [GNOME/gcr!68]
>>  - gcr: Add G_SPAWN_CLOEXEC_PIPES flag to all the g_spawn commands
>>  - docs: Port from gtk-doc to gi-docgen [GNOME/gcr!76]
>>  - Unbreak build without systemd [GNOME/gcr!75]
>>  - Several CI fixes
>>  - Updated translations
>> 
>>  gcr 3.41.0:
>>  - Port ssh-agent from gnome-keyring [GNOME/gcr!67]
>>  - build: Fix parallel build failure due to missing marshal 
>> dependency [GNOME/gcr!68]
>>  - Fix warnings by dropping `volatile` for g_once_init_inter 
>> locations [GNOME/gcr!69]
>>  - tests: More robust against GTask unref race condition 
>> [GNOME/gcr!72,GNOME/gcr#84]
>>  - Updated translations
>> 
>>  Signed-off-by: Markus Volk <f_l_k@t-online.de 
>> <mailto:f_l_k@t-online.de>>
>>  ---
>>   ...build-fix-one-parallel-build-failure.patch | 37 -----------
>>   ...ectly-handle-disabled-ssh_agent-opti.patch | 36 +++++++++++
>>   ...1d02bb0148ca787ac4aead164d7c8ce2c4d8.patch | 61 
>> -------------------
>>   .../gcr/{gcr3_3.40.0.bb => gcr3_3.41.1.bb}    | 17 ++++--
>>   4 files changed, 47 insertions(+), 104 deletions(-)
>>   delete mode 100644 
>> meta/recipes-gnome/gcr/gcr3/0001-gcr-meson.build-fix-one-parallel-build-failure.patch
>>   create mode 100644 
>> meta/recipes-gnome/gcr/gcr3/0001-meson.build-correctly-handle-disabled-ssh_agent-opti.patch
>>   delete mode 100644 
>> meta/recipes-gnome/gcr/gcr3/b3ca1d02bb0148ca787ac4aead164d7c8ce2c4d8.patch
>>   rename meta/recipes-gnome/gcr/{gcr3_3.40.0.bb => gcr3_3.41.1.bb} 
>> (69%)
>> 
>>  diff --git 
>> a/meta/recipes-gnome/gcr/gcr3/0001-gcr-meson.build-fix-one-parallel-build-failure.patch 
>> b/meta/recipes-gnome/gcr/gcr3/0001-gcr-meson.build-fix-one-parallel-build-failure.patch
>>  deleted file mode 100644
>>  index 81c4bdbbcc..0000000000
>>  --- 
>> a/meta/recipes-gnome/gcr/gcr3/0001-gcr-meson.build-fix-one-parallel-build-failure.patch
>>  +++ /dev/null
>>  @@ -1,37 +0,0 @@
>>  -From cb3708bad88e713e4ccf705cb8c14c5996cd9d06 Mon Sep 17 00:00:00 
>> 2001
>>  -From: Changqing Li <changqing.li@windriver.com 
>> <mailto:changqing.li@windriver.com>>
>>  -Date: Fri, 23 Apr 2021 16:32:38 +0800
>>  -Subject: [PATCH] gcr/meson.build: fix one parallel build failure
>>  -
>>  -ui/gcr-live-search.c includes gcr/gcr-marshal.h. Because missing
>>  -dependency, following error occurred intermittently during doing 
>> parallel
>>  -build:
>>  -
>>  - -o ui/libgcr-ui-3.so.1.0.0.p/gcr-live-search.c.o -c 
>> ../gcr-3.38.1/ui/gcr-live-search.c
>>  -../gcr-3.38.1/ui/gcr-live-search.c:32:10: fatal error: 
>> gcr/gcr-marshal.h: No such file or directory
>>  -   32 | #include "gcr/gcr-marshal.h"
>>  -      |          ^~~~~~~~~~~~~~~~~~~
>>  -compilation terminated.
>>  -
>>  -Upstream-Status: Submitted 
>> [<https://gitlab.gnome.org/GNOME/gcr/-/merge_requests/68/diffs>]
>>  -
>>  -Signed-off-by: Changqing Li <changqing.li@windriver.com 
>> <mailto:changqing.li@windriver.com>>
>>  ----
>>  - gcr/meson.build | 1 +
>>  - 1 file changed, 1 insertion(+)
>>  -
>>  -diff --git a/gcr/meson.build b/gcr/meson.build
>>  -index 06c3a63..d9d4b8f 100644
>>  ---- a/gcr/meson.build
>>  -+++ b/gcr/meson.build
>>  -@@ -181,6 +181,7 @@ gcr_base_dep = declare_dependency(
>>  -   sources: [
>>  -     gcr_enums_gen[1],
>>  -     gcr_oids[1],
>>  -+    gcr_marshal_gen[1],
>>  -   ],
>>  - )
>>  -
>>  ---
>>  -2.17.1
>>  -
>>  diff --git 
>> a/meta/recipes-gnome/gcr/gcr3/0001-meson.build-correctly-handle-disabled-ssh_agent-opti.patch 
>> b/meta/recipes-gnome/gcr/gcr3/0001-meson.build-correctly-handle-disabled-ssh_agent-opti.patch
>>  new file mode 100644
>>  index 0000000000..d3c26d97ae
>>  --- /dev/null
>>  +++ 
>> b/meta/recipes-gnome/gcr/gcr3/0001-meson.build-correctly-handle-disabled-ssh_agent-opti.patch
>>  @@ -0,0 +1,36 @@
>>  +From 9b67bb18d8409e0e693cc6000507acbd73a30eab Mon Sep 17 00:00:00 
>> 2001
>>  +From: Alexander Kanavin <alex@linutronix.de 
>> <mailto:alex@linutronix.de>>
>>  +Date: Wed, 16 Nov 2022 11:27:24 +0100
>>  +Subject: [PATCH 1/2] meson.build: correctly handle disabled 
>> ssh_agent option
>>  +
>>  +Existing code produces these errors:
>>  +| gcr/meson.build:61:0: ERROR: Unable to get the path of a 
>> not-found external program
>>  +| gcr/meson.build:101:5: ERROR: Unknown variable "ssh_add_path".
>>  +
>>  +Signed-off-by: Alexander Kanavin <alex@linutronix.de 
>> <mailto:alex@linutronix.de>>
>>  +
>>  +Upstream-Status: Backport [ 
>> <https://gitlab.gnome.org/GNOME/gcr/-/commit/9b67bb18d8409e0e693cc6000507acbd73a30eab> 
>> ]
>>  +---
>>  + meson.build | 12 ++++++++----
>>  + 1 file changed, 8 insertions(+), 4 deletions(-)
>>  +
>>  +diff --git a/meson.build b/meson.build
>>  +index 3f35173..11d7fa7 100644
>>  +--- a/meson.build
>>  ++++ b/meson.build
>>  +@@ -96,8 +98,10 @@ conf.set('HAVE_TIMEGM', 
>> cc.has_function('timegm'))
>>  + conf.set('HAVE_MLOCK', cc.has_function('mlock'))
>>  + conf.set_quoted('GPG_EXECUTABLE', gpg_path)
>>  + conf.set_quoted('LIBGCRYPT_VERSION', libgcrypt_dep.version())
>>  +-conf.set_quoted('SSH_ADD_EXECUTABLE', ssh_add_path)
>>  +-conf.set_quoted('SSH_AGENT_EXECUTABLE', ssh_agent_path)
>>  ++if get_option('ssh_agent')
>>  ++  conf.set_quoted('SSH_ADD_EXECUTABLE', ssh_add_path)
>>  ++  conf.set_quoted('SSH_AGENT_EXECUTABLE', ssh_agent_path)
>>  ++endif
>>  + conf.set10('WITH_SYSTEMD', with_systemd)
>>  + config_file = configure_file(
>>  +   output: 'config.h',
>>  +--
>>  +2.34.1
>>  +
>>  diff --git 
>> a/meta/recipes-gnome/gcr/gcr3/b3ca1d02bb0148ca787ac4aead164d7c8ce2c4d8.patch 
>> b/meta/recipes-gnome/gcr/gcr3/b3ca1d02bb0148ca787ac4aead164d7c8ce2c4d8.patch
>>  deleted file mode 100644
>>  index ae4e2ac1eb..0000000000
>>  --- 
>> a/meta/recipes-gnome/gcr/gcr3/b3ca1d02bb0148ca787ac4aead164d7c8ce2c4d8.patch
>>  +++ /dev/null
>>  @@ -1,61 +0,0 @@
>>  -From b3ca1d02bb0148ca787ac4aead164d7c8ce2c4d8 Mon Sep 17 00:00:00 
>> 2001
>>  -From: Jakub Jirutka <jakub@jirutka.cz <mailto:jakub@jirutka.cz>>
>>  -Date: Wed, 12 Jan 2022 00:24:20 +0100
>>  -Subject: [PATCH] meson: Fix unknown kw argument in 
>> gnome.generate_gir
>>  -
>>  -This argument has been removed in Meson 0.61.0:
>>  -
>>  -    gck/meson.build:130:2: ERROR: gnome.generate_gir got unknown 
>> keyword arguments "packages"
>>  -
>>  
>> -<https://github.com/mesonbuild/meson/commit/f8fc5cb860465718fe7c79a1bf1fe00659f138de>:
>>  -
>>  -> The packages argument to gnome.generate_gir was allowed, but 
>> never did anything, so stop passing it.
>>  -
>>  -Fixes #89
>>  -Upstream-Status: Backport
>>  -Signed-off-by: Alexander Kanavin <alex.kanavin@gmail.com 
>> <mailto:alex.kanavin@gmail.com>>
>>  ----
>>  - gck/meson.build | 1 -
>>  - gcr/meson.build | 1 -
>>  - ui/meson.build  | 1 -
>>  - 3 files changed, 3 deletions(-)
>>  -
>>  -diff --git a/gck/meson.build b/gck/meson.build
>>  -index 756b486..a21a1e9 100644
>>  ---- a/gck/meson.build
>>  -+++ b/gck/meson.build
>>  -@@ -131,7 +131,6 @@ if get_option('introspection')
>>  -     sources: gck_gir_sources,
>>  -     namespace: 'Gck',
>>  -     nsversion: '@0@'.format(gck_major_version),
>>  --    packages: gck_deps,
>>  -     export_packages: 'gck-@0@'.format(gck_major_version),
>>  -     includes: [ 'GObject-2.0', 'Gio-2.0' ],
>>  -     header: 'gck/gck.h',
>>  -diff --git a/gcr/meson.build b/gcr/meson.build
>>  -index 2233a44..c83641b 100644
>>  ---- a/gcr/meson.build
>>  -+++ b/gcr/meson.build
>>  -@@ -190,7 +190,6 @@ if get_option('introspection')
>>  -     sources: [ gcr_base_public_sources, gcr_base_headers ],
>>  -     namespace: 'Gcr',
>>  -     nsversion: '@0@'.format(gcr_major_version),
>>  --    packages: gcr_base_deps,
>>  -     export_packages: 'gcr-base-@0@'.format(gcr_major_version),
>>  -     includes: [
>>  -       'GObject-2.0',
>>  -diff --git a/ui/meson.build b/ui/meson.build
>>  -index e656ea2..32ee057 100644
>>  ---- a/ui/meson.build
>>  -+++ b/ui/meson.build
>>  -@@ -152,7 +152,6 @@ if get_option('introspection')
>>  -     export_packages: 'gcr-ui-@0@'.format(gcr_major_version),
>>  -     identifier_prefix: 'Gcr',
>>  -     symbol_prefix: 'gcr',
>>  --    packages: gcr_ui_deps,
>>  -     includes: [
>>  -       'GObject-2.0',
>>  -       'Gio-2.0',
>>  ---
>>  -GitLab
>>  -
>>  diff --git a/meta/recipes-gnome/gcr/gcr3_3.40.0.bb 
>> b/meta/recipes-gnome/gcr/gcr3_3.41.1.bb
>>  similarity index 69%
>>  rename from meta/recipes-gnome/gcr/gcr3_3.40.0.bb
>>  rename to meta/recipes-gnome/gcr/gcr3_3.41.1.bb
>>  index 5318768db6..273728f6c8 100644
>>  --- a/meta/recipes-gnome/gcr/gcr3_3.40.0.bb
>>  +++ b/meta/recipes-gnome/gcr/gcr3_3.41.1.bb
>>  @@ -20,24 +20,27 @@ GTKDOC_MESON_OPTION = "gtk_doc"
>>   inherit gnomebase gtk-icon-cache gtk-doc features_check 
>> upstream-version-is-even vala gobject-introspection gettext mime 
>> mime-xdg
>>   UPSTREAM_CHECK_REGEX = 
>> "[^\d\.](?P<pver>\d+\.(?!9\d+)(\d*[02468])+(\.\d+)+)\.tar"
>> 
>>  -SRC_URI = " \
>>  
>> -	https://download.gnome.org/sources/gcr/3.40/gcr-${PV}.tar.xz;name=archive 
>> <https://download.gnome.org/sources/gcr/3.40/gcr-$%7BPV%7D.tar.xz;name=archive> 
>> \
>>  -	file://0001-gcr-meson.build-fix-one-parallel-build-failure.patch 
>> <file://0001-gcr-meson.build-fix-one-parallel-build-failure.patch/> \
>>  -	file://b3ca1d02bb0148ca787ac4aead164d7c8ce2c4d8.patch" 
>> <file://b3ca1d02bb0148ca787ac4aead164d7c8ce2c4d8.patch/>
>>  -
>>  -SRC_URI[archive.sha256sum] = 
>> "b9d3645a5fd953a54285cc64d4fc046736463dbd4dcc25caf5c7b59bed3027f5"
>>  +SRC_URI = 
>> "https://download.gnome.org/sources/gcr/3.41/gcr-${PV}.tar.xz;name=archive 
>> <https://download.gnome.org/sources/gcr/3.41/gcr-$%7BPV%7D.tar.xz;name=archive>"
>>  +SRC_URI += 
>> "file://0001-meson.build-correctly-handle-disabled-ssh_agent-opti.patch" 
>> <file://0001-meson.build-correctly-handle-disabled-ssh_agent-opti.patch/>
>>  +SRC_URI[archive.sha256sum] = 
>> "bb7128a3c2febbfee9c03b90d77d498d0ceb237b0789802d60185c71c4bea24f"
>> 
>>   S = "${WORKDIR}/gcr-${PV}"
>> 
>>   PACKAGECONFIG ??= " \
>>  +	${@bb.utils.filter <mailto:${@bb.utils.filter>('DISTRO_FEATURES', 
>> 'systemd', d)} \
>>   	${@bb.utils.contains 
>> <mailto:${@bb.utils.contains>('DISTRO_FEATURES', 'x11', 'gtk', '', 
>> d)} \
>>   	${@bb.utils.contains 
>> <mailto:${@bb.utils.contains>('DISTRO_FEATURES', 'wayland', 'gtk', 
>> '', d)} \
>>   "
>>   PACKAGECONFIG[gtk] = "-Dgtk=true,-Dgtk=false,gtk+3"
>>  +PACKAGECONFIG[ssh_agent] = 
>> "-Dssh_agent=true,-Dssh_agent=false,libsecret,openssh"
>>  +#'Use systemd socket activation for server programs'
>>  +PACKAGECONFIG[systemd] = 
>> "-Dsystemd=enabled,-Dsystemd=disabled,systemd"
>> 
>>   FILES:${PN} += " \
>>       ${datadir}/dbus-1 \
>>       ${datadir}/gcr-3 \
>>  +    ${systemd_user_unitdir}/gcr-ssh-agent.socket \
>>  +    ${systemd_user_unitdir}/gcr-ssh-agent.service \
>>   "
>> 
>>   # <http://errors.yoctoproject.org/Errors/Details/20229/>
>>  @@ -50,5 +53,7 @@ do_write_config:append() {
>>       cat >${WORKDIR}/meson-${PN}.cross <<EOF
>>   [binaries]
>>   gpg2 = '${bindir}/gpg2'
>>  +ssh-add = '${bindir}/ssh-add'
>>  +ssh-agent = '${bindir}/ssh-agent'
>>   EOF
>>   }
>>  --
>>  2.34.1
>> 
> 
>> 
>> 
>> 
> 
> 
> --
> Alexandre Belloni, co-owner and COO, Bootlin
> Embedded Linux and Kernel engineering
> https://bootlin.com <https://bootlin.com/>
> 
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#175511): 
> <https://lists.openembedded.org/g/openembedded-core/message/175511>
> Mute This Topic: <https://lists.openembedded.org/mt/95988912/3618223>
> Group Owner: openembedded-core+owner@lists.openembedded.org 
> <mailto:openembedded-core+owner@lists.openembedded.org>
> Unsubscribe: 
> <https://lists.openembedded.org/g/openembedded-core/unsub> 
> [f_l_k@t-online.de <mailto:f_l_k@t-online.de>]
> -=-=-=-=-=-=-=-=-=-=-=-
> 


[-- Attachment #2: Type: text/html, Size: 12796 bytes --]

  reply	other threads:[~2023-01-05 15:42 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-01-01 14:24 [oe-core][PATCH] gcr3: update 3.40.0 -> 3.41.1 Markus Volk
2023-01-04 22:55 ` Alexandre Belloni
2023-01-05 15:42   ` Markus Volk [this message]
2023-01-05 15:46     ` Alexander Kanavin
2023-01-05 15:48       ` Markus Volk
2023-01-05 17:59         ` Alexandre Belloni
2023-01-05 18:04           ` Markus Volk

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=DMR0OR.Q1D80VIV2M3N1@t-online.de \
    --to=f_l_k@t-online.de \
    --cc=alexandre.belloni@bootlin.com \
    --cc=openembedded-core@lists.openembedded.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