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

Enhancement: Remove Dependency on log4Net #19

Open
ferventcoder opened this issue Sep 22, 2011 · 5 comments
Open

Enhancement: Remove Dependency on log4Net #19

ferventcoder opened this issue Sep 22, 2011 · 5 comments
Assignees
Labels

Comments

@ferventcoder
Copy link
Member

RH should use log4net if it is there, but not depend on it. This applies only to the DLL versions of roundhouse (msbuild and lib)

@jwynveen
Copy link

Related to this is that the version of log4net seems to be explicitly set in Roundhouse.Tasks. I have 1.2.11 in my project due to another dependency, but it appears Roundhouse.Tasks required 1.2.10 because a specific version is set in the project file.

How soon until this might be updated? I can't build my site with Roundhouse in it at this time.

@ferventcoder
Copy link
Member Author

I would move the roundhouse dependencies you have off to another project in the meantime. I have plans to finish this in the second quarter.

@Liwoj
Copy link
Contributor

Liwoj commented Mar 9, 2018

Hi
I personally don't have a problem with log4net dependency (i'm only using console app)

Just wanted to mention there is pretty good way of abstracting logging more and more libraries are using and thats LibLog

@BiggerNoise
Copy link
Member

Thanks for the heads up.

@erikbra
Copy link
Member

erikbra commented Oct 13, 2019

I think migrating to using Microsoft.Extensions.Logging.ILogger would be the preferred way to go now. Combined with proper structured logging (e.g using Serilog).

@erikbra erikbra added the Logging Related to logging label Jul 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

5 participants