← Back to team overview

kicad-developers team mailing list archive

Re: Fwd: [PATCH] Request to apply patch for redirect console output to single file

 

More information it not better.  The optimal amount of information is the "minimal needed
information" when troubleshooting.  Extraneous information simply reduces the signal to
noise ratio.  And this opinion is a strong one I hold, established by doing this for 30
years, 40 hours a week.


I don't see how this can be done conditionally compiled, without a lot of noise at the
conditional compile points.


BTW, can you add my grandmother's maiden name to the output?  Somebody might need that.



On 07/09/2014 03:00 PM, Nhat Khai wrote:
> Please look at the log output. It give all the pin point where is the code, threadID,
> level. Would it be much easier to locale the problem for all developer?
> 
> Below is the example of the output look like in file name
> Debug_eeschema.20140709_153421.thread25760.log when run eeschema:
> 15:34:21 #25760# LVL#03 [      single_top.cpp:127 :OnInit         ] Program Initializing
> 15:34:21 #25760# LVL#03 [      single_top.cpp:127 :OnInit         ] wxLogMessage is on
> 15:34:21 #25760# LVL#06 [      single_top.cpp:127 :OnInit         ] wxLogDebug   is on
> 15:34:21 #25760# LVL#01 [      single_top.cpp:127 :OnInit         ] wxLogError   is on
> 15:34:21 #25760# LVL#03 [      single_top.cpp:81  :set_lib_env_var]
> LIB_ENV_VAR:'C:\projects\kicad-winbuilder\kicad\bin;C:\projects\kicad-winbuilder\env\mingw-w64\mingw32\bin;C:\projects\kicad-winbuilder\env\cmake\cmake-2.8.12.2-win32-x86\bin;C:\projects\kicad-winbuilder\env\bazaar;C:\projects\kicad-winbuilder\env\patch\bin;C:\projects\kicad-winbuilder\env\python;C:\projects\kicad-winbuilder\env\swig\swigwin-2.0.10;C:\projects\kicad-winbuilder\env\tee;C:\projects\kicad-winbuilder\env\libxslt-1.1.26.win32\bin;C:\projects\kicad-winbuilder\env\libxml2-2.7.8.win32\bin;C:\projects\kicad-winbuilder\env\iconv-1.9.2.win32\bin;C:\projects\kicad-winbuilder\env\zlib-1.2.5\bin;C:\projects\kicad-winbuilder\kicad\bin;C:\projects\kicad-winbuilder\kicad;C:\WINDOWS\System32'
> 15:34:21 #25760# LVL#06 [           image.cpp:2805:AddHandler     ] Adding duplicate image
> handler for 'PNG file'
> 15:34:21 #25760# LVL#06 [           image.cpp:2805:AddHandler     ] Adding duplicate image
> handler for 'GIF file'
> 15:34:21 #25760# LVL#06 [           image.cpp:2805:AddHandler     ] Adding duplicate image
> handler for 'JPEG file'
> 15:34:21 #25760# LVL#06 [        pgm_base.cpp:656 :SetLanguagePath] Adding locale lookup
> path: C:\projects\kicad-winbuilder\kicad\share\internat
> 15:34:21 #25760# LVL#06 [        pgm_base.cpp:656 :SetLanguagePath] Adding locale lookup
> path: C:\Program Files (x86)\KiCad\share\internat
> 15:34:22 #25760# LVL#03 [    search_stack.cpp:151 :Show           ] kiface SEARCH_STACK:
> 15:34:22 #25760# LVL#03 [    search_stack.cpp:154 :Show           ]   [
> 0]:C:\projects\kicad-winbuilder\kicad\share\library
> 15:34:22 #25760# LVL#03 [    search_stack.cpp:154 :Show           ]   [
> 1]:C:\projects\kicad-winbuilder\kicad\share\template
> 15:34:22 #25760# LVL#03 [    search_stack.cpp:154 :Show           ]   [ 2]:C:\Program
> Files (x86)\KiCad\share\library
> 15:34:22 #25760# LVL#03 [    search_stack.cpp:154 :Show           ]   [ 3]:C:\Program
> Files (x86)\KiCad\share\template
> 15:34:22 #25760# LVL#07 [      draw_frame.cpp:793 :AdjustScrollBar] (KicadScrollSettings)
> Center Position = ( 5846, 4134 ), scale = 0.1118771166
> 15:34:22 #25760# LVL#07 [      draw_frame.cpp:969 :AdjustScrollBar] (KicadScrollSettings)
> Drawing = (16483, 8268), Client = (16482.36973, 8268), Offset = (-2395, 0),
> SetScrollbars(1, 1, 1844, 925, 0, 0)
> 15:34:22 #25760# LVL#03 [    search_stack.cpp:151 :Show           ] configCreate SEARCH_STACK:
> 15:34:22 #25760# LVL#03 [    search_stack.cpp:154 :Show           ]   [
> 0]:C:\projects\kicad-winbuilder\kicad\share\library
> 15:34:22 #25760# LVL#03 [    search_stack.cpp:154 :Show           ]   [
> 1]:C:\projects\kicad-winbuilder\kicad\share\template
> 15:34:22 #25760# LVL#03 [    search_stack.cpp:154 :Show           ]   [ 2]:C:\Program
> Files (x86)\KiCad\share\library
> 15:34:22 #25760# LVL#03 [    search_stack.cpp:154 :Show           ]   [ 3]:C:\Program
> Files (x86)\KiCad\share\template
> 15:34:22 #25760# LVL#06 [         project.cpp:330 :configCreate   ] Use template file
> 'C:\projects\kicad-winbuilder\kicad\share\template\kicad.pro <http://kicad.pro>' as
> project file.
> 15:34:22 #25760# LVL#03 [    search_stack.cpp:151 :Show           ] LoadLibraries
> SEARCH_STACK:
> 15:34:22 #25760# LVL#03 [    search_stack.cpp:154 :Show           ]   [
> 0]:C:\projects\kicad-winbuilder\kicad\share\library
> 15:34:22 #25760# LVL#03 [    search_stack.cpp:154 :Show           ]   [
> 1]:C:\projects\kicad-winbuilder\kicad\share\template
> 15:34:22 #25760# LVL#03 [    search_stack.cpp:154 :Show           ]   [ 2]:C:\Program
> Files (x86)\KiCad\share\library
> 15:34:22 #25760# LVL#03 [    search_stack.cpp:154 :Show           ]   [ 3]:C:\Program
> Files (x86)\KiCad\share\template
> 15:34:38 #25760# LVL#06 [        files-io.cpp:317 :OpenProjectFile] Loading schematic
> C:\projects\svn\CockpitUSBSwitch\kicad_files\USBSwitcher.sch
> 15:34:38 #25760# LVL#03 [    search_stack.cpp:151 :Show           ] configCreate SEARCH_STACK:
> 15:34:38 #25760# LVL#03 [    search_stack.cpp:154 :Show           ]   [
> 0]:C:\projects\svn\CockpitUSBSwitch\kicad_files
> 15:34:38 #25760# LVL#03 [    search_stack.cpp:154 :Show           ]   [
> 1]:C:\projects\kicad-winbuilder\kicad\share\library
> 15:34:38 #25760# LVL#03 [    search_stack.cpp:154 :Show           ]   [
> 2]:C:\projects\kicad-winbuilder\kicad\share\template
> 15:34:38 #25760# LVL#03 [    search_stack.cpp:154 :Show           ]   [ 3]:C:\Program
> Files (x86)\KiCad\share\library
> 
> 
> 
> ---------- Forwarded message ----------
> From: *Dick Hollenbeck* <dick@xxxxxxxxxxx <mailto:dick@xxxxxxxxxxx>>
> Date: Wed, Jul 9, 2014 at 3:34 PM
> Subject: Re: [PATCH] Request to apply patch for redirect console output to single file
> To: Nhat Khai <nhatkhai@xxxxxxxxx <mailto:nhatkhai@xxxxxxxxx>>
> 
> 
> On 07/09/2014 01:41 PM, Nhat Khai wrote:
>> Dear Dick Hollenbeck,
>>   I like apply this patch to redirect all the console output to single file which help
>> debug this software a bit better, and easily use with tail, baretail, or logEpxert tool
>> during develop the code.
>>
>> --
>> Nhat *Khai* Nguyen
> 
> 
> I don't take personal email about KiCad from strangers.  Historically we simply redirect
> the program output from the command line.  I like that better than your patch.
> 
> So if I see your patch on the mailing list, I will reject it.
> 
> Sorry, we did not get off to a better start.
> 
> Please join the mailing list.
> 
> Dick
> 
> 
> 
> 
> -- 
> Nhat *Khai* Nguyen
> 
> 
> _______________________________________________
> Mailing list: https://launchpad.net/~kicad-developers
> Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
> Unsubscribe : https://launchpad.net/~kicad-developers
> More help   : https://help.launchpad.net/ListHelp
> 



Follow ups

References