search

johnlui / AliyunOSS

阿里云 OSS 官方 SDK 的 Composer 封装，支持任何 PHP 项目，包括 Laravel、Symfony、TinyLara 等等。
489 stars 187 forks source link

将依赖用composer来管理 #1

Closed nauxliu closed 9 years ago

nauxliu commented 9 years ago

删除lib目录，使用composer来管理这部分依赖。 naux\oss升级了依赖的包版本，并修复了官方sdk的一点儿小bug

johnlui commented 9 years ago

你可知道你的这次行为将会永远留在互联网上？

nauxliu commented 9 years ago

抱歉，你什么意思我没看懂。 我不过是将OSS依赖的lib guzzle升级成了3.9。并修复了官方SDK listObjects函数的一个bug而已。没觉得这个pull request有什么不合理的地方。

johnlui commented 9 years ago

修复 bug 是合理的，但是抽离出来一个你自己的包并且把我的包肢解是不合理的。

nauxliu commented 9 years ago

首先，我并不是针对你的包单独抽取出一个包。但是将你的包拆开本来就是我的目的，你的包作为一个composer包，包来依赖也理应用composer来管理（OSS SDK，Guzzle），好处你自然也知道。然后，composer本身提供类加载器，把 Symfony\Component\ClassLoader 去掉也是理所当然，没必要多引入一个类加载器。 最后，就算你不接受我引入我自己的包，拒绝这次pr，你也可以说明原因拒绝掉。那你说我的行为会永远留在互联网上是什么意思，我的什么行为？ 我偷了，我抢了？为什么一上来就人格攻击？知道这句话有多伤人？

2015-01-29 9:44 GMT+08:00 JohnLui notifications@github.com:

修复 bug 是合理的，但是抽离出来一个你自己的包并且把我的包肢解是不合理的。

— Reply to this email directly or view it on GitHub https://github.com/johnlui/AliyunOSS/pull/1#issuecomment-71954230.

johnlui commented 9 years ago

我的包是一个整体。我对阿里云的核心包进行了部分修改。你可知道你抽离出的代码中有我的工作？

我知道我说的话有多伤人，就跟我看到你提交这个 pull 时候的感觉一样。

我认为你的行为是不能容忍的，所以喷了一下你，如果你认为这就是人身攻击，那我向你道歉，只是提醒一句你以后肯定还会遇到许多这样的“人身攻击”，请先做好心理准备。