-
Notifications
You must be signed in to change notification settings - Fork 3
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
Bugfixes, autotools update #1
base: master
Are you sure you want to change the base?
Conversation
tarjanm-movidius
commented
Mar 26, 2022
- Updated autotools
- Fixed all warnings to C++/17 standard
- Fixed memory leak when reading std::map by element
- Fixed NULL pointer segfault
- Fixed overlapping window raise issue
- Updated docs, added man page
i've checked locally fbpager plus your patches ontop of it. the whole build-system should be moved more towards what fluxbox is using: even with your patches I ran into to problems to build fbpager ontop of aarch64. I then used
The rest of your changes look valid. If you want, rework the build-system stuff by even further reducing it "to the max" and then I ll merge the whole thing. |
Hi Mathias, |
@tarjanm-movidius then i ll cherry-pick the relevant changes from your branch and skip the build-part. can't tell you "when" though. thanks anyway! |
Merging shelved changes, cppcheck trivial fixes and autogen
@tarjanm-movidius ah, cool. I'll try your branch on the weekend. The delay was not intentionally, I am sorry about it. |