On Thu, Feb 25, 2010 at 10:48 AM, Ross Mellgren <[email protected]> wrote:

> My concern about putting this in LiftRules is that not all applications
> that could benefit from the new logging support will use webkit, since the
> logging is in common.
>
> Maybe a bridge method in LiftRules to put it in a convenient place, but I
> think the actual work should be exposed nearby the logging library.
>

I think this is correct.  If the logging stuff is going to live in Common,
then it should be usable just by importing common without any dependence on
WebKit.


>
> -Ross
>
> On Feb 25, 2010, at 1:42 PM, Marius wrote:
>
> > Why not? LiftRules is about configuring a lift app at startup.
> >
> > On 25 feb., 16:56, Naftoli Gugenheim <[email protected]> wrote:
> >> Why should an initLogger method be in LiftRules?
> >>
> >> -------------------------------------
> >>
> >> Marius<[email protected]> wrote:
> >>
> >> Then perhaps:
> >>
> >> LiftRules.initLogger(Log4J)
> >>
> >> On Feb 25, 12:16 pm, Jeppe Nejsum Madsen <[email protected]> wrote:
> >>
> >>> On Thu, Feb 25, 2010 at 10:32 AM, Marius <[email protected]>
> wrote:
> >>>> I'd opt in for something like:
> >>
> >>>> LiftRules.logger = Log4J
> >>
> >>> Agree this fits the current idioms, but how should this be triggered?
> >>> The new logging code is in lift-common so cannot call stuff in
> >>> LiftRules.
> >>
> >>> Note we're not talking about Loggers (objects that have warn & info
> >>> methods) but about configuring the logging system, which needs to be
> >>> done before Loggers can be used.
> >>
> >>> /Jeppe
> >>
> >> --
> >> You received this message because you are subscribed to the Google
> Groups "Lift" group.
> >> To post to this group, send email to [email protected].
> >> To unsubscribe from this group, send email to
> [email protected]<liftweb%[email protected]>
> .
> >> For more options, visit this group athttp://
> groups.google.com/group/liftweb?hl=en.
> >
> > --
> > You received this message because you are subscribed to the Google Groups
> "Lift" group.
> > To post to this group, send email to [email protected].
> > To unsubscribe from this group, send email to
> [email protected]<liftweb%[email protected]>
> .
> > For more options, visit this group at
> http://groups.google.com/group/liftweb?hl=en.
> >
>
> --
> You received this message because you are subscribed to the Google Groups
> "Lift" group.
> To post to this group, send email to [email protected].
> To unsubscribe from this group, send email to
> [email protected]<liftweb%[email protected]>
> .
> For more options, visit this group at
> http://groups.google.com/group/liftweb?hl=en.
>
>


-- 
Lift, the simply functional web framework http://liftweb.net
Beginning Scala http://www.apress.com/book/view/1430219890
Follow me: http://twitter.com/dpp
Surf the harmonics

-- 
You received this message because you are subscribed to the Google Groups 
"Lift" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/liftweb?hl=en.

Reply via email to