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

Drop support for Perl 5.8.x #620

Closed
ikedas opened this issue May 19, 2019 · 6 comments
Closed

Drop support for Perl 5.8.x #620

ikedas opened this issue May 19, 2019 · 6 comments
Labels
Milestone

Comments

@ikedas
Copy link
Member

ikedas commented May 19, 2019

Expected Behavior

Sympa will support Perl 5.10.1 or later.

Current Behavior

Sympa has to support Perl 5.8.1 or later.

@racke
Copy link
Contributor

racke commented May 19, 2019

Facts: Even Perl 5.10 is more than 11 years old. CentOS 6 comes with Perl 5.10. So there is really no need to support anything older.

@ikedas
Copy link
Member Author

ikedas commented May 21, 2019

Precisely, Perl 5.10.0 was a version a bit messed. We'd be better to abandon it (RHEL/CentOS have done).

@racke
Copy link
Contributor

racke commented May 21, 2019

I don't really care for old Perl versions, what would be your suggestion for the minimum Perl version? And if we decide on one, I suggest that we inform the community beforehand the next release.

@ikedas
Copy link
Member Author

ikedas commented May 23, 2019

As I wrote at first, 5.10.1 is my suggestion.

@ikedas ikedas pinned this issue Jun 10, 2019
@ldidry
Copy link
Contributor

ldidry commented Jun 21, 2019

I agree. One of the strength of Sympa is that it supports old Perl but dropping 5.8 is kind of needed if we want to be able to use some modern (11 years old though, but it’s better than 5.8 which is 17 years old) Perl syntax, but there is no rush to jump on cutting-edge Perl versions. 5.10.1 is enough for now.

@ikedas ikedas added this to the 6.2.46 milestone Jul 7, 2019
ikedas added a commit that referenced this issue Jul 16, 2019
@ikedas
Copy link
Member Author

ikedas commented Jul 16, 2019

A patch merged. This issue will be closed when the next beta (maybe 6.2.45b.1) builds sanely.

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

No branches or pull requests

3 participants