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

Upgrade NLog to 4.5 RTM for .NET Core support #200

Closed
jonorossi opened this issue Jul 11, 2016 · 18 comments
Closed

Upgrade NLog to 4.5 RTM for .NET Core support #200

jonorossi opened this issue Jul 11, 2016 · 18 comments
Milestone

Comments

@jonorossi
Copy link
Member

We are currently using NLog 4.4.0-beta13 to get .NET Core support, we need to upgrade to an RTM version once it is released.

@jonorossi jonorossi added this to the v4.0 milestone Jul 11, 2016
@jonorossi
Copy link
Member Author

The main NLog .NET Core issue for 4.4 that hasn't been closed is NLog/NLog#641.

@jonorossi
Copy link
Member Author

New main issue for .NET Core support in NLog: NLog/NLog#1559

@jonorossi
Copy link
Member Author

The NLog guys have gone quiet since July so it might be time to drop NLog 4.4 beta support with .NET Core support from Castle Core 4.0 and revert back to NLog 4.3.x. It'll unblock us from not being able to release a 4.0 final.

Thoughts?

@jonorossi
Copy link
Member Author

jonorossi commented Nov 14, 2016

NLog 4.x won't support .NET Core, that support will come in NLog 5.x because there are missing APIs that they have decided to wait until .NET Standard 2.0 before releasing something that will work, this won't happen until Q1 2017.

http://nlog-project.org/2016/10/30/nlog-roadmap-netcore-structural-logging.html

Decision made, we'll revert to the stable NLog, 4.3.x, which unblocks us to release Castle Core v4 final. Will remove this task from the milestone when the change is in.

@jonorossi jonorossi changed the title Upgrade NLog to 4.4.0 RTM Upgrade NLog to 5.x RTM for .NET Core support Nov 14, 2016
@DixonDs
Copy link

DixonDs commented Jan 9, 2017

Are there any estimates when Castle.Core 4.0 RTM will be released? We have a couple of libraries depended on it, which we keep as prerelease mainly because they have prerelease dependency Castle.Core

@jonorossi
Copy link
Member Author

@DixonDs are you using NLog? This issue is about supporting NLog on .NET Core.

@DixonDs
Copy link

DixonDs commented Jan 9, 2017

@jonorossi Sorry, that was probably a wrong issue to ask that question. I've just got an impression that it is the only outstanding issue that holding Castle.Core 4.0 release. Do you happen to have gitter.im channel?

@jonorossi
Copy link
Member Author

@DixonDs no problem, nope no active chat room. I just added a log4net issue into the milestone too. I've just been swamped for the last few weeks so pull requests on these last two remaining issues would allow me to cut the release.

@thomaslevesque
Copy link
Contributor

Hi @jonorossi, do you plan to release the logging integration packages along with the main library for 4.0? From what I understand, it might be a while before NLog is ready for .NET Core, and none of the integration projects have been migrated to .NET Core yet. It might be a good idea to release Castle.Core 4.0 alone for now, and release the integration packages later. What do you think?

(If I seem eager to see Castle.Core 4.0 released, it's because I am 😉... it's about the only thing keeping us from releasing FakeItEasy 3.0)

@jonorossi
Copy link
Member Author

@thomaslevesque yes, logging integration packages will be released as they have been in the betas. As per my comment on Nov 15th, this issue is only still in the milestone until we revert to a stable NLog for .NET 4.x.

Sorry about not getting the final release out, I kept snoozing Blair's GitHub comment notification email from last month where he said you guys weren't in a rush for it, didn't know you were ready to release. I'll try to get it sorted tomorrow.

@thomaslevesque
Copy link
Contributor

@jonorossi thanks! I don't want to rush you, I was just wondering if something was keeping you from releasing.

@jonorossi
Copy link
Member Author

@thomaslevesque no problem, I was hoping to get it done around Christmas. Just have to make the time to sort out the 3 remaining issues and cut the release.

@blairconrad
Copy link
Contributor

You're good folk, @jonorossi.

@jonorossi
Copy link
Member Author

NLog 4.4.0-beta13 is now unlisted and the 4.4.x line no longer supports .NET Core. I've upgraded to NLog 4.4.1 for the .NET Framework targets.

Removing this issue from our v4.0 milestone.

@jonorossi jonorossi removed this from the v4.0 milestone Jan 24, 2017
@ghost
Copy link

ghost commented May 9, 2017

Looks like this one is back in play: https://www.nuget.org/packages/NLog/5.0.0-beta07

Should we jump after: #241

@jonorossi
Copy link
Member Author

Looks like this one is back in play: https://www.nuget.org/packages/NLog/5.0.0-beta07

Should we jump after: #241

I think we should wait until the final version so we don't get blocked releasing the next version with a beta version for .NET Framework.

@jonorossi
Copy link
Member Author

NLog 4.5 is RTM! NLog 4.5 adds structured logging and .NET Standard support/UPW without breaking changes!
(https://twitter.com/NLogOfficial/status/978038367903539200)

@jonorossi jonorossi changed the title Upgrade NLog to 5.x RTM for .NET Core support Upgrade NLog to 4.5 RTM for .NET Core support Mar 27, 2018
@snakefoot
Copy link
Contributor

Created PR #340

@jonorossi jonorossi added this to the vNext milestone Apr 20, 2018
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

5 participants