Skip to content

Conversation

khwilliamson
Copy link
Contributor

  • 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
@khwilliamson khwilliamson requested a review from Grinnz September 18, 2025 15:47

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>>.
Copy link
Contributor

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.

Copy link
Contributor Author

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
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing opening markup

Copy link
Contributor Author

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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants