Skip to content
This repository has been archived by the owner on Jul 2, 2024. It is now read-only.

Log event to system eventlog when switching desktops #62

Open
DonatJR opened this issue Jul 8, 2018 · 4 comments
Open

Log event to system eventlog when switching desktops #62

DonatJR opened this issue Jul 8, 2018 · 4 comments

Comments

@DonatJR
Copy link
Contributor

DonatJR commented Jul 8, 2018

logging of service install / uninstall is already done by the installer / system

@DonatJR
Copy link
Contributor Author

DonatJR commented Jul 15, 2018

I'm not sure it is worth it to include event logging just for the switch desktop event. The process of setting up event logging seems to be rather involved, maybe we can wait until we really need it or have identified more events that need to be logged.
Some resources:

@SailReal If you agree, I will move this issue to the backlog

@SailReal
Copy link
Contributor

Thanks for figuring out 👍.

I also think we should wait with the implementation until we have more usecases.
So I'll put it back in the backlog.

@bencikpeter
Copy link
Contributor

bencikpeter commented Aug 1, 2018

Wouldn´t it be worth it to implement some general logging capability into the app? all those std::cout calls should IMO be replaced with some logger calls.

Maybe some more lightweight logging framework could serve the purpose, because as @DonatJR pointed out, ETW is pretty heavyweight. Maybe something like Log4Cplus?

@DonatJR
Copy link
Contributor Author

DonatJR commented Aug 1, 2018

@bencikpeter
Yes, having a logging framework would be awesome! There is an issue for it in the backlog (#9). Can you recommend Log4Cplus? I have never used it but if you think it suits the needs for this project I am okay with commiting to it.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants