#54755 closed defect (fixed)
gtkspell2 @2.0.16_6: fails to build with gtk-doc @1.26
Reported by: | majoc-at-astro (majoc-at-astro) | Owned by: | kurthindenburg (Kurt Hindenburg) |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | ports | Version: | 2.4.1 |
Keywords: | Cc: | dbevans (David B. Evans) | |
Port: | gtkspell2 gtk-doc |
Description
Full log included, but this seems to be the nub of it:
:info:build gtk-doc: Rebuilding template files :info:build cd . && gtkdoc-mktmpl --module=gtkspell :info:build /bin/sh: gtkdoc-mktmpl: command not found :info:build make[2]: *** [tmpl-build.stamp] Error 127
Please see also:
https://bugs.archlinux.org/task/55210
.... which shows the same problem, and suggests disabling gtk-doc while building gtkspell as a kludgearound.
Feel free to raise this as a bug against gtk-doc instead (or as well), if that's where the true bug lies. Apologies for possibly-inappropriate CC: I see no maintainer for gtkspell2.
Attachments (3)
Change History (18)
Changed 7 years ago by majoc-at-astro (majoc-at-astro)
Attachment: | gtkspell2.log.2017-09-03_23.30.37 added |
---|
comment:1 Changed 7 years ago by majoc-at-astro (majoc-at-astro)
PS: knock-on effects include taking out inkscape and rednotebook.
comment:2 Changed 7 years ago by Russell-Jones-OxPhys (Russell Jones)
Happened in 1.26 https://github.com/GNOME/gtk-doc/blob/GTK_DOC_1_26/NEWS
GTK-Doc 1.26 (Aug 11 2017) ============== Remove tmpl support (gtkdoc-mktmpl). Port all tools from bash/perl to python. ...
configure.ac says GTK_DOC_CHECK([1.14],[--flavour no-tmpl])
as per https://developer.gnome.org/gtk-doc-manual/unstable/settingup_autoconf.html.en , so I wonder if it's the line in docs/Makefile.am that's causing the build to try and use gtkdoc-mktmpl.
comment:3 follow-up: 4 Changed 7 years ago by ggmessier
I can confirm that the issue is fixed by switching back to gtk-doc 1.25 using the procedure described here: wiki:howto/InstallingOlderPort. Both gtkspell2 (and inkscape) build now.
comment:4 follow-up: 5 Changed 7 years ago by majoc-at-astro (majoc-at-astro)
Replying to ggmessier:
I can confirm that the issue is fixed by switching back to gtk-doc 1.25 using the procedure described here: wiki:howto/InstallingOlderPort. Both gtkspell2 (and inkscape) build now.
Apologies, but that wouldn't work for us, as our build systems install everything from scratch in the interests of repeatability. Please see the above diff: an admitted quick-and-dirty to omit the documentation, which I can confirm works for us. I'll do better later. Would adding a +docs variant be acceptable?
Thanks in advance.
comment:5 follow-up: 6 Changed 7 years ago by mf2k (Frank Schima)
Replying to majoc-at-astro:
Would adding a +docs variant be acceptable?
Yes, that would be a good workaround.
comment:6 Changed 7 years ago by majoc-at-astro (majoc-at-astro)
Replying to mf2k:
Replying to majoc-at-astro:
Would adding a +docs variant be acceptable?
Yes, that would be a good workaround.
Please find diff for same attached: found to build successfully without +docs, and to correctly fail with it. Desciption string provisional.
Changed 7 years ago by majoc-at-astro (majoc-at-astro)
Attachment: | Portfile-gtkspell2.2.diff added |
---|
Patch to make documentation optional
comment:7 Changed 7 years ago by majoc-at-astro (majoc-at-astro)
Corrected diff (re)attached: now adds gtk-doc back correctly. My bad. Apologies.
comment:8 follow-up: 9 Changed 7 years ago by mf2k (Frank Schima)
I don't understand the variant description you are using. The description on most +docs
variants is something like "Build documentation". Can it be changed?
comment:9 Changed 7 years ago by majoc-at-astro (majoc-at-astro)
Replying to mf2k:
I don't understand the variant description you are using. The description on most
+docs
variants is something like "Build documentation". Can it be changed?
That was a placeholder to suggest it shouldn't be used. Will change.
Changed 7 years ago by majoc-at-astro (majoc-at-astro)
Attachment: | Portfile-gtkspell2.diff added |
---|
Add variant to make documentation optional
comment:10 Changed 7 years ago by kurthindenburg (Kurt Hindenburg)
Owner: | set to kurthindenburg |
---|---|
Resolution: | → fixed |
Status: | new → closed |
comment:11 Changed 7 years ago by ryandesign (Ryan Carsten Schmidt)
As I'm sure you can guess, the port doesn't build when the +docs variant is used:
/bin/sh: gtkdoc-mktmpl: command not found
So why did we add the variant?
This version of gtkspell is from 2009 so I doubt its developers will be fixing this problem. I don't see the same problem with gtkspell3 though, even if I add --enable-gtk-doc
, so maybe a fix can be backported.
comment:12 Changed 7 years ago by kurthindenburg (Kurt Hindenburg)
It was a quick fix since a lot of port were broken. I meant to return to this. I don't see any reference to _tmpl in gtkspell3.
Perhaps +docs should be just removed
comment:13 Changed 7 years ago by kurthindenburg (Kurt Hindenburg)
comment:14 Changed 7 years ago by dbevans (David B. Evans)
Just a few clarifications.
Removing the +doc variant is truly appropriate as this port (as do most that use gtk-doc) have the (pregenerated) docs included as part of the tarball. Thus the variant attempts to redundantly regenerate the existing docs (and fails). Without the variant, the pregenerated docs are installed as part of the port.
gtkspell2 is, indeed, the legacy version of gtkspell and is no longer being maintained. It provides only a gtk2 compatible API. gtkspell3 is the current version of gtkspell and contains APIs for both gtk2 and gtk3 although the the gtk2 API is incompatible with gtkspell2. So gtkspell2 is still necessary for gtk2 apps that use the old API (inkscape for instance). Most newer apps have now moved to gtk3 so the newer gtk2 API is not widely used.
Hope this is helpful
logfile for gtkspell2 build