[aerogear-dev] Android testing: Show logcat via Logger or System.out/err?

[TadeasKriz]

That wouldn't make much a difference. The problem is not warnings but the thing that Logger prints two lines. If there will be a stack-trace (often for error logs in logcat) it won't be readable. I was thinking that by default, the System.out would be better. It can be easily changed in the arquillian.xml with one property so those who would need the Logger's features would switch it.


On Jul 11, 2013, at 4:51 PM, Corinne Krych <corinnekrych at gmail.com> wrote:

> By default could you display Error level only?
> avoid warning level which clutter your log
> 
> On Jul 11, 2013, at 4:40 PM, TadeasKriz <tkriz at redhat.com> wrote:
> 
>> As you said, second looks better. So I was thinking that if you'll need to configure logger you'll enable it, but otherwise it's just very confusing for a newcomer.
>> 
>> On Jul 11, 2013, at 4:29 PM, Corinne Krych <corinnekrych at gmail.com> wrote:
>> 
>>> It's a better practice to log with a logger rather than system.out. 
>>> Logger are configurable.
>>> Indent looks better on second output though
>>> 
>>> ++
>>> Corinne
>>> On Jul 11, 2013, at 4:24 PM, TadeasKriz <tkriz at redhat.com> wrote:
>>> 
>>>> Hey guys!
>>>> 
>>>> I'd like to hear your opinions on one decision I'm about to make. I've been working on implementation of logcat into arquillian-droidium and now I'm trying to decide, which kind of logging should be enabled by default. Whether it should be logging though Logger or through System.out (or disabled completely by default). These two logs are here: https://gist.github.com/TadeasKriz/e7c91f3e7e230e596f20
>>>> _______________________________________________
>>>> aerogear-dev mailing list
>>>> aerogear-dev at lists.jboss.org
>>>> https://lists.jboss.org/mailman/listinfo/aerogear-dev
>>> 
>>> _______________________________________________
>>> aerogear-dev mailing list
>>> aerogear-dev at lists.jboss.org
>>> https://lists.jboss.org/mailman/listinfo/aerogear-dev
>> 
>> _______________________________________________
>> aerogear-dev mailing list
>> aerogear-dev at lists.jboss.org
>> https://lists.jboss.org/mailman/listinfo/aerogear-dev
> 
> _______________________________________________
> aerogear-dev mailing list
> aerogear-dev at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/aerogear-dev

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.jboss.org/pipermail/aerogear-dev/attachments/20130711/a425938c/attachment.html