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

Hangfire does not work under dnxcore50 #476

Closed
darcythomas opened this issue Nov 9, 2015 · 6 comments
Closed

Hangfire does not work under dnxcore50 #476

darcythomas opened this issue Nov 9, 2015 · 6 comments

Comments

@darcythomas
Copy link

Hangfire does not work under dnxcore50 .Net Core

To replicate:
New MVC5 beta8 project
Nuget: Install-package hangfire
Build <- fails

@odinserj
Copy link
Member

odinserj commented Nov 9, 2015

Hangfire doesn't support .NET Core framework yet. I've made some investigations under the dnx branch (it is a playground, and nothing else), and found some things that should be done:

  • All the dependencies should support .NET Core too. Some already support it, but there is a burden in package versions.
  • Dashboard should be abstracted away from OWIN to support both OWIN and new HTTP abstractions, since Microsoft.Owin package that is heavily used does not support DNX (I'm in doubt that it will support DNX).
  • Build scripts should be either modified, and it is interesting for me how to plug in ILMerge here.

@yngndrw
Copy link
Contributor

yngndrw commented Nov 9, 2015

Regarding ILMerge, please see this thread: aspnet/dnx#819

A long read (I only skimmed it), but it does look like there will be first-class support for merging in dependencies in the new system. It's still in the design stage so it might take a while to be realised, however.

@odinserj
Copy link
Member

odinserj commented Nov 9, 2015

Yep, I'm keeping an eye on this, and built-in support for internalizing would be awesome. But it is not implemented yet. Hope this will be available before ASP.NET 5 RTM.

@stajs
Copy link

stajs commented Nov 10, 2015

FWIW, it does work under dnx46 beta8.

@odinserj I'm confused by your comment about the Dashboard, have you changed your mind from this previous comment? Of course, no problem in changing your mind and great news if there is a better method to integrate with dnx. And as I said, it is working great under dnx46 beta8 so far 💃 by using your code snippet and referencing Microsoft.AspNet.Owin.

@odinserj
Copy link
Member

FWIW, it does work under dnx46 beta8.

Yep, because it is based on full framework.

@odinserj I'm confused by your comment about the Dashboard, have you changed your mind from this previous comment?

Yes, I was wrong. Dashboard code is heavily based on Microsoft.Owin, that does not support .NET Core. There are two ways – port Microsoft.Owin to .NET Core or use new abstractions. IMO it is better to choose the latter.

@odinserj
Copy link
Member

odinserj commented Jul 5, 2016

.NET Core support is almost here, see #604 🎉

@odinserj odinserj closed this as completed Jul 5, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

4 participants