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

MongoCSharpDriver 1.4.2 now has fluent extensions #1

Open
gavinkilbride opened this issue May 10, 2012 · 6 comments
Open

MongoCSharpDriver 1.4.2 now has fluent extensions #1

gavinkilbride opened this issue May 10, 2012 · 6 comments

Comments

@gavinkilbride
Copy link
Contributor

I updated the assemblies for the MongoCSharpDriver and FluentMongo but this broke the unit tests. FluentMongo had a dependency on version 1.3.1. I therefore removed the reference to FluentMongo and used the MongoCSharpDriver extension methods for chaining (no code change, just references) and it all works beautifully. Just thought I'd let you know in case someone else did the same. Great work btw and will let you know how my little project goes, if interested :) This has just saved me days of work. Thank you very much.

@alanning
Copy link
Owner

Thanks Gavin. Glad you found it useful!

I'm actually on vacation now... Do you think you could submit a pull
request with your changes? :-)

Out of curiosity would a Nuget package be useful? I have one for a
related project, MongoWSAT (https://nuget.org/packages/MongoWSAT) but
not for this one.

On Thu, May 10, 2012 at 9:07 AM, Gavin Kilbride
[email protected]
wrote:

I updated the assemblies for the MongoCSharpDriver and FluentMongo but this broke the unit tests. FluentMongo had a dependency on version 1.3.1. I therefore removed the reference to FluentMongo and used the MongoCSharpDriver extension methods for chaining (no code change, just references) and it all works beautifully. Just thought I'd let you know in case someone else did the same. Great work btw and will let you know how my little project goes, if interested :) This has just saved me days of work. Thank you very much.


Reply to this email directly or view it on GitHub:
#1

@gavinkilbride
Copy link
Contributor Author

Hi Adrian
Us Microsoft developers love it when our life is made easy so yes a Nuget package would be most welcome. I suspect a few more people may stumble across it this way too. The first thing I do when looking for a third party solution is search Nuget. It was only when finding nothing suitable on Nuget that I Googled "Mongo Role Provider".

Although I've never done one before I'll happily do a pull request, it's a trivial change. Thanks for your efforts.

Gav

@alanning
Copy link
Owner

Hi Gav,

I'm back from vacation and just merged in your pull-request. Thank you!

Was hoping to get your take on including the mongo driver dll directly vs. using a nuget dependency. It is currently included directly with the rationale being that it makes the installation faster. The nuget package for the mongo driver pulls down a lot of unnecessary stuff so it takes quite a while. It would simplify the MongoProviders Nuget package to use a dependency, though, so I'm leaning towards changing it.

Another consideration is that most dev's using the MongoProviders package would probably already have the mongo driver package for use with their regular db storage code.

What's your feeling?

@efosao
Copy link

efosao commented May 18, 2012

Many thanks for this project Adrian,

It's been a great time-saver!

Just to chime in on that question, I'd recommend using a 'reference' (rather than embedding the dll) for the nuget package.

  • The reason being that if you are using nuget to manage the MongoProvider, odds are that you also used it to install the Mongo driver in the first place, and also
  • While having the mongo driver dll might get you started faster, it might be an inconvenience down the road when updating the components separately.

@alanning
Copy link
Owner

Hi Efosa,

Thanks for the input and I'm glad this was able to save you some time!
I agree with your points so that's what we'll do. I'll post back
here once the Nuget package is up (early next week).

Cheers!

On Fri, May 18, 2012 at 5:08 PM, Efosa Oyegun
[email protected]
wrote:

Many thanks for this project Adrian,

It's been a great time-saver!

Just to chime in on that question, I'd recommend using a 'reference' (rather than embedding the dll) for the nuget package.

  • The reason being that if you are using nuget to manage the MongoProvider, odds are that you also used it to install the Mongo driver in the first place, and also
  • While having the mongo driver dll might get you started faster, it might be an inconvenience down the road when updating the components separately.

Reply to this email directly or view it on GitHub:
#1 (comment)

@efosao
Copy link

efosao commented May 19, 2012

Thanks Adrian!

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

3 participants