search

xybu / onedrive-d-old

Microsoft OneDrive client on Linux.
http://xybu.me/projects/onedrive-d/
GNU Lesser General Public License v3.0
820 stars 143 forks source link

ConfigSet had missing logger #150

Closed CGenie closed 9 years ago

Dr-Xperience commented 9 years ago

Well noob here so please be kind. The logging is not working I have an empty file in /var/log/onedrive_d.log. (Fedora 21)

xybu commented 9 years ago

The log is very likely to be written to stderr. Don't remember the old code base though.

Sincerely, Xiangyu Bu

Sent from my Windows Phone

From: Anubhav Arunmailto:notifications@github.com Sent: ‎8/‎14/‎2015 11:53 AM To: xybu/onedrive-d-oldmailto:onedrive-d-old@noreply.github.com Cc: Xiangyu Bumailto:xybu92@live.com Subject: Re: [onedrive-d-old] ConfigSet had missing logger (#150)

Well noob here so please be kind. The logging is not working I have an empty file in /var/log/onedrive_d.log. (Fedora 21)

Reply to this email directly or view it on GitHub: https://github.com/xybu/onedrive-d-old/pull/150#issuecomment-131158722

Dr-Xperience commented 9 years ago

The daemon doesn't write log on stderr. The instruction for old version i.e. this repository said to touch /var/log/onedrive_d.log file to maintain logs.

CGenie commented 9 years ago

Hm, I actually don't get the problems you're having. I checked out master, executed onedrive-d start --debug, all seems to work fine. I don't have the log in /var/log/onedrive_d.log, didn't create that file either and everything works, even though I have "LOG_FILE_PATH": "/var/log/onedrive_d.log" set in config_v2.json.

xybu commented 9 years ago

Honestly that param wasn't used by logger...

CGenie commented 9 years ago

Yeah, it seems that in the meanwhile code changed and there is no more ConfigSet class here. :)

xybu commented 9 years ago

I have forgotten almost everything in this repo... Focused on this: https://github.com/xybu/onedrive-d