-
Notifications
You must be signed in to change notification settings - Fork 594
INSTALL: Add more pod markup #23726
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
base: blead
Are you sure you want to change the base?
INSTALL: Add more pod markup #23726
Conversation
khwilliamson
commented
Sep 18, 2025
- This set of changes does not require a perldelta entry.
This gives more complete advice to someone reading this document about how to read it.
This adds =for apidoc lines to INSTALL for use by autodoc.pl
|
||
After it runs, Configure will perform variable substitution on all the | ||
*.SH files and offer to run make depend. | ||
*.SH files and offer to run S<C<make depend>>. |
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.
Just tested out this construct in an HTML renderer (never used S before) and C<S<foo bar>>
successfully applies the nowrap style but S<C<foo bar>>
does not due to the nesting of span and code elements. Either seems to work for commandline perldoc.
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.
That's good to know, and we should document this issue. It is a problem when the C<> is for just a subset of what needs to be on the same line. It also needs to be documented that for links to the same pod, you should use C<L/<...>> vs on a different pod, L<C<...>>
INSTALL
Outdated
option when prompted by Configure. If your libraries are not in a | ||
directory normally searched by your C compiler and linker, then you will | ||
need to include the appropriate -L/your/directory option when prompted | ||
need to include the appropriate -L/>I<your/directory> option when prompted |
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.
Missing opening markup
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.
thanks
There's more that could be added; but this is a lot
a0e474f
to
a6ba45e
Compare