After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 782748 - AT-SPI2 is missing roles that are present in ATK
AT-SPI2 is missing roles that are present in ATK
Status: RESOLVED FIXED
Product: at-spi
Classification: Platform
Component: api
unspecified
Other Linux
: Normal normal
: ---
Assigned To: Joanmarie Diggs (IRC: joanie)
At-spi maintainer(s)
Depends on: 782041
Blocks:
 
 
Reported: 2017-05-17 16:09 UTC by Joanmarie Diggs (IRC: joanie)
Modified: 2017-05-18 18:33 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
proposed patch (1.58 KB, patch)
2017-05-17 16:21 UTC, Joanmarie Diggs (IRC: joanie)
committed Details | Review

Description Joanmarie Diggs (IRC: joanie) 2017-05-17 16:09:13 UTC
+++ This bug was initially created as a clone of Bug #782041 +++

Bug 720065 resulted in several roles being added to ATK. Making the corresponding change in AT-SPI2 seems to have never happened. In particular, these roles seem absent:

* ATK_ROLE_DESCRIPTION_LIST
* ATK_ROLE_DESCRIPTION_TERM
* ATK_ROLE_DESCRIPTION_VALUE

https://git.gnome.org/browse/atk/commit/?id=9cd7ce3812

N.B. This bug depends on bug 782041 because we need the role in at-spi2-core first.
Comment 1 Joanmarie Diggs (IRC: joanie) 2017-05-17 16:21:57 UTC
Created attachment 352037 [details] [review]
proposed patch

Note that this patch bumps the at-spi2-core dependency to version 2.25.1. Technically, it should be 2.25.2 -- assuming the patch for bug 782041 lands and winds up in the 2.25.2 release. I went with the .1 micro version so that at-spi2-atk would build but at the same time hint strongly at the fact that at-spi2-core would almost certainly need to be upgraded as well.

Please review. Thanks!
Comment 2 Mike Gorse 2017-05-18 18:09:35 UTC
Comment on attachment 352037 [details] [review]
proposed patch

I've bumped the at-spi2-core version to 2.25.2 (I really should have done this after rolling the 2.25.1 release anyhow). We should check for version 2.25.2; you can make that change before committing if you'd like. Anyhow, the patch looks fine.
Comment 3 Joanmarie Diggs (IRC: joanie) 2017-05-18 18:33:10 UTC
Comment on attachment 352037 [details] [review]
proposed patch

(In reply to Mike Gorse from comment #2)
> Comment on attachment 352037 [details] [review] [review]
> proposed patch
> 
> I've bumped the at-spi2-core version to 2.25.2 (I really should have done
> this after rolling the 2.25.1 release anyhow). We should check for version
> 2.25.2; you can make that change before committing if you'd like. Anyhow,
> the patch looks fine.

Thanks for the review. Committed with the updated version check:
https://git.gnome.org/browse/at-spi2-atk/commit/?id=28bf17e