search

frostealth / yii2-aws-s3

An Amazon S3 component for Yii2
MIT License
91 stars 56 forks source link

Requires PHP7 #11

Closed newscloud closed 8 years ago

newscloud commented 8 years ago

Cool library. Lots of potential.

I received this error. Do you really need PHP7 for this? I was unable to install with satisfiable names below.

Your requirements could not be resolved to an installable set of packages.

Problem 1

frostealth commented 8 years ago

There are used scalar type hinting, return type declarations and null coalescing operator. Why don't you update to the latest version?

davidjeddy commented 8 years ago

@frostealth Large scale application take months to years to update the under lying architecture.

@newscloud Recommend using the v1 version. As per semver when the first place digit changes that means backwards breaking changes. PHP5 -> 7 BC changes. yii2-aws-s3 1.x -> 2.x : BC changes. I feel your pain as v2 has some nice changes but the system this is being used on is only PHP 5.6.

frostealth commented 8 years ago

@davidjeddy thank you for your detailed answer for @newscloud 👍

frostealth commented 8 years ago

@newscloud I agree with @davidjeddy and also recommend you to use the first version for your projects with php5.

newscloud commented 8 years ago

@davidjeddy thanks. what's the version number for v1? or is it in a different repo? I don't see it clearly.

davidjeddy commented 8 years ago

@newscloud Here you go: screen shot 2016-04-14 at 13 19 39

frostealth commented 8 years ago

@newscloud v1.0.0 on packagist and in this repo.

newscloud commented 8 years ago

Got it! Thank you!

On Thu, Apr 14, 2016 at 10:20 AM, Ivan Kudinov notifications@github.com wrote:

@newscloud https://github.com/newscloud v1.0.0 on packagist https://packagist.org/packages/frostealth/yii2-aws-s3#v1.0.0 and in this repo https://github.com/frostealth/yii2-aws-s3/tree/1.x.

— You are receiving this because you were mentioned. Reply to this email directly or view it on GitHub https://github.com/frostealth/yii2-aws-s3/issues/11#issuecomment-210059087

Jeff Reifman http://lookahead.io http://jeffreifman.com