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

Sample not working #6

Open
JalpeshVadgama opened this issue Mar 25, 2016 · 7 comments
Open

Sample not working #6

JalpeshVadgama opened this issue Mar 25, 2016 · 7 comments

Comments

@JalpeshVadgama
Copy link

Hi,

I have downloaded the sample source code from the GitHub. Also, I have checked that I have latest dnx version in my machine. See attached screenshot.

image

But still when I try to run this solution its giving following errors.

image

I think its not supporting dnx 4.5.1 or any other problem I have can you please suggest what I am doing wrong.

@VoronFX
Copy link

VoronFX commented Mar 26, 2016

I've just succeed to run sample by fixing some things. See my fork
VoronFX@75a2ff4

PS :
Cause my brain goes crazy from all this dependency hell, this fixes was made by trying different snippets I've found in threads. So I don't fully understand what actually changed but at least it working.
There is so few good examples, and all are broken in some way. I am starting to think that it's impossible to make all this work together. =( ... but still not going to give up.

@Icestorm0141
Copy link

@VoronFX thank you! That fixed the build!

Its worth nothing though, doesnt your fix downgrade this from .NET Core to .NET 4.5.1?

@VoronFX
Copy link

VoronFX commented Mar 28, 2016

Not sure.
They changing names and breaking things so fast that I am giving up on this.
Can waste no more time for understanding this.
I will stop on rc1. Sadly OpenIddict can't work on rc1, but it think it's not worth such pain.

@kevinchalet
Copy link
Contributor

@Icestorm0141 this sample app doesn't use .NET Core (dnxcore50 or netstandardapp1.5 in the recent builds, that will be replaced by netcoreapp1.0 in the near future) but OpenIddict fully supports it.

@VoronFX sadly, most of the tooling issues we're seeing are caused by the DNX -> CLI migration. It should get more stable when the RC2 tooling is released.

FWIW, I do my best to keep the aspnet-contrib/OpenIddict projects compatible with both worlds, but it's sometimes really painful and extremely time-consuming for me, specially when the issues are caused by the CoreFX or ASP.NET Core packages themselves.

Are you still seeing issues with the latest packages? Everything should work fine with both DNX and CLI ATM.

@Icestorm0141
Copy link

@PinpointTownes Doing a pull from this repo, the solution won't even compile. So yes, still seeing issues.

@kevinchalet
Copy link
Contributor

@Icestorm0141 I meant with the official OpenIddict samples, not this repo.
Since I have no write access for this repository, @capesean will have to fix it 😄

@kevinchalet
Copy link
Contributor

@Icestorm0141 I just sent a PR that should fix all the existing issues: #7.

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

4 participants