Use the new tycho-document-bundle-plugin:schema-to-html to generate doc#1489
Conversation
|
I think one quite obvious enhancement to the process is that we simply add all reactor projects to the search path I can't think of a situation where I do not want to find a reference and give me an error instead (@akurtakov error are currently only printed to console but never failed the build, wdt should we make it an option to fail or simply fail on any error or leave it as is?). Second is, currently in the build.xml there are some copy tasks left: some copy the exsd and html others only html ... so I wonder what is the intend here, e.g. do they originate from "referenced schemas" actually and the mojo should simply always copy referenced ones? Also include the exsd? Or is this something referenced in comments of the schema and we can't automate this? Maybe @merks as the only exsd schema expert I'm aware of :-) |
@akurtakov need to check why removing genhtml breaks help index but I though we stopped generating help index at all? edit: it seems there was a reference in the management section but removed from the childs. |
19df017 to
a226df3
Compare
|
About the copy it seems these htmls are already check in to version control and not generated .. anyone knows why? |
|
Indexes are not part of the published bundles. Maybe I didn't manage to disable generating them fully. |
|
Alright I removed that now, next issue is:
I think I can create a dummy file but probably simply remove the inclusion of that file in eclipse-junit-tests 😱? |
The new tycho-document-bundle-plugin:schema-to-html can replace the ant task "pde.convertSchemaToHTML" even if we still need a quite verbose configuration this already has two benefits: - configuration is located at maven so much more visible no need to look around - the mojo has access to the full build and dependency information and can be enhanced in the future to remove the need for manual configuration Also - remove obsolete execution from management section - do not include non existing files anymore
5e773f1 to
58bcdf8
Compare
|
Build is green now, if no one objects I would like to merge this to see if ibuild is happy as well... |
|
The build is still running at the moment. |
Yes I squashed and rebased it so lets wait a bit.. |
|
Build succeeded. I'm pushing and starting new I-build to verify everything is fine. |
The new tycho-document-bundle-plugin:schema-to-html can replace the ant task "pde.convertSchemaToHTML" even if we still need a quite verbose configuration this already has two benefits: