Skip to content
This repository has been archived by the owner on Oct 27, 2020. It is now read-only.

Added the possibility to specify which mmdb to download during composer install #6

Merged
merged 3 commits into from
Feb 11, 2017

Conversation

jaumebosch
Copy link
Contributor

@jaumebosch jaumebosch commented Feb 6, 2017

fixed non-existing path

composer.json Outdated
@@ -32,6 +37,7 @@
},
"minimum-stability": "stable",
"extra": {
"maxmind-db-path": "Tests/GeoLite2-Country.mmdb"
"maxmind-db-path": "app/geolite2.mmdb",
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

that's the problem of failed tests. app folder doesn't exist + bug in \Insomnia\MaxMindGeoIpBundle\Command\LoadDatabaseCommand (check 17dc4df for fix)

By default extra is ignored in root composer.json so it's safe to leave it as Tests/geolite2.mmdb

@mente mente merged commit 0ab29e0 into mente:master Feb 11, 2017
@rvanlaak rvanlaak mentioned this pull request Feb 23, 2017
rvanlaak added a commit to rvanlaak/MaxMindGeoIpBundle that referenced this pull request Feb 23, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants