Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix failures on older Perl versions #15

Merged
merged 5 commits into from
Oct 18, 2021

Conversation

ehuelsmann
Copy link
Contributor

@ehuelsmann ehuelsmann commented Oct 16, 2021

Description

Address most -if not all- failures on http://matrix.cpantesters.org/?dist=Cucumber-TagExpressions+4.1.0

Motivation & context

The module declares a minimum Perl version of 5.14 which is true, on code level, but to make it true for the distribution, it needs to declare its dependencies better. Also, for the tests to succeed, the syntax needed to be downgraded.

Type of change

  • Bug fix (non-breaking change which fixes an issue)

Checklist:

  • I have read the CONTRIBUTING document.
  • My code follows the code style of this project.
  • I have added tests to cover my changes.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have added an entry to the "Unreleased" section of the CHANGELOG, linking to this pull request.

Although List::Util is a core module (better: it's dual life), it only grew support for 'any' and 'all'
in 1.33, which doesn't come with some of the older Perl versions. Specifying the number explicitly
makes older Perls install this module from CPAN.
Copy link
Contributor

@aurelien-reeves aurelien-reeves left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good

We have to find out why the python build is failing on macos, but maybe that could be part of another PR?

@jenisys
Copy link
Contributor

jenisys commented Oct 18, 2021

@aurelien-reeves
I will look into it later today.

The problem with python/macOS seems to an OOPS in the pipenv tooling.
Maybe a newer version would help: pipenv==2015.5.29 (newest release).

BACKGROUND:
The pipenv tool contains other tools as vendor package.
One of these either contains errors or is outdated.
The class collections.MutableMapping is available in Python2.
In Python3 it was moved to collections.abc.MutableMapping.
Normally, you use the six package to provide a solution that works both in Python2 and Python3.
For whatever reason that was not done for the problematic package/patched part.

SEE ALSO:

@jenisys jenisys mentioned this pull request Oct 18, 2021
6 tasks
@aslakhellesoy aslakhellesoy merged commit 73db2a8 into cucumber:main Oct 18, 2021
netbsd-srcmastr pushed a commit to NetBSD/pkgsrc that referenced this pull request Dec 17, 2023
5.0.0 (2023-01-02)
Added
* [JavaScript] Support for EcmaScript modules (aka ESM). (#5)
* [Java] Enabled reproducible builds
Changed
* Only allow escape character \ in front of (, ), \ or whitespace. Throw
  error otherwise. (#17)
Fixed
* [Perl] Fixed missing dependency as well as new syntax in the tests
  (cucumber/tag-expressions#15 ehuelsmann)
* Document escaping. (#16, #17)
* [Ruby], [Perl] Empty expression evaluates to true
* [Go] Fix module name (#82)

5.0.1 (2023-01-03)
Fixed
* [Java] Fix scm and project urls

5.0.2 (2023-07-15)
Added
* [Python] Make tests pass against shared test data (except:
  backslash-escaped) (#18 by jenisys)
Fixed
* [All] Not.to_string() conversion has unneeded double-parenthesis if binary
  operator is contained (#94 by jenisys)

5.0.3 (2023-08-10)
Fixed
* [Perl] Fixed CPAN upload in release process (by ehuelsmann)

5.0.4 (2023-08-10)
Fixed
* [Perl] Corrected working directory for CPAN upload action (by ehuelsmann)

5.0.5 (2023-08-11)
Fixed
* [Perl] Fixed inclusion of CHANGELOG.md causing release to fail (by
  ehuelsmann)

5.0.6 (2023-08-13)
Fixed
* [Perl] Fixed test failures when running tests out-of-tree by resticting
  testdata based tests to run in development only (by ehuelsmann)

6.0.0 (2023-10-13)
Added
* [Ruby] Initial rubocop gems and basic compliance added (More work to come)
  (#133)
Changed
* [Ruby] Minimum ruby version is now bumped from 1.9 to 2.3 (#133)
Fixed
* [Perl] Include README.md and LICENSE in the release tarball (by ehuelsmann)
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants