[reposted from kde-devel] Building KDE statically

2019-03-01 Thread Jonathan Schultz
Hello KDE developers, If anyone is interested, here is a brief report on something I have been working on in my spare time. TLDR: Here are some scripts to build KDE frameworks and okular statically using gcc/musl and cross-building for mingw: https://github.com/jschultz/kde-static Look in the

[Differential] [Commented On] D2075: Fix bug in kfiledialog.cpp that causes crashing when native widgets are used.

2017-01-11 Thread Jonathan Schultz
jonathans added a comment. I wrote the first patch to be as minimal as possible and to be consistent with the previous coding style. I therefore left the early returns in place. I wrote the latest patch based on my interpretation of your (@kfunk) feedback that it makes more sense to tes

[Differential] [Updated, 122 lines] D2075: Fix bug in kfiledialog.cpp that causes crashing when native widgets are used.

2017-01-10 Thread Jonathan Schultz
jonathans set the repository for this revision to R239 KDELibs4Support. jonathans updated this revision to Diff 10021. jonathans marked 2 inline comments as done. jonathans added a comment. Restricted Application added a project: Frameworks. Found a couple of places in old code where d->w made

[Differential] [Commented On] D2075: Fix bug in kfiledialog.cpp that causes crashing when native widgets are used.

2017-01-10 Thread Jonathan Schultz
jonathans added a comment. Apparently updating the diff has had the side-effect of: > https://phabricator.kde.org/source/kdelibs4support/ KDELibs4Support as the repository for this revision. Please excuse my bumbling - I don't know why that happened or how to repair it. :( REVISIO

[Differential] [Updated, 76 lines] D2075: Fix bug in kfiledialog.cpp that causes crashing when native widgets are used.

2017-01-10 Thread Jonathan Schultz
jonathans removed R239 KDELibs4Support as the repository for this revision. jonathans updated this revision to Diff 10020. jonathans added a comment. Following @kfunk comments, I've changed the patch to test d->w rather than d->native, and used nullptr instead of 0 where appropriate. CHANGES

[Differential] [Commented On] D2075: Fix bug in kfiledialog.cpp that causes crashing when native widgets are used.

2016-12-02 Thread jonathans (Jonathan Schultz)
jonathans added a comment. Agreed that would be more robust. In writing the patch I was seeking consistency with those functions that already did the test, so those would also need to be updated. Are there any situations where the two tests would yield a different result, ie d->native is tru

Bug in kdelibs4support/kio/kfiledialog

2016-06-07 Thread Jonathan Schultz
Sorry for cross-posting, but kde-frameworks-devel seems to be a more active list than kde-devel where I previously raised this issue. I'd like to file a bug report for kdelibs4support/kio/kfiledialog but don't seem to be able to find a relevant category in KDE Bugzilla. If someone can point me