Register "cperl" as an alias of Perl 5 #4067
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Emacs ships with an enhanced major-mode for editing Perl with embedded C sections (called `cperl-mode`). This commit enables Linguist to identify Perl files containing cperl modelines.
Alhadis
changed the title
cperl as an alias of Perl 5
JJ
reviewed
Mar 13, 2018
There was a problem hiding this comment.
There's cperl (a CPanel version of Perl https://github.com/perl11/cperl), and there's cperl mode: https://www.emacswiki.org/emacs/CPerlMode which is the better Perl mode for emacs. In a roundabout way, you're right in asking for cperl to be added. I don't know what adding an alias entails, though. It's basically a different implementation, but it uses the same name for the interpreter, "perl".
|
Oh right, I forgot there was a fork of Perl called "CPerl". 😀 (I remember being confused by timestamps before realising the major-mode was unrelated to the fork...) Will update. |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR enables recognition of Perl files containing cperl modelines:
Description
Emacs ships with an enhanced major-mode for editing Perl 5 with embedded C sections (called
cperl-mode, appropriately enough). One of the sample files added in #4066 reminded me about the mode, and I noticed it isn't being picked up by Linguist.Since the mode has to be manually activated per file (since it isn't the default), the odds of it being used in embedded modeline are pretty high.