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

Make "level" type public #12

Open
GoogleCodeExporter opened this issue Apr 15, 2016 · 4 comments
Open

Make "level" type public #12

GoogleCodeExporter opened this issue Apr 15, 2016 · 4 comments

Comments

@GoogleCodeExporter
Copy link

It's currently impossible to maintain references to log4go level values outside 
the package.

For example, I want to have the log level configurable through a flag. To do 
this I map a symbolic name like "info" to a numeric integer level. With no 
ability to reference the level type, this is impossible.

Patch attached.

Original issue reported on code.google.com by a...@swapoff.org on 10 Jul 2012 at 6:36

Attachments:

@GoogleCodeExporter
Copy link
Author

Is there an expected update with the fix ?
If not, I will need to clone the project to use in our development.

Original comment by va...@percona.com on 31 May 2013 at 2:08

@GoogleCodeExporter
Copy link
Author

I have a fork here with the fix applied: https://github.com/alecthomas/log4go

I've been using it for the last year without issues.

Original comment by a...@swapoff.org on 31 May 2013 at 2:09

@GoogleCodeExporter
Copy link
Author

wow, great.
what about to merge fixes for 
https://code.google.com/p/log4go/issues/detail?id=11 ?
That one is annoying too.

Original comment by va...@percona.com on 31 May 2013 at 2:26

@GoogleCodeExporter
Copy link
Author

Good call, fixed.

Original comment by a...@swapoff.org on 31 May 2013 at 2:30

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant