-
-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
RFC: Change examples to use async/await pattern + drop VS2010 compatibility #381
Comments
What about keeping the core libraries at .Net 4.0, creating a separate solution As it seems rare to add/remove projects from the solutions as it stands, maintaining two shouldn't be too much work. Thoughts? |
We could of course do it like that also, but it raises the bar a bit for contributing to the project. It's quite nice to just be able to open the .sln and hack away. Open question to everyone: How many of you are still using VS2010? (and have no access to VS2012 or 2013) Just to make it extremely explicit: 2010 users will still be able to use CefSharp as a consumer of the library (since it's .NET 4.0 for at least the near future), it's just that they won't be able to contribute to the project that might be a problem. |
VS2012 and VS2013 here, only use VS2010 for legacy stuff (doesn't include CefSharp) |
VS2010 here and no chance to change that anytime soon. |
personally i use vs 2013 so i dont mind dropping vs2010 support.
👍 it is up to the user to decide if they want to use await GetSourceAsync() or GetSourceAsync().ContinueWith() so i dont see any reason why we should upgrade cefsharp to 4.5 as long as we dont need any features of it. |
👍 for the same reasons as Jan nicely summed up + we still have the I think we can move this to 31.0.1 though - so that's how I marked it. It's not our contract, just polish of examples. |
I am using vs2010. |
@kropewnicki Are you using VS2010 to compile CefSharp from source? |
@amaitland Yes. |
@kropewnicki Are you simply building to get access to features in If your just obtaining a build of master, then we can probably look at providing |
@amaitland |
OK. So we have at least two people @ray007 and @kropewnicki that are still using VS2010. Any reason that you cannot upgrade to either VS2012 or 2013? I guess you don't have MSDN partner download access or similar? (I do myself) |
@kropewnicki What local modifications have you implemented? Is it something worth contributing to the project as a whole? IMHO |
@perlun My team is all using vs2010. our company does volume license for products, not MSDN. |
@kropewnicki, OK. Interesting to hear how popular VS2010 still is. Any way, we might still drop VS2010... What does the rest of you feel? How long (how many more years) is it worthwhile to support that one? Given that C++/CLI support is extremely poor in VS2010 (no intellisense), I do not think it makes that much sense to support it for active development. If we decide to keep it, it's more a matter of being nice to people like @kropewnicki and @ray007 (and others). But it has to be weighed against the cost of maintaining the support (which is always greater than zero, which you sometimes tend to forget - TANSTAAFL). |
I mean, we did support VS2008 just some year ago, until it was dropped... Maybe the time has come to also pull the plug on VS2010, since it actually lacks things very useful to us ( |
@perlun I was thinking about this issue this morning. I was wondering about if you could drop vs2010, but add a minimal vs2010 example, that uses nuget to get the nightly build for cefsharp. |
The project I am trying to integrate cefsharp into uses VS2010. Part of a much bigger project. No signs that the team here is moving or can move to VS2012 or 13 anytime soon. |
The only problem with this is that:
So, it's definitely doable, but these preconditions need to be met at the moment. I don't know, I have a feeling that it's more important to get 31.0.0 out the door asap rather than spend time on the nightlies at the moment. What do you feel? |
That is not a problem. We are not (and I repeat, not) speaking about upgrading CefSharp to in any way require .NET 4.5/VS2012/VS2013 as a "consumer" of the project. We are merely talking about our own .csproj/.sln files; it would make sense to start migrating the examples etc. to .NET 4.5 because it has useful features that we would make good use of being able to recommend to our users as the kind of "best practices". I say 👍 for bumping the examples + starting to use async/await for them. It will still be possible to consume stuff using VS2010, it just requires a bit more "understanding how stuff works" for the VS2010 users (because they won't be able to copy/paste code from our examples as easily). Fair enough to me - it's soon 2 years since VS2012 was being released so kind of "optimizing for VS2012+" seems very reasonable for me. So, in other words: support for VS2010 as a mere consumer of CefSharp will remain, but CefSharp itself will be upgraded to only support 2012 and 2013 (which simplifies things for us since we don't have to build as many different |
For version It's ticked over to http://www.visualstudio.com/en-us/news/vs2013-community-vs.aspx |
Commit amaitland@fc65192 resolves this Issue. |
Great, thanks! I think we should not bump the core libraries to .Net 4.5 yet, unless it really causes problems for us to have them on 4.0. As for me - has been extremely unavailable from the project for the last few months. Will maybe come in for a minor return within the next few months, since we will start upgrading to CefSharp3 (i.e. 37 or 39 or whatever the version is nowadays. 😄) But anyway, don't count on me as a "major" contributor this year, but more a minor collaborator amongst the peers. |
Agree! (on the 4.0 vs 4.5 situation) Anyho' good to "hear" from you @perlun 👍 ... I'm also still mostly lurking in the background 😵 and @amaitland + a few other semi-regular contributors have recently done a lot of good work! |
Hi,
One question that has arisen after the discussion we had in #373 (as to the "be or not to be" for a locking-synchronous-Task-Wait-based approach), and the suggested approach for these kind of things for the future, is the way we we should recommend that these kind of things be done.
I mean, normally if you would have a method like this:
...which you would then call perhaps like this:
All fine and dandy. Only one problem:
In any way, I feel that since this is the "preferred approach" that we want to recommend people to use, it would be good if we would use it ourselves. 😃 So, here's my recommendation:
async
. Note: only the example projects; CefSharp still needs to be .NET 4.0 for a little while, since there might be user's who cannot upgrade to .NET 4.5 so easily (I happen to be one of them. 😉)Comments?
The text was updated successfully, but these errors were encountered: