Skip to content
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

gherkin-perl: Gherkin uses feature from perl5.12 but only requires perl5.10.1 language:perl #1494

Closed
jwrightecs opened this issue Apr 22, 2021 · 1 comment · Fixed by #1495

Comments

@jwrightecs
Copy link

gherkin-perl: Gherkin uses feature from perl5.12 but only requires perl5.10.1

Summary

When running gherkin on perl 5.10.1 (the minimum supported version), the "..." lines in Gherkin::Pickles::Compiler cause a syntax error, perl minimum version should be increased or '...' feature not used

Expected Behavior

'gherkin' command should parse and run and return ndjson formatted messages

Current Behavior

gherkin command fails with syntax error

$ gherkin
syntax error at /home/gherkin/perl5/lib/perl5/Gherkin/Pickles/Compiler.pm line 35, near "..."
syntax error at /home/gherkin/perl5/lib/perl5/Gherkin/Pickles/Compiler.pm line 171, near "..."
Compilation failed in require at /home/gherkin/perl5/lib/perl5/Gherkin.pm line 9.

Possible Solution

One of

  • Gherkin module should have 5.12 as the minimum version, and not be installable on 5.10.1, or
  • Gherkin::Pickles::Compiler should use 'die "Unimplemented";' instead of '...', making it 5.10.1 compatible

Steps to Reproduce (for bugs)

  1. perlbrew use 5.10.1
  2. cpanm Gherkin
  3. gherkin

Context & Motivation

Trying to use Gherkin to parse files on an old system (Centos 6, Perl 5.10.1)

Your Environment

  • Version used: Gherkin-18.1.1
  • Operating System and version: CentOS 6
  • Link to your project:
@jwrightecs jwrightecs changed the title gherkin-perl: Gherkin uses feature from perl5.12 but only requires perl5.10.1 language:perl gherkin-perl: Gherkin uses feature from perl5.12 but only requires perl5.10.1 language:perl Apr 22, 2021
@ehuelsmann
Copy link
Contributor

Thanks for the report! We'll get this fixed (back to 5.10.1) and organize a release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants