Skip to content

It does not work with Laravel 5.1 #31

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

Closed
bart opened this issue Jul 12, 2015 · 18 comments
Closed

It does not work with Laravel 5.1 #31

bart opened this issue Jul 12, 2015 · 18 comments

Comments

@bart
Copy link

bart commented Jul 12, 2015

Would you please update the dependencies to Laravel ~5.1? Thanks dude.

Getting:

toin0u/geocoder-laravel 0.5.0 requires illuminate/support 5.0.*
@bart
Copy link
Author

bart commented Jul 12, 2015

Okay it seems like you have to give packagist a refresh, because in https://packagist.org/packages/toin0u/geocoder-laravel#dev-master Laravel 5.0.* is set as requirement with the last update from 2015-04-16 07:50 UTC which is wrong.

@gilbertsmith
Copy link

Getting the same thing. Need the dependencies updated to 5.0 +

@martinbean
Copy link

+1.

Package needs updating on Packagist, as it’s still reporting illuminate/support 5.0.* is required, therefore I can’t install it in my 5.1.* Laravel application.

@richcoremedia
Copy link

waiting for the update at ~L5.1

@jakeryansmith
Copy link

+1

@martinbean
Copy link

@toin0u @willdurand Can one of you guys take a look at this when you get a couple of minutes spare, please?

@jbdivups
Copy link

+1

@martinbean
Copy link

What’s the hold up with this?

@bart
Copy link
Author

bart commented Jul 21, 2015

None of the main contributors does trigger a refresh on packagist. This is very annoying.

@mxkxf
Copy link

mxkxf commented Jul 22, 2015

+1

@willdurand
Copy link
Member

I don't have admin rights on Packagist. @toin0u is on vacations right now, so let's wait a bit.

@pbean
Copy link

pbean commented Jul 27, 2015

Hey folks

I'm on a tight timeline for a project and I needed this package so I put together a work around. I'll include it here hoping to help others if they absolutely need it since the git master branch already has the package dependencies updated.

add this above your require in your composer.json

"repositories": [
      {
        "type": "vcs",
        "url": "https://github.com/geocoder-php/GeocoderLaravel"
      }
    ],

This will pull from github instead once you have that placed you can then add the following as the package in require

"toin0u/geocoder-laravel": "dev-master",

That will pull the master branch from git and everything works okey dokey to bypass the composer package dependency. I ended up going down another route with everything for now and will probably circle back to using this later on this week but the few things i tested seemed to work.

Just remember to revert back to an appropriate version from packagist and remove the repository when everything is all updated.

@bart
Copy link
Author

bart commented Jul 28, 2015

Thanks for your reply @pbean. This is a good temporary workaround for sure, but I wanna wait for @toin0u being back from holidays.

@willdurand Do you know when he is back?

@willdurand
Copy link
Member

Ok folks, I've released 0.6.0. Let me know if it works for you.

@martinbean
Copy link

@willdurand The issue isn’t the version of GeocoderLaravel. The issue is it’s not been updated on Packagist: https://packagist.org/packages/toin0u/geocoder-laravel

If you look, it says it requires version 5.0.* of illuminate/support, which means it’s locked to Laravel version 5.0.* only.

@willdurand
Copy link
Member

Ok. It should be better now.

@bart
Copy link
Author

bart commented Jul 28, 2015

Looks good on packagist now, thanks a lot!

@martinbean
Copy link

W00t!

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

9 participants