-
-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Log file in LMMS #3876
Comments
Some context #3561 (comment) as to why this is so important, at least on Windows. |
Logs make every thing slow.Logs never help with bugs. Of course, stderr should be visible on every OS. If it is not on Windows, redirect the stream to a file. |
No, it's simply not there at all. |
For the record though, Wine has access to it without |
Context please? What I'm stating is that the console is gone in a GUI app for Windows. You need to compile it as a console app if you want output and that console will appear until closed. It's a limitation of the OS. What are you suggesting? |
@gi0e5b06 Check this: |
Good enough for Windows.
|
So logging is only a performance problem when it affects the system you use? |
Writing to a file is faster than writing to the console. |
Please explain or stop contradicting yourself. Logs are standard in large applications. If logs are needed for support, they're helpful, regardless of the system. If they drag down the software, they should do it to all platforms equally. Perhaps we disable them by default but these blanket statements are unnecessarily provocative. Please spend more time explaining and less time dropping conversation bombs.
Yet another bomb. Please don't be presumptuous. Making unnecessary and different support structures between end-users makes support more difficult. Also, differentiating behavior requires a larger barrier to entry for people helping. Despite what you THINK YOU KNOW, non-tech people use Linux and Mac and other non-Windows platforms. Your statements are baiting more off-topic banter. Let's do the project a favor and work towards what works, not make blanket statements around the last comment in the thread. |
I write code, not books ;) I'm not aware of any large desktop application writing logs, and if there is, I don't know where these logs and I never ever looked at them. If anyone is using an other DAW, I would be interested to see what their logs look like. |
I know several desktop applications that write logs and they usually live within the user's home directory (e.g. In regards to how many DAWs use logs... I don't know any DAW that crashes as often as LMMS, so it may be a loaded question. Anyway... I decided to take a look at Ardour and they ask their users to download a debug build and run it through gdb on Windows. This may be a good compromise for our users, because as others have stated before, the logs usually don't help anyway. |
Gentlemen! |
Regarding that "logs make everything slow": if we decided to use logging we must make sure to never put it into any code that's potentially executed from the audio thread anyway. So it would likely only end up in glue code and GUI related code where a few nanoseconds shouldn't matter. However, with the current code base where the distinction between realtime and non-realtime code is not made implementing some logging might therefore lead to audio problems (crackles, dropouts, distortion). I also disagree that "logs never help". Depending on the application they can be very helpful (I work with log files every day). So as @tresf has already noted we should stop using such blanket statements. Whenever I read such statements I definitively do not think that they are made by (professional) software developer but rather by people who refer to themselves as "coders" or "hackers". Software development / engineering is about much more than just writing code. |
We should distinguish between operation log and error log. I'm OK with logging anything that should be fixed by the user or the developper. I quite disagree on logging qInfo stuff. If your app outputs more than a few lines during a session, there is a problem. @NETMANSKY Qt has qDebug, qInfo, qWarning, qCritical, qFatal). Qt5 adds QMessageLogger. Qt5.6 adds support for syslog and journald. @michaelgregorius We are talking about desktop application, especially DAWs. And let's not make it personnal (I actually have more experience than you). OTOH, I like several of your patches, including some unmerged ones than I plan to apply soon. So let's keep the good work. |
The ad hominem isn't tolerated, this is the first warning. Let's keep this civil, please. |
@tresf warning for who? Because I'm saying exactly the same thing than you. Anyway, back to the topic:
|
For this statement: "I actually have more experience than you".
Agreed, we're all saying the same thing objectively, so let's move on. My thoughts... From the feedback and points above, the logging doesn't really seem like it will help LMMS at this point. The main reasons are:
In regards to the other questions....
So I think we can close this once we have debug binaries available for download. I'm thinking we can shim this into our travis-ci logic and add some logic so that they show underneath the Beta downloads on our downloads page. |
Consolidated into #5468. |
LMMS lacks log file to trap bugs. It will help devs to fix troubles.
The text was updated successfully, but these errors were encountered: