GNOME Bugzilla – Bug 781236
Building docs with meson fails due to gtkdoc-scangob behaviour change
Last modified: 2017-04-13 13:20:57 UTC
Created attachment 349748 [details] Output of gstreamer module When using meson build system in JHBuild, the installation phase will result error in "ninja install" in the step "Building documentation for gstreamer". See the attachment for a complete output. distro: Arch Linux 64bit softwares: jhbuild (updated from GNOME repository + 'gnome-world' module set) gtk-doc 1.25.1 (built with jhbuild) Workaround: I successfully disable building documentation appending '-Ddisable_gtkdoc' as meson argument
Yes, this has come up on IRC. The question is: is this a gtk-doc bug or a meson issue or a gstreamer issue. My bet is: either meson issue or a gtk-doc issue. I think it would be good to check how gtkdoc-scangobj is called exactly. That can probably be checked by running something like strace -f ninja gstreamer-1.0-doc or running strace -f on the command line that comes with the error.
Created attachment 349750 [details] Strace output See attached file.
Created attachment 349751 [details] XZ file with output of "strace -f ninja install"
Looks like this was caused by backwards incompatible changes in gtk-doc: https://github.com/mesonbuild/meson/pull/1618 Thanks for providing a patch. Wonder if this should be fixed in gtk-doc too, to make it accept the old way as well.
Looks like this change in gtk-doc behaviour was not necessarily intentional, so re-opening and moving to gtk-doc for now. <ensonic> tpm, let me check, the porting should not change things, please tell us rather than working around it
<ebassi> `$TYPES_FILE = $TYPES_FILE ? $TYPES_FILE : "$OUTPUT_DIR/$MODULE.types";` <ebassi> There was a fallback <ensonic> tpm, commented on the commit, I thing the perl option parser just ignored the extra arg
The perl option parser was ignoring the extra arg and then the fallback kicked in. Now in python we fail with the extra arg. In any case, please fix the typo on the meson patch (see my comment). Closing this.
Thanks Stefan.