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

将依赖用composer来管理 #1

Closed
wants to merge 2 commits into from
Closed

将依赖用composer来管理 #1

wants to merge 2 commits into from

Conversation

nauxliu
Copy link

@nauxliu nauxliu commented Jan 28, 2015

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

@johnlui
Copy link
Owner

johnlui commented Jan 28, 2015

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

@johnlui johnlui closed this Jan 28, 2015
@nauxliu
Copy link
Author

nauxliu commented Jan 28, 2015

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

@johnlui
Copy link
Owner

johnlui commented Jan 29, 2015

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

@nauxliu
Copy link
Author

nauxliu commented Jan 29, 2015

首先,我并不是针对你的包单独抽取出一个包。但是将你的包拆开本来就是我的目的,你的包作为一个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
#1 (comment).

@johnlui
Copy link
Owner

johnlui commented Jan 29, 2015

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

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

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

johnlui pushed a commit that referenced this pull request Oct 22, 2015
添加阿里云OSS中文件的删除,复制,移动函数
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

Successfully merging this pull request may close these issues.

2 participants