search

zendframework / zend-log

Log component from Zend Framework
BSD 3-Clause "New" or "Revised" License
62 stars 51 forks source link

add mongodb writer to writer plugin manager #38

Closed prolic closed 8 years ago

prolic commented 8 years ago

resolves https://github.com/zendframework/zend-log/issues/37

weierophinney commented 8 years ago

Ouch... there's another problem.

Evidently, Travis is installing ext/mongodb, vs ext/mongo, with current builds. ext/mongodb does not ship the MongoClient or Mongo classes that were present in the various 1.* builds. As such, tests currently do not run.

I'm not quite sure how to resolve this, but I'm a bit reluctant to expose the mongo writer unless we know it also works with ext/mongodb. Thoughts?

prolic commented 8 years ago

I'd say let's make two separate writers: 1) mongo 2) mongodb for ext-mongo and ext-mongodb.

prolic commented 8 years ago

About travis: I can install both extensions for travis, this is no big deal.

sandrokeil commented 8 years ago

I guess we run into the same problems as mentioned in the zend-session component. We should use an extra repo (e.g. zend-log-mongodb) for specific writer, which relies on dependencies.

prolic commented 8 years ago

@weierophinney If you want me to change this to separate repos, please create those repos for me so I can make the split. Otherwise it should be ready to merge.

samsonasik commented 8 years ago

ping @weierophinney

weierophinney commented 8 years ago

Merged to develop for release with 2.9.0.