Bug 135615 - if `--with-logfile=` in confiugre is set, it should also enable log to file in default config
Summary: if `--with-logfile=` in confiugre is set, it should also enable log to file i...
Status: NEW
Alias: None
Product: LibreOffice Online
Classification: Unclassified
Component: LibreOffice (show other bugs)
Version:
(earliest affected)
unspecified
Hardware: All All
: medium enhancement
Assignee: Not Assigned
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-08-10 13:43 UTC by anhubert
Modified: 2020-08-14 06:45 UTC (History)
1 user (show)

See Also:
Crash report or crash signature:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description anhubert 2020-08-10 13:43:09 UTC 
If someone uses `./configure --with-logfile=/foo/bar/lool.log` the loolwsd.xml config file path is changed accordingly, but still `<file enable="false">` remains. 

So hereby I request following change to configure.ac
```
if test -n "$with_logfile" ; then
   LOOLWSD_LOGFILE="$with_logfile"
   LOOLWSD_LOG_TO_FILE="true"
fi
```
I actually wanted to do a change request in Gerrit, but I don't know how.
Comment 1 Aron Budea 2020-08-14 06:45:15 UTC 
Thanks, please see how to setup your environment to submit patches here:
https://wiki.documentfoundation.org/Development/GetInvolved#Prepare_to_submit_patches