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
DevGuide Cleanup and Improvments #4281
DevGuide Cleanup and Improvments #4281
Conversation
- Security Section - Moved building instructions to building/index. - Restructured to make governance and permissions sections more consistent - Document fnmatch expressions - Simplified anchors at the top of devguide sections. - Formatting in alternate_interfaces_to_data - Fixed minor typos
Also added news and converted opendds_idl options from a table.
So the merge works
docs/building/index.rst
Outdated
- Xerces-C++ v3.x | ||
- OpenSSL v1.0.2+, v1.1, or v3.0.1+ |
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.
Should we consider moving these to some kind of file or is this the only file that contains these versions?
Future work.
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.
This information is already in the dependencies, so I will just link to Xerces and OpenSSL there. In the future though most of what I added here should probably be generalized and maybe moved to dependencies.
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.
Yeah, thinking about it more, unless it's trivial like package names or a special case, I don't really want to be in the business of telling people how to build and install FOSS that isn't ours.
Co-authored-by: Justin Wilson <wilsonj@unityfoundation.io>
doccon
syntax highlighting tobatch
.doccon
was the wrong choice because it was for DOS/batch sessions, not actual batch commands.:omgspec:
.