-
Notifications
You must be signed in to change notification settings - Fork 811
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Doxygen and texi documentation cleanup #1943
Conversation
@gjanssens Thanks very much for taking care of this! I'm in favor of deleting libgnucash/docs. Suggestions for the keeper files:
|
Perhaps dia is closer to svg than to png. |
@@ -1,5 +1,6 @@ | |||
# CMakeLists.txt for libgnucash/backend/xml | |||
|
|||
add_subdirectory(DTD) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@gjanssens You forgot to add DTD to the dist-list at the bottom. That's why the ubuntu CI failed: It runs distcheck instead of check.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jralls yea, I saw that and have the local fix already in my tree. I didn't push yet as there were some extra tweaks to do.
The file mostly had short descriptions that were very similar to the descriptions of business objects it referred to. The exception was an explanation of how the billing terms and tax tables handled their immutable copies. That bit has been moved to its own group under the Business group and is referenced from the BillTerm and TaxTables group. This commit also fixes rendering of the Address doxygen info and expands the BillTerm info to explain how proximo due dates work.
docygen_mainpage.c None of them had useful information describing actual implementation concepts or details.
It's extremely out of date and we no longer use it to keep track of our design decisions and suggestions. Only a section on KVP policy has been move into kvp_doc.txt as it may make sense there.
Note: this also means the target directory changes from libgnucash/docs/html to [toplevel]/doxygen/html
Relevant diagrams have been added in the wiki as png files. The html files with financial calculations are copyrighted by someone outside of the gnucash project. I'd rather not add that info to the wiki. Similar information should be easy to find on the internet.
I have done so for loans.txt, lots.txt and libgnucash/engine/TaxTableBillTermImmutability.txt
In the PR I moved it to Hacking, with the intention to eventually move it to the wiki. I forgot to do so in this PR, but have done so in a follow-up commit
Good idea. Done
I don't think they warrant their own cmake-module so I have moved the lines to the root CMakeLists.txt file Note this will inevitably also change the output directory. So we'll need @derekatkins to update the nightly build script to read the built documentation from the new directory. It will now be built into [builddir]/doxygen/html instead of [builddir]/libgnucash/doc/html.
I did so for the two files that were still relevant. @fellen svg is not enabled in our wiki, so I went with png.
Done, and fixed my initial distcheck failure.
I did not move these into the wiki. There were copyright notices in these files that made me suspect they were snatched from elsewhere long time ago. I'm not comfortable publishing this in our wiki. Additionally it would be more work than I care to do (and have time for) for information that is available online as you say. |
@gjanssens Thanks again. |
Following a note by @jralls on a bug earlier this week, I have set out to clean up the documentation directory in engine.
As already suggested most of it is horribly out of date and no longer relevant. I have removed all of that. I did keep a few bits that seemed helpful still as they are explaining a few concepts in the code.
Other than that I have cleaned out the main doxygen page a bit.
Question: I'm tempted to eliminate the directory libgnucash/docs altogether.
The files that are still in there could move to other locations. Here are my proposals:
Thoughts ?