-
Notifications
You must be signed in to change notification settings - Fork 216
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
backintime CLI shows Qt5 warning (QWidget::render: Cannot render with an inactive painter) #1404
Comments
There are several issues that mention the above "cannot render with an inactive painter" message in other contexts:
|
To all affected users: Could you please provide me the output of this (very long ;-) command line to give me more context: The output of |
Internal developer note: The Qt5 source code that throws the warning is here (may be an older code version but good enough to dig deeper): https://codebrowser.dev/qt5/qtbase/src/gui/painting/qpainter.cpp.html#1765
https://codebrowser.dev/qt5/qtbase/src/corelib/global/qnamespace.h.html#QInternal::Pixmap The only location where BiT is directly using and rendering a Pixmap is in the backintime/qt/qtsystrayicon.py Lines 102 to 109 in e22c7f2
|
I encountered the issue on my Arch installation. Prior to 1.3.3 I installed multiple versions without error. When skipping the unit test during the install the applications runs fine as it always did in the past. In case it is of importance (since the error message mentions a widget): I'm running a standalone WM and the BiT tray icon was never correctly displayed for me (see issue #1306).
|
@scaramangado THX for your diagnostics info!
Which WM do you use (and what exactly do you mean with "standalone WM")? |
I use the awesome window manager, standalone as in "no full desktop environment". |
@scaramangado THX a lot, after installing awesome with BTW: Tiling and ad-hoc window configuration is unbelievably cool and easy in PS: I am just struggling to change the keyboard layout to "DE" ;-) |
Gotcha! The unit test indeed fails due to a missing dependency: There is no supported theme installed that provides a BiT logo icon for the system tray icon. The Qt5 warnings are thrown by this line of code: backintime/qt/qtsystrayicon.py Line 109 in e22c7f2
The pixmap contains the BiT logo but if it is not installed it cannot be rendered. The bug can also be reproduced without the failing unit test by calling
(requires at least one configured backup profile) and shows one more output line that clearly indicates the problem:
The last line is thrown when this line of code is executed: backintime/qt/qtsystrayicon.py Line 134 in e22c7f2
Why this last warning is not contained in the actual unit test output is unknown to me, esp. because it is also a Qt5 warning: https://codebrowser.dev/qt5/qtbase/src/widgets/util/qsystemtrayicon.cpp.html#316 |
@scaramangado Could you please install the package The unit test should then succeed and the BiT logo correctly shown in the application as well as in the system tray. If you have already installed BiT you can simply test it with
if at least one profile is configured (otherwise only the unit test can tell us if the failure still occurs)... If you don't want to reinstall BiT you can call the unit tests directly in the AUR clone folder (if it still exists):
The unit tests should succeed now and none of the above Qt5 warnings be shown... With If you uninstall |
Installing the icon package does fix the tests for me indeed. |
@scaramangado THX a lot for investing your time to hunt this annoying bug down. I will fix this together with the master issue #1306 (together with all the other systray and icon issues that other users reported). I think I need at least until end of next week for this (requires a lot of testing on different VMs and scenarios...). |
Doesn't work on my machine. If this should be investigated before releasing the fix I'll test things on my machine if you tell me what to do. |
I have fixed all known systray and app icon issues (hopefully) with my PR #1480 (on dev branch) Re-testing and feed-back welcome! |
Not sure but I think we could this issue after the upcoming release (mid to end of Jan 2024) and if no re-test fails. |
Asked on "bit-dev" if someone experience the error since version 1.3.4. |
Closing this ticket based on the comment above. Feel free to reopen Best regards, |
This issue is based on user feedback on AUR for Arch Linux regarding a failing unit test of the new BiT release v1.3.3 which is basically cause by this unexpected additional output in the "actual" part of the unit test assertion:
The exact output was:
The AUR package is based on commit 8495f9d (which is our v1.3.3 Git tag for the release).
The unexpected output occurs not on every installation but only under unclear side conditions.
The text was updated successfully, but these errors were encountered: