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 746609 - Fix warning when book cannot be read
Fix warning when book cannot be read
Status: RESOLVED FIXED
Product: devhelp
Classification: Applications
Component: General
unspecified
Other All
: Normal normal
: ---
Assigned To: devhelp-maint
devhelp-maint
Depends on:
Blocks:
 
 
Reported: 2015-03-22 16:25 UTC by Michael Catanzaro
Modified: 2015-03-23 09:22 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
Fix warning when book cannot be read (935 bytes, patch)
2015-03-22 16:25 UTC, Michael Catanzaro 		committed Details | Review

Description Michael Catanzaro 2015-03-22 16:25:23 UTC 
When starting devhelp I see the following warning:

(devhelp:27746): Devhelp-WARNING **: Failed to read '(null)': Error on line 1 char 1: Document was empty or contained only whitespace

Should be:

(devhelp:32157): Devhelp-WARNING **: Failed to read '/home/mcatanzaro/jhbuild/install/share/gtk-doc/html/gegl/gegl.devhelp': Error on line 1 char 1: Document was empty or contained only whitespace
Comment 1 Michael Catanzaro 2015-03-22 16:25:26 UTC 
Created attachment 300081 [details] [review]
Fix warning when book cannot be read

priv->path is always NULL at this point
Comment 2 Aleksander Morgado 2015-03-23 08:53:35 UTC 
Fixed, thanks.
Comment 3 Frederic Peters 2015-03-23 08:57:26 UTC 
Do note we are currently in hard code freeze, this shouldn't have been pushed without asking for approval.
Comment 4 Aleksander Morgado 2015-03-23 09:18:27 UTC 
(In reply to Frederic Peters from comment #3)
> Do note we are currently in hard code freeze, this shouldn't have been
> pushed without asking for approval.

Ouch, ouch... should I just revert? Sorry, didn't realize...
Comment 5 Frederic Peters 2015-03-23 09:22:21 UTC 
nope, let's keep it that way.