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 775742 - "make upload" just continues if hotdoc is not found
"make upload" just continues if hotdoc is not found
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: documentation
git master
Other Linux
: Normal critical
: 1.11.1
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2016-12-07 09:15 UTC by Sebastian Dröge (slomo)
Modified: 2016-12-08 07:41 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Sebastian Dröge (slomo) 2016-12-07 09:15:52 UTC
$ make upload
make: hotdoc: Command not found
rsync -rvaz -e ssh --links --delete /home/slomo/Projects/gstreamer/head/gst-docs/built_doc/html/ gstreamer.freedesktop.org:/srv/gstreamer.freedesktop.org/public_html/documentation || /bin/true
Comment 1 Tim-Philipp Müller 2016-12-07 22:05:18 UTC
I'm not sure if we can do much about this. If hotdoc is not installed, the Makefile won't include the hotdoc .mk file bit, so it makes no sense to rely on any of the hotdoc targets here.

I guess we can check manually for hotdoc before proceeding?

Mathieu, any better ideas?
Comment 2 Mathieu Duponchelle 2016-12-07 22:17:53 UTC
Not really, I would only enable the upload rule `if ENABLE_DOCUMENTATION`, that ensures not only hotdoc is present, but also the other prerequisites as checked by the HOTDOC_CHECK macro.
Comment 3 Mathieu Duponchelle 2016-12-07 22:19:18 UTC
Ah but we don't have a configure phase in there, right :)

Well my suggestion would be to have one, or (even better) work on a meson module :)
Comment 4 Olivier Crête 2016-12-07 22:36:18 UTC
Can't we just add a phony target that runs "hotdoc --help >/dev/null" and make all other relevant targets depends on that. If it is not present, it will just fail.
Comment 5 Mathieu Duponchelle 2016-12-08 01:17:53 UTC
Guess that could work, doesn't seem like the cleanest solution to me though, mostly depends on the time one is willing to spend on this.
Comment 6 Tim-Philipp Müller 2016-12-08 07:41:59 UTC
Pushed something like what Olivier suggested now. Could probably be improved, but seems to work.

commit 40317e7bf35221ee8431a85d186495f5499c8f2a
Author: Tim-Philipp Müller <tim@centricular.com>
Date:   Thu Dec 8 07:38:41 2016 +0000

    Makefile: check for hotdoc before attempting upload
    
    https://bugzilla.gnome.org/show_bug.cgi?id=775742

I gather from your comments Mathieu that this or the meson part is not something you plan to work on, so closing the bug, otherwise please re-open.