All of lore.kernel.org
 help / color / mirror / Atom feed
From: Adrian Perez de Castro <aperez@igalia.com>
To: Thomas Devoogdt <thomas@devoogdt.com>
Cc: Joseph Kogut <joseph.kogut@gmail.com>,
	Daniel Lang <d.lang@abatec.at>,
	Thomas Devoogdt <thomas.devoogdt@barco.com>,
	James Knight <james.knight@collins.com>,
	buildroot@buildroot.org,
	Fabrice Fontaine <fontaine.fabrice@gmail.com>
Subject: Re: [Buildroot] [PATCH v7 2/5] package/at-spi2-atk: remove package
Date: Fri, 5 May 2023 14:56:33 +0300	[thread overview]
Message-ID: <20230505145633.GD3311548@igalia.com> (raw)
In-Reply-To: <20230428075920.3750005-2-thomas@devoogdt.com>


[-- Attachment #1.1: Type: text/plain, Size: 7278 bytes --]

On Fri, 28 Apr 2023 09:59:17 +0200 Thomas Devoogdt <thomas@devoogdt.com> wrote:
> From: Daniel Lang <d.lang@abatec.at>
> 
> at-spi2-atk has been merged into at-spi2-core with version 2.45.1.
> 
> https://gitlab.gnome.org/GNOME/at-spi2-core/-/blob/AT_SPI2_CORE_2_45_1/NEWS
> https://gitlab.gnome.org/GNOME/at-spi2-core/-/merge_requests/78
> 
> Signed-off-by: Daniel Lang <d.lang@abatec.at>
> Signed-off-by: Thomas Devoogdt <thomas.devoogdt@barco.com>
> Tested-by: Adrian Perez de Castro <aperez@igalia.com>

Reviewed-by: Adrian Perez de Castro <aperez@igalia.com>

> ---
> v3:
>  - rebased on https://patchwork.ozlabs.org/project/buildroot/list/?series=336254
>  - needed to fix the webkitgtk 2.38.5 compilation
>    (https://patchwork.ozlabs.org/project/buildroot/patch/20230220090923.2155635-2-thomas.devoogdt@barco.com/)
> v5: rebased + grouped patches
> v6: no change
> v7: no change
> ---
>  Config.in.legacy                              |  7 +++
>  DEVELOPERS                                    |  1 -
>  package/Config.in                             |  1 -
>  .../0001-meson-add-tests-option.patch         | 43 -------------------
>  package/at-spi2-atk/Config.in                 | 22 ----------
>  package/at-spi2-atk/at-spi2-atk.hash          |  5 ---
>  package/at-spi2-atk/at-spi2-atk.mk            | 19 --------
>  7 files changed, 7 insertions(+), 91 deletions(-)
>  delete mode 100644 package/at-spi2-atk/0001-meson-add-tests-option.patch
>  delete mode 100644 package/at-spi2-atk/Config.in
>  delete mode 100644 package/at-spi2-atk/at-spi2-atk.hash
>  delete mode 100644 package/at-spi2-atk/at-spi2-atk.mk
> 
> diff --git a/Config.in.legacy b/Config.in.legacy
> index 366b9d6949..db75899a9b 100644
> --- a/Config.in.legacy
> +++ b/Config.in.legacy
> @@ -146,6 +146,13 @@ endif
>  
>  comment "Legacy options removed in 2023.05"
>  
> +config BR2_PACKAGE_AT_SPI2_ATK
> +	bool "at-spi2-atk removed"
> +	select BR2_LEGACY
> +	select BR2_PACKAGE_AT_SPI2_CORE
> +	help
> +	  at-spi2-atk is now part of at-spi2-core.
> +
>  config BR2_PACAKGE_OPENFPGALOADER_CMSIS
>  	bool "openfpgaloader cmsis option name fixed"
>  	select BR2_LEGACY
> diff --git a/DEVELOPERS b/DEVELOPERS
> index d9e95a3d05..9206659428 100644
> --- a/DEVELOPERS
> +++ b/DEVELOPERS
> @@ -1632,7 +1632,6 @@ F:	support/testing/tests/package/test_avocado.py
>  F:	support/testing/tests/package/test_python_aexpect.py
>  
>  N:	Joseph Kogut <joseph.kogut@gmail.com>
> -F:	package/at-spi2-atk/
>  F:	package/at-spi2-core/
>  F:	package/clang/
>  F:	package/earlyoom/
> diff --git a/package/Config.in b/package/Config.in
> index eaac32a01a..70c5c2eea8 100644
> --- a/package/Config.in
> +++ b/package/Config.in
> @@ -1554,7 +1554,6 @@ endmenu
>  
>  menu "Graphics"
>  	source "package/assimp/Config.in"
> -	source "package/at-spi2-atk/Config.in"
>  	source "package/at-spi2-core/Config.in"
>  	source "package/atk/Config.in"
>  	source "package/atkmm/Config.in"
> diff --git a/package/at-spi2-atk/0001-meson-add-tests-option.patch b/package/at-spi2-atk/0001-meson-add-tests-option.patch
> deleted file mode 100644
> index 3a2aefa090..0000000000
> --- a/package/at-spi2-atk/0001-meson-add-tests-option.patch
> +++ /dev/null
> @@ -1,43 +0,0 @@
> -From 466d04a57b32417dbf2d5e818af1b7593af5f802 Mon Sep 17 00:00:00 2001
> -From: Fabrice Fontaine <fontaine.fabrice@gmail.com>
> -Date: Sun, 3 May 2020 10:29:36 +0200
> -Subject: [PATCH] meson: add tests option
> -
> -Add tests option to allow the user to build at-spi2-atk without tests,
> -it is especially useful as one of the test requires libxml-2.0
> -
> -Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
> -[Upstream status:
> -https://gitlab.gnome.org/GNOME/at-spi2-atk/-/merge_requests/22]
> ----
> - meson.build       | 4 +++-
> - meson_options.txt | 4 ++++
> - 2 files changed, 7 insertions(+), 1 deletion(-)
> -
> -diff --git a/meson.build b/meson.build
> -index 9fce5d6..61bc93d 100644
> ---- a/meson.build
> -+++ b/meson.build
> -@@ -61,4 +61,6 @@ install_data('at-spi2-atk.desktop',
> - 
> - subdir('droute')
> - subdir('atk-adaptor')
> --subdir('tests')
> -+if get_option('tests')
> -+  subdir('tests')
> -+endif
> -diff --git a/meson_options.txt b/meson_options.txt
> -index 24c9228..a4abdc2 100644
> ---- a/meson_options.txt
> -+++ b/meson_options.txt
> -@@ -2,3 +2,7 @@ option('disable_p2p',
> -        description: 'Disable peer-to-peer DBus connections',
> -        type: 'boolean',
> -        value: false)
> -+option('tests',
> -+       description: 'Whether to build tests',
> -+       type: 'boolean',
> -+       value: true)
> --- 
> -2.26.2
> -
> diff --git a/package/at-spi2-atk/Config.in b/package/at-spi2-atk/Config.in
> deleted file mode 100644
> index 9a7cfde0af..0000000000
> --- a/package/at-spi2-atk/Config.in
> +++ /dev/null
> @@ -1,22 +0,0 @@
> -config BR2_PACKAGE_AT_SPI2_ATK
> -	bool "at-spi2-atk"
> -	depends on BR2_PACKAGE_XORG7 # at-spi2-core
> -	depends on BR2_TOOLCHAIN_HAS_THREADS # glib2
> -	depends on BR2_USE_MMU # glib2
> -	depends on BR2_USE_WCHAR # glib2
> -	select BR2_PACKAGE_ATK
> -	select BR2_PACKAGE_AT_SPI2_CORE
> -	select BR2_PACKAGE_LIBGLIB2
> -	help
> -	  The At-Spi2 Atk package contains a library that bridges
> -	  ATK to At-Spi2 D-Bus service.
> -
> -	  https://www.freedesktop.org/wiki/Accessibility/AT-SPI2/
> -
> -comment "at-spi2-atk needs a toolchain w/ wchar, threads"
> -	depends on BR2_USE_MMU
> -	depends on !BR2_USE_WCHAR || !BR2_TOOLCHAIN_HAS_THREADS
> -
> -comment "at-spi2-atk depends on X.org"
> -	depends on BR2_USE_MMU
> -	depends on !BR2_PACKAGE_XORG7
> diff --git a/package/at-spi2-atk/at-spi2-atk.hash b/package/at-spi2-atk/at-spi2-atk.hash
> deleted file mode 100644
> index 9c564c32af..0000000000
> --- a/package/at-spi2-atk/at-spi2-atk.hash
> +++ /dev/null
> @@ -1,5 +0,0 @@
> -# From http://ftp.acc.umu.se/pub/gnome/sources/at-spi2-atk/2.34/at-spi2-atk-2.34.2.sha256sum
> -sha256  901323cee0eef05c01ec4dee06c701aeeca81a314a7d60216fa363005e27f4f0  at-spi2-atk-2.34.2.tar.xz
> -
> -# locally calculated
> -sha256  dc626520dcd53a22f727af3ee42c770e56c97a64fe3adb063799d8ab032fe551  COPYING
> diff --git a/package/at-spi2-atk/at-spi2-atk.mk b/package/at-spi2-atk/at-spi2-atk.mk
> deleted file mode 100644
> index 232b577230..0000000000
> --- a/package/at-spi2-atk/at-spi2-atk.mk
> +++ /dev/null
> @@ -1,19 +0,0 @@
> -################################################################################
> -#
> -# at-spi2-atk
> -#
> -################################################################################
> -
> -AT_SPI2_ATK_VERSION_MAJOR = 2.34
> -AT_SPI2_ATK_VERSION = $(AT_SPI2_ATK_VERSION_MAJOR).2
> -AT_SPI2_ATK_SOURCE = at-spi2-atk-$(AT_SPI2_ATK_VERSION).tar.xz
> -AT_SPI2_ATK_SITE = \
> -	http://ftp.gnome.org/pub/gnome/sources/at-spi2-atk/$(AT_SPI2_ATK_VERSION_MAJOR)
> -AT_SPI2_ATK_LICENSE = LGPL-2.1+
> -AT_SPI2_ATK_LICENSE_FILES = COPYING
> -AT_SPI2_ATK_CPE_ID_VENDOR = gnome
> -AT_SPI2_ATK_INSTALL_STAGING = YES
> -AT_SPI2_ATK_DEPENDENCIES = atk at-spi2-core libglib2 host-pkgconf
> -AT_SPI2_ATK_CONF_OPTS = -Dtests=false
> -
> -$(eval $(meson-package))
> -- 
> 2.34.1
> 
> _______________________________________________
> buildroot mailing list
> buildroot@buildroot.org
> https://lists.buildroot.org/mailman/listinfo/buildroot
> 

Cheers,
—Adrián

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 195 bytes --]

[-- Attachment #2: Type: text/plain, Size: 150 bytes --]

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

  reply	other threads:[~2023-05-05 11:56 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-04-28  7:59 [Buildroot] [PATCH v7 1/5] package/at-spi2-core: bump to version 2.48.0 Thomas Devoogdt
2023-04-28  7:59 ` [Buildroot] [PATCH v7 2/5] package/at-spi2-atk: remove package Thomas Devoogdt
2023-05-05 11:56   ` Adrian Perez de Castro [this message]
2023-04-28  7:59 ` [Buildroot] [PATCH v7 3/5] package/atk: " Thomas Devoogdt
2023-05-05 11:56   ` Adrian Perez de Castro
2023-04-28  7:59 ` [Buildroot] [PATCH v7 4/5] package/libgtk3: re-enable atk-bridge Thomas Devoogdt
2023-05-05 11:57   ` Adrian Perez de Castro
2023-04-28  7:59 ` [Buildroot] [PATCH v7 5/5] package/libgtk3: fix update-icon-cache on empty directory Thomas Devoogdt
2023-05-05 11:57   ` Adrian Perez de Castro
2023-05-05 11:56 ` [Buildroot] [PATCH v7 1/5] package/at-spi2-core: bump to version 2.48.0 Adrian Perez de Castro
2023-05-05 11:58   ` Adrian Perez de Castro
2023-05-08 11:35 ` Yann E. MORIN
2023-05-08 14:52   ` Thomas Devoogdt
2023-05-08 16:05 ` Yann E. MORIN

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=20230505145633.GD3311548@igalia.com \
    --to=aperez@igalia.com \
    --cc=buildroot@buildroot.org \
    --cc=d.lang@abatec.at \
    --cc=fontaine.fabrice@gmail.com \
    --cc=james.knight@collins.com \
    --cc=joseph.kogut@gmail.com \
    --cc=thomas.devoogdt@barco.com \
    --cc=thomas@devoogdt.com \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.