[rfc][icedtea-web] Reprint of application logs to console

Andrew Azores aazores at redhat.com
Mon Mar 24 14:58:23 UTC 2014


On 03/24/2014 06:34 AM, Jiri Vanek wrote:
> On 03/21/2014 04:57 PM, Omair Majid wrote:
>> Hi
>>
>> I haven't reviewed the patch, but just wanted to comment on something.
>>
>> * Jiri Vanek <jvanek at redhat.com> [2014-03-21 11:51]:
>>> If application is calling system.out/err.print(ln) for some kind of
>>> debugging, then it is *maybe* good to show it in our debug console.
>>> With its powerful features it may be usefull debugging tool for
>>> developers.
>>
>> I would replace 'maybe' with 'definitely'.
>
> Thanx for support!
>>
>>> tbh, I'm not sure if this is wonted feature, but I somehow think it
>>> should be there. I also think previous console did it. So should
>>> new.  And ...  it really looks cool :)
>>
>> I think this is definitely worth having. And yes, I implemented this in
>> the older console. The console that is part of the proprietary plugin
>> does this too.
>>
>  In this case I would like to push it to head asap, as the code is 
> perfect as usually, can I consider it as "ok for head" ? :)
>
>
> J.
>

I think it looks okay, although I don't really know what's going on in a 
lot of the console, so some of it I don't know what it does (eg why the 
canChange field is needed).

Thanks,

-- 
Andrew A



More information about the distro-pkg-dev mailing list