-
Notifications
You must be signed in to change notification settings - Fork 268
fix-UInt32-overflow in common/tools.py #2268
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
Conversation
|
Hello Ian, But I am sorry to tell you that the problem is still fixed. There is a fix in the latest dev version targeting upcoming release 1.6.0 (see changelog). See commit 4ec1f82. But for Debian stable that fix is to complex and risky to be allowed in current "stable" Debian. I do have something else and more simple in mind. Please just tell me if you experience that UInt32 overflow problem in your Debian 13 installation? Are you aware of other users? In general I do have some minor issues/fixes that I consider for 1.5.6 in hope that Debian will allow them in current "stable" (13, Trixie). See #2258 as a meta issue about it. The point is I do not like that way. Stable is stable and shouldn't be changed; that is why Debian is rock solid. Anyway, there are exceptions of course. But I need to keep the diff between 1.5.5 and 1.5.6 as simple as possible, to lower the risk of introducing new bugs. What do you think? And what can you tell me about your experience with that overflow problem? Regards, |
|
Thanks for your reply, I should have done a deeper dive into the repo issues to see the status. However it was relatively straightforward to fix this problem locally and test. I'm not a Debian user so I can't comment on releases. In the Arch/Manjaro world I keep to the stable branch of the rolling release, so I'm sure I will get the 1.6.0 when available. For the time being my fix works without issues and will be overwritten on update. I'm trying to get more experience in contributing to open-source projects, so thanks for your feedback. Cheers, |
|
Hello Ian,
But it is also not that easy, especially in this project, to keep track of all issues. I am also not very good in writing well formulated commit messages and changelog entries.
I think I'll fix #2258 and release a 1.5.6 out-of-the-line just containing some minor fixes/workarounds. Regards, |
* fix: Always use 0 as window ID value when inhibiting suspend via D-Bus power manager (#2084, #2268, #2192) * fix: Crash in "Manage profiles" dialog when using "qt6ct" (#2128) * fix: Open online changelog if local CHANGES file is missing (#2266) Close #2084 Close #2268 Close #2192 Close #2128 Close #2266
This hotfix release (for Debian GNU/Linux 13 "Trixie" stable) fixes four minor issues with as few code modifications as possible. * Fixed: Always use 0 as window ID value when inhibiting suspend via D-Bus power manager (#2084, #2268, #2192) * Fixed: Crash in "Manage profiles" dialog when using "qt6ct" (#2128) * Fixed: Open online changelog if local CHANGES file is missing (#2266) * Fixed: Disable the use of browser in root-mode
|
Fixed in release 1.5.6. Hope it will make it into current Debian stable. |
|
Still no response from Debian maintainer. There is an open ticket about it without any response from him. Maybe it would help the process if you could also express your interest in an update from 1.5.5 to 1.5.6 ? |
Proposed fix in backintime/common/tools.py to resolve the out-of-range error in release 1.5.5-1 when attempting restore of folder and contents (kernel: 6.16.8-1 kde plasma: 6.3.6 qt: 6.9.2 python: 3.13)