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

CentOS7 EOL に向けたシステム要件の変更 #681

Closed
nanasess opened this issue Feb 15, 2023 · 10 comments
Closed

CentOS7 EOL に向けたシステム要件の変更 #681

nanasess opened this issue Feb 15, 2023 · 10 comments
Milestone

Comments

@nanasess
Copy link
Contributor

nanasess commented Feb 15, 2023

CentOS7 の EOL が 2024年6月30日となっているため、システム要件の下限バージョン変更を検討する。

  • PHP5.4.16 → PHP7.4.33

合わせて、 Smarty のバージョンアップを実施する(#615)

@nanasess
Copy link
Contributor Author

ちなみに、 RHEL9 の PHP8.0 であれば 2032年までサポートされる模様
image

@nobuhiko
Copy link
Contributor

nobuhiko commented Aug 7, 2023

システム要件の変更にあわせて、2.18にするとかもありですよね。

@nanasess
Copy link
Contributor Author

nanasess commented Aug 7, 2023

@nobuhiko はい、ありですね。

@seasoftjapan
Copy link
Contributor

PHP 5.4 対応の EC-CUBE 2.17.3 をリリースする計画が無ければ、master はこの Issue を反映して、PHP 8 対応を強化する体制に移行する方が良いのではないかと思いますが、どうでしょうか?

@nanasess
Copy link
Contributor Author

@seasoftjapan はい、今後は基本的にPHP8対応の強化で問題ないと思います。
master ブランチのシステム要件変更をマージするのは、 2024年6月30日以降で良いかなと思いましたが、もっと早めたい意向などありますか?

@seasoftjapan
Copy link
Contributor

現況、master や Weekly build で、新規構築しようと思うと PHP 8 で構成しにくいケースがあります。

すぐ思いつく範囲ですと、#615 (comment) ですが、なかなか独自に Smarty をアップデートしようとは思えず、PHP 7.4 を正式サポートする OS バージョンに落とすなど。

とりあえずは、#615 が早く取り込まれて欲しいとは思います。現況で直ちに master へマージという事にはならないと思いますが、安定次第マージされるという状況にあれば、#615 にも取り組み易いです。

ブランチ分けるのが定石とも思いますが、今後 PHP 5.4 対応のリリースが無いならば、そこに手間を割くまでも無いと考えた次第です。

@nanasess
Copy link
Contributor Author

@seasoftjapan
以下のような感じでいかがでしょうか?

  • master ブランチはシステム要件を変更し、 PHP7.4+ とする
  • 現在の master からPHP5.4互換のブランチを作成し、2024年6月末に最終版のビルドのリリースタグを作成する
    • 互換性に問題のない PR もマージしておく

@seasoftjapan
Copy link
Contributor

@nanasess はい。それでしたら、完璧だと思います。

@nanasess
Copy link
Contributor Author

nanasess commented Mar 4, 2024

@seasoftjapan
システム要件変更のPRしましたので、問題なければ approve おねがいします🙇‍♂️
#854

@nanasess
Copy link
Contributor Author

master ブランチは #854 で変更しましたのでクローズします

@nanasess nanasess added this to the 2.18(仮) milestone Dec 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants