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

Removed explicit dependency to logging plugin so people can change the logging framework #1

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

fterrier
Copy link

When following the instructions in the Grails guide (http://grails.org/doc/2.0.x/guide/conf.html#logging), run-app fails because of that explicit dependency.

To reproduce, add to an app using the cache-headers plugin:
...
inherits("global") {
excludes "grails-plugin-logging", "log4j"
}

and then grails run-app

@jarias
Copy link

jarias commented Jun 19, 2012

Any idea why this patch hasn't been merge? I also need this, is this plugin still maintain?

@blemoine
Copy link

Hi,
our team also needs to remove the logging plugin, can you merge this pull-request please ?

@aurelienmaury
Copy link

Yes, and make a little release of it so we can update from central. Please, please, please

@literalice
Copy link

+1, for the grails logback plugin - http://grails.org/plugin/logback

@ecowden
Copy link

ecowden commented Apr 23, 2013

+1. This is a major problem for us.

@graemerocher
Copy link

It has been merged into https://github.com/grails-plugins/grails-cache-headers which is the new home for cache-headers

rosenfeld pushed a commit to rosenfeld/grails-cache-headers that referenced this pull request Oct 16, 2013
Removed explicit dependency to logging plugin so people can change the logging framework
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 this pull request may close these issues.

7 participants