Bug 74013 - Doxygen script is including directories that don't exist
Summary: Doxygen script is including directories that don't exist
Status: RESOLVED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: LibreOffice (show other bugs)
Version:
(earliest affected)
4.0.3.1 rc
Hardware: Other All
: medium normal
Assignee: Not Assigned
URL:
Whiteboard: target:4.3.0
Keywords:
Depends on:
Blocks:
 
Reported: 2014-01-24 11:33 UTC by Chris Sherlock
Modified: 2014-01-26 23:18 UTC (History)
0 users

See Also:
Crash report or crash signature:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Chris Sherlock 2014-01-24 11:33:34 UTC
When I run make docs, I get the following at the start:

cleaning up
generating doxygen docs
Calling /doxygen /home/chris/repos/LibreOffice/core/solenv/inc/doxygen.cfg with
Input:      accessibility/source accessibility/inc include/accessibility 
Output:     /home/chris/repos/LibreOffice/core/docs/accessibility
Include:     /home/chris/repos/LibreOffice/core/include    
Version:    master
Tagfile:    /home/chris/repos/LibreOffice/core/docs/accessibility/accessibility.tags
Ref-Tags:   
Title:      LibreOffice Module accessibility
Warning: Tag `SYMBOL_CACHE_SIZE' at line 349 of file /home/chris/repos/LibreOffice/core/solenv/inc/doxygen.cfg has become obsolete.
To avoid this warning please remove this line from your configuration file or upgrade it using "doxygen -u"
Warning: tag INPUT: input source `include/accessibility' does not exist
Warning: source include/accessibility is not a readable file or directory... skipping.

Lots more errors also. 

---

The first warning is to remove SYMBOL_CACHE_SIZE as this was removed with Doxygen 1.8.4 (and we don't use it anyway). Probably just run doxygen -u on the config file (already submitted a patch here: https://gerrit.libreoffice.org/#/c/7610/

The second warning appears to be caused because mkdocs.sh has the following line:

DOXYGEN_INPUT=`printf "%s" "$PROJECT/source $PROJECT/inc include/$PROJECT "`

This won't work for modules like accessibility that don't have an include/PROJECT directory yet.
Comment 1 Chris Sherlock 2014-01-24 13:31:51 UTC
Also... we need to increase DOT_GRAPH_MAX_NODES
Comment 2 Commit Notification 2014-01-26 10:02:39 UTC
Chris Sherlock committed a patch related to this issue.
It has been pushed to "master":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=756873f014646ce1c8ef540bee05bd420680b663

fdo#74013 Upgrade doxygen config file



The patch should be included in the daily builds available at
http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
http://wiki.documentfoundation.org/Testing_Daily_Builds
Affected users are encouraged to test the fix and report feedback.
Comment 3 Commit Notification 2014-01-26 19:31:53 UTC
Chris Sherlock committed a patch related to this issue.
It has been pushed to "master":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=247cd33817457789645827a52d79a1876b9760a3

fdo#74013 make makedocs.sh look in correct project header directories



The patch should be included in the daily builds available at
http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
http://wiki.documentfoundation.org/Testing_Daily_Builds
Affected users are encouraged to test the fix and report feedback.
Comment 4 Chris Sherlock 2014-01-26 23:18:49 UTC
Fixed.