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

pear-pear.php.net to packagist #241

Merged

Conversation

nanasess
Copy link
Contributor

@nanasess nanasess commented Feb 5, 2019

pear-pear.php.net を packagist のリポジトリへ変更

  • pear/* が存在するパッケージは、そのまま利用する
  • silverorange/mdb2 - 元のパッケージと互換性のある 2.5.2-b.5 を利用
  • bondas83/mdb2_driver_mysqli - 元のパッケージと互換性のある 1.5.3 を利用
  • nanasess/mdb2_driver_pgsql - Packagist に存在しないため、 pear-pear.php.net のソースを元に新規作成
  • nanasess/net_useragent_mobile - Packagist に存在しないため https://github.com/pear/Net_UserAgent_Mobile の trunk から新規作成
    • 本体のコードでは利用されていない(到達しないコードで利用されている)
  • nanasess/services_json - Packagist に存在しないため https://github.com/pear/Services_JSON の trunk から新規作成
    • json_* 関数が存在しない場合のみ利用される

@nobuhiko
Copy link
Contributor

nobuhiko commented Feb 5, 2019

  • json_* 関数が存在しない ってPHP5.4以降でありえるんでしたっけ?

@nobuhiko
Copy link
Contributor

nobuhiko commented Feb 5, 2019

  • Net_UserAgent_Mobile はガラケー対応をなくしているので、不要になっているかもです

@coveralls
Copy link

coveralls commented Feb 5, 2019

Coverage Status

Coverage increased (+0.001%) to 22.497% when pulling 554fe6d on nanasess:deprecated-pear-repositories into 98e4ffe on EC-CUBE:improve/php7.

@nanasess
Copy link
Contributor Author

nanasess commented Feb 5, 2019

@nobuhiko Services_JSON も、 Net_UserAgent_Mobile も、 本体に依存するコードが残ってるのでパッケージ作成しました。本当は消してしまいたいんですけどね。。。

@nobuhiko
Copy link
Contributor

nobuhiko commented Feb 6, 2019

消しちゃいましょう〜

@chihiro-adachi chihiro-adachi added this to the 2.17.0 milestone Feb 6, 2019
@chihiro-adachi
Copy link
Contributor

@nanasess @nobuhiko
不要ライブラリの削除、このPull Requestで続けます?
もしくは別Issueにしてこちらはいったんマージしてしまいましょうか?

@nanasess
Copy link
Contributor Author

nanasess commented Feb 8, 2019

@chihiro-adachi Net_UserAgent_Mobile は、モバイルサイトのほうのコードを結構多めに修正しないといけなかったと記憶しているので、別にしましょ

@chihiro-adachi
Copy link
Contributor

@nanasess
ありがとうございます。Issue立てておきました。#242

@chihiro-adachi
Copy link
Contributor

@nobuhiko @seasoftjapan
修正内容問題なければ、Approvedおねがいしますー

@nobuhiko nobuhiko self-requested a review February 8, 2019 08:33
@chihiro-adachi
Copy link
Contributor

間違えてcloseしてしまいました。
テスト通りしだいマージします。

@chihiro-adachi chihiro-adachi merged commit 390bde4 into EC-CUBE:improve/php7 Feb 12, 2019
@nanasess nanasess deleted the deprecated-pear-repositories branch May 23, 2019 03:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants