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

Support for CoreCLR #96

Closed
TobbenTM opened this issue Apr 1, 2016 · 26 comments
Closed

Support for CoreCLR #96

TobbenTM opened this issue Apr 1, 2016 · 26 comments

Comments

@TobbenTM
Copy link

TobbenTM commented Apr 1, 2016

With the latest release of AngleSharp, they now support the CoreCLR runtime (AngleSharp/AngleSharp#271), would it be possible for premailer to also include the target runtime?

Thanks

@ghost
Copy link

ghost commented Apr 21, 2016

@TobbenTM

Would you be interested in trying out my fork for coreclr rc1 compatibility? I compiled this against dnx451 and dnxcore50.

https://github.com/NickolasAcosta/PreMailer.Net/tree/coreclr-rc1

I need this for a personal project of mine, so I took some initiative. I'll get back to you with my results tommorow. All tests pass but do not run for dnxcore50 as moq is no longer compatible.

@TobbenTM
Copy link
Author

@NickolasAcosta

Sorry for the late response, finally got around to downloading and testing your build, works great! Have not run into any problems at all. Lets hope to get your functionality merged upstream

@martinnormark
Copy link
Contributor

@NickolasAcosta If all unit tests are passing, please create a Pull Requests and we can hopefully merge!

@ghost
Copy link

ghost commented May 10, 2016

@TobbenTM Glad it's working out for you!

@martinnormark
Unit tests do not execute on the "dnxcore50" runtime. This is because moq is not available for coreclr.
Unit tests do execute and pass on the "dnx451" runtime.

Moq is currently waiting on castlecore to update to rc2.
devlooped/moq#168

Would you like to wait for rc2?

@martinnormark
Copy link
Contributor

@NickolasAcosta I've created a dotnet-core branch. If you make a PR against that, I'll merge and publish a pre-release package on Nu Get.

@ghost
Copy link

ghost commented May 18, 2016

@TobbenTM Have you made the jump to RC2 yet?

@TobbenTM
Copy link
Author

@NickolasAcosta I have not, and probably won't have the time to migrate until after the summer unfortunately.

@TobbenTM
Copy link
Author

@NickolasAcosta I have successfully migrated my project to RC2, and your port of PreMailer.Net still works for all of my use-cases.

Furthermore, moq is now available and working great for .Net Core (RC2), though I have not run the tests for PreMailer.Net yet.

@ghost
Copy link

ghost commented Jun 24, 2016

Glad to hear it @TobbenTM . RTM is in a few days, so I will probably take a look at the new Moq this week.

@freykeld
Copy link

freykeld commented Jul 3, 2016

Any ETA on a version with CoreCLR 1.0.0 RTM support?

@ghost
Copy link

ghost commented Jul 5, 2016

I will attempt to upgrade that branch today after work. It will give me an excuse to play with the new moq.

@ghost
Copy link

ghost commented Jul 5, 2016

Current alpha version of Moq is not compatible with .net core rtm yet. Hang tight!

devlooped/moq#168

@freykeld
Copy link

freykeld commented Jul 5, 2016

@NickolasAcosta
Thanks for the update, i will keep a close on this issue.

@ghost
Copy link

ghost commented Jul 6, 2016

Try my fork. No unit tests, but there are no compiler errors either :)

Use at your own risk obviously.
https://github.com/NickolasAcosta/PreMailer.Net/tree/dotnet-core

Many libraries are in are still in RC2 and working on RTM support. Hopefully it all settles soon!

@freykeld
Copy link

freykeld commented Jul 6, 2016

@NickolasAcosta
Works fine for now, thanks.
My project is in a early alpha so this will do fine for at least a few months 👍

@freykeld
Copy link

freykeld commented Aug 2, 2016

Any news on this issue? I have to deploy first version of my site soon.

@martinnormark
Copy link
Contributor

martinnormark commented Aug 2, 2016

The PR is here: #122, still a few things to iron out.

martinnormark added a commit that referenced this issue Aug 3, 2016
martinnormark added a commit that referenced this issue Aug 3, 2016
martinnormark added a commit that referenced this issue Aug 3, 2016
martinnormark added a commit that referenced this issue Aug 3, 2016
martinnormark added a commit that referenced this issue Aug 3, 2016
@martinnormark martinnormark mentioned this issue Aug 3, 2016
4 tasks
@jakejscott
Copy link

@martinnormark Good shit, I'm just upgrading a project to .net core and this made my day!

@exiled78
Copy link

@superlogical - would you mind explaining how you included this in your .net core project? I am also migrating and am a bit new to this. I downloaded this branch and added it as a project to the solution, then added "PreMailer.Net": "1.6.0-*", to the dependencies in project.json in my startup project (Web Api).

Is this the best/only way?

I would rather not have the Premailer project sitting in my solution and would rather just use it as a nuget package, but unless I am mistaken, only v1.5.5 is available on nuget which fails when added.

Thanks!

@xavierpena
Copy link

xavierpena commented Feb 20, 2017

I tried @exiled78 solution and I got the following error:

Package PreMailer.Net 2.0.0-alpha is not compatible with netcoreapp1.0 (.NETCoreApp,Version=v1.0). Package PreMailer.Net 2.0.0-alpha supports: net45 (.NETFramework,Version=v4.5)
One or more packages are incompatible with .NETCoreApp,Version=v1.0.

Edit: what I did instead was

  1. Clone the dotnet-core branch of PreMailer.Net, using: git clone -b dotnet-core https://github.com/milkshakesoftware/PreMailer.Net

  2. Add the project PreMailer.Net to my solution

  3. Add a reference to this added project

And voilà, it works with .NET Core.

@exiled78
Copy link

@xavierpena yes, that is what I ended up doing also.

@ghost
Copy link

ghost commented Feb 25, 2017

For future reference, you can also directly download Nuget packages from this release, which adds support for .NET Core on master branch in a very simple way. Corresponding pull request is #141.

@alexzaytsev-newsroomly
Copy link

Any news here? as .net core 2.0.0 is available and nuget says there is a .Net Framework 4.5 dependency in AngleSharp.

@martinnormark
Copy link
Contributor

The latest pre-release version should work: https://www.nuget.org/packages/PreMailer.Net/2.0.0-beta2

@reinaldocoelho
Copy link
Contributor

This is still an open issue?
Whe have It working on NetStandard?

@martinnormark
Copy link
Contributor

Works on NETStandard now.

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 a pull request may close this issue.

8 participants