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

Install Notes #12

Closed
wants to merge 1 commit into from
Closed

Install Notes #12

wants to merge 1 commit into from

Conversation

KryptoKnight
Copy link

Adding a note to make sure people use the correct version of the mongo-c-driver

@razvancrainea
Copy link
Member

Hi, Mark!

The modules README files are automatically generated based on a docbook XSL file. This allows us to generate README files in different formats, like HTTP or PDF. The downside is that changing the README file directly is not a good idea, since it will be overwritten the next time the README will be regenerated.
Therefore, in order to make persistent changes (that are visible everywhere, including the website documentation), you'll have to edit directly the docbook file. For the cache_mongo module, you can find it in OpenSIPS sources at: modules/cachedb_mongodb/doc/cachedb_mongodb_admin.xml

Best regards,
Răzvan

@vladpaiu
Copy link
Member

Hello,

Thanks for the notice. It seems the latest Mongo C driver has a slightly changed API, but does not expose a specific VERSION macro, so we can't differentiate between various lib versions at compile time :(

I've added the mention of the 0.6 lib dependency in the module documentation.

Best Regards,
Vlad

@vladpaiu vladpaiu closed this Jul 10, 2013
@46labs 46labs mentioned this pull request Oct 7, 2013
@mnunzi mnunzi mentioned this pull request Nov 7, 2013
@apsaras apsaras mentioned this pull request Mar 26, 2014
@ankogan ankogan mentioned this pull request Feb 6, 2016
@apsaras apsaras mentioned this pull request Mar 5, 2016
@caloveri caloveri mentioned this pull request Jul 24, 2020
@adigeo adigeo mentioned this pull request Apr 6, 2021
@pdraganov1 pdraganov1 mentioned this pull request Feb 2, 2022
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.

None yet

3 participants