richterger / Perl-LanguageServer

Language Server for Perl
Other
222 stars 53 forks source link

minimum version of perl 5.16 not 5.18 #110

Closed willt closed 1 year ago

willt commented 2 years ago

I believe this works with perl 5.16 just fine. Just change 5.18 to 5.16 and remove the no experiential warnings. This will help people forced to be stuck in old perl (redhat/company policy/etc) Thoughts?

finwarman commented 2 years ago

I've created a fork of this module with Perl v5.16 compatibility, if anyone unsure how to edit it for themselves:

https://github.com/finwarman/perl-5.16-Perl-LanguageServer/releases/tag/V2.3.0

bgeels commented 1 year ago

I thought for a second this would work with 5.16 as well. Got it installed with @finwarman's fork. While it mostly works, I found that the debugger didn't actually work. It would claim that the DB::DB subroutine was not defined.

The earliest version of perl I got working with the debugger was 5.26.3 ( earlier versions may work, haven't tested many but 5.18 had the same issue as 5.16 ).

What I ended up doing for working on an old el7 system with 5.16 that seems to have worked is...

Development Server Setup

VSCode / Client Setup

Now in the Settings for the VSCode Perl Extension...

That seems to have gotten things working. Obviously this means that you're debugging with a different perl interpreter than the project would be using in production which could lead to interesting surprises.

richterger commented 1 year ago

Fixed in 2.5.0