#30 Config dir line printed to STDOUT because of typo

None
closed
nobody
None
2018-02-25
2018-02-13
Anonymous
No

When I run recoll from the command line, it prints out the following:

:3:common/rclinit.cpp:308::Configuration directory: /home/leon/.recoll

This is causing a glitch in the emacs-helm interface to recoll.

The problem was introduced with commit 70c36ea3e1e30aab1f736957826c68c59a42555e. The line added in that commit (line 308 in rclinit.cpp) invokes "LOGINF". I believe this is missing an "O" and should be "LOGINFO".

Discussion

  • medoc
    medoc
    2018-02-13

    Thanks for looking into this, but as far as I can see, LOGINF and LOGINFO are just synonyms (there is a #define LOGINFO LOGINF in log.h), and I don't think that the log can print to stdout at all (or I'm really missing something). It always print to the designated file, or to stderr.

    Can you try to run as 

    recoll [options] 2> /dev/null

    to check if this is really going to stdout ?

     
    Reply
    Link Attach

  • medoc
    medoc
    2018-02-16

    • status: open --> closed
    • milestone: -->
     
    Reply
    Link Attach

  • medoc
    medoc
    2018-02-16

    I see no way LOG could print to stdout. Printing to stderr should not disturb your app, or you can always set loglevel to 0.

     
    Reply
    Link Attach

  • Anonymous
    Anonymous
    2018-02-25

    Thank you for looking into this. I apologize --- you are correct. The output was stderr rather than stdout. I changed the log level and all is well now.

     
    Reply
    Link Edit
    Delete


    Attach

Anonymous
Anonymous

Preview Formatting Help

Cancel   Add attachment