-
Notifications
You must be signed in to change notification settings - Fork 602
Grammar and POD formatting corrections #23990
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?
Conversation
As requested by hmartink. Fixes: GH Perl#23932
| package manager (e.g., C<sudo apt-get install perl>, or | ||
| C<sudo dnf install perl>). Note that sometimes one needs to install some | ||
| extra system packages in order to be able to use CPAN frontends, and that | ||
| extra system packages in order to be able to use CPAN packages, and that |
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.
I don't understand this change. I think "CPAN frontends" was deliberate here, though "CPAN clients" may be clearer
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.
I think it's unclear to begin with, certainly some CPAN packages need extra system libraries, but this may be referring to needing things like gcc and make to install them?
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.
When reviewing GH #23932, I didn't understand what frontends meant in this context; the requestor's suggestion of packages made more sense. But I'm wedded to that; I just want to get that ticket closed.
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.
To me ("using" CPAN) "modules" would be the most commonly heard phrase here, but "distributions" would make sense too I guess.
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.
I think this is talking about "installers" like carton/cpm/cpan{,+,-} as "cpan frontends"
Like "You need to install the dependencies if the installers, and likely also a compiler"
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.
@shlomif, this section was your patch from 2019. Would you like to comment here?
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.
I think this is talking about "installers" like carton/cpm/cpan{,+,-} as "cpan frontends"
Yeah, exactly
As requested by hmartink.
Fixes: GH #23932