Skip to content

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.

Sign up for GitHub

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

Open
jkeenan wants to merge 1 commit into
base: blead
Choose a base branch
from
+3 −3
Open

Grammar and POD formatting corrections #23990

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions README.linux
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -16,9 +16,9 @@ version 5 (hereafter just Perl) is compiled and/or runs.
=head2 Deploying Perl on Linux

Normally one can install F</usr/bin/perl> on Linux using your distribution's
package manager (e.g: C<sudo apt-get install perl>, or
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
Copy link
Contributor

@Leont Leont Dec 8, 2025

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

Copy link
Contributor

@Grinnz Grinnz Dec 8, 2025

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?

Copy link
Contributor Author

@jkeenan jkeenan Dec 8, 2025

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.

Copy link
Contributor

@rwp0 rwp0 Dec 9, 2025

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.

Copy link

@guest20 guest20 Dec 9, 2025

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"

Copy link
Contributor Author

@jkeenan jkeenan Dec 9, 2025

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?

Copy link
Contributor

@Leont Leont Dec 9, 2025

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

messing with the system's perl is not always recommended. One can use
L<perlbrew|https://perlbrew.pl/> to avoid such issues.

Expand All @@ -32,7 +32,7 @@ Linux. As of May 2019, the last stable release took place on 2017, and one can
buy support contracts for them.

There are some special instructions for building Perl with Sun Studio on
Linux. Following the normal C<Configure>, you have to run make as follows:
Linux. Following the normal C<Configure>, you have to run F<make> as follows:

LDLOADLIBS=-lc make

Expand Down
Loading