Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

gnome.gdbus_codegen does not match the behavior described by the documentation #9820

Closed
xry111 opened this issue Jan 13, 2022 · 1 comment
Closed

Comments

@xry111
Copy link
Contributor

xry111 commented Jan 13, 2022

Describe the bug

The doc says:

annotations: (Added 0.43.0) list of lists of 3 strings for the annotation for 'ELEMENT', 'KEY', 'VALUE'

But now meson only accepts one list of 3 strings.

To Reproduce

Try to build gdm-41.3 (https://download.gnome.org/sources/gdm/41/gdm-41.3.tar.xz).

../libgdm/meson.build:11:0: ERROR: gnome.gdbus_codegen keyword argument "annotations" must be made up of 3 strings for ELEMENT, KEY, and VALUE

Expected behavior

gdm should build successfully.

Or, if this is an intentionally behavior change, the documentation (https://mesonbuild.com/Gnome-module.html#gnomegdbus_codegen) shall be updated.

system parameters

  • Native build
  • Linux From Scratch
  • Python 3.10.1
  • Meson 0.61.0
  • Ninja 1.10.2
@xry111
Copy link
Contributor Author

xry111 commented Jan 13, 2022

Seems already fixed in master.

@xry111 xry111 closed this as completed Jan 13, 2022
ford-prefect pushed a commit to PipeWire/pipewire that referenced this issue Mar 11, 2023
Currently, among others, the `pipewire-pulse` executable is an
exact copy of the `pipewire` executable. Use meson's
`install_symlink()` to avoid the need for compiling the same thing
multiple times. Also use `custom_target()` so that the aliases
are available in an uninstalled environment.

Do the same for `pw-cat`. The benefit is that all aliases
of `pw-cat` are now available in an uninstalled environment.

This commit increasese the minimum meson version to 0.61.1
as that is needed for `install_symlink()`.

The reason for using 0.61.1 instead of 0.61.0 is the following bug:
mesonbuild/meson#9820
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant