-
Notifications
You must be signed in to change notification settings - Fork 37
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
repair and simplify sync log collection page. #375
base: master
Are you sure you want to change the base?
Conversation
See https://forum.sailfishos.org/t/ist-the-documentation-for-getting-caldav-carddav-sync-logs-faulty/16220 why the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why the changes in the name of the headings?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In general I agree with the changes but there is a better way to do this.
systemctl --user edit msyncd
- Paste this into the opened editor:
[Service]
Environment=QTCONTACTS_SQLITE_TRACE=1
Environment=MSYNCD_LOGGING_LEVEL=8
Environment=QTCONTACTS_SQLITE_TWCSA_TRACE=1
- systemctl --user restart msyncd
- journalctl --user --boot --unit msyncd.service
- If the --user filter doesn't work try /usr/bin/msyncd instead of --user ---unit msyncd.service
To revert remove rm ~/.config/systemd/user/msyncd.service.d/override.conf
Doing so will ensure that the logfile will only contain output from the msyncd service.
In general calling journalctl to log something from the user session without filters is bad as it contains to much irrelevant information.
See this post why to prefer 'tee' to 'cat'. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGMT.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The proposed changes make sense and look good to me.
I'm not sure if
tee
is included in a standard installation, though.