This isn't really a problem on desktop, but when cross-compiling this can
interfere badly with the toolchain file. Specifically, this fixes building
modules relying on the ANDROID_API define.
I don't really know why cxxflags is ' ' when being empty, so there is
likely a cleaner/better way to fix this, but my Perl isn't good enough
for that.
| No Linters Available |
| No Unit Test Coverage |
| Buildable 6928 | |
| Build 6946: arc lint + arc unit |
I'll take a look and fix but if further review is needed may I invite you to the trial GitLab instance at https://invent.kde.org/kde/kdesrc-build/merge_requests ? (kdesrc-build is a participant in the trial).
| modules/ksb/BuildSystem/KDE4.pm | ||
|---|---|---|
| 173 | You're right there's a better way, by adding a couple of lines to trip leading and trailing whitespace. I may have a utility function that does this already but if not it's probably time to add it. | |
This bug should be fixed (at its precise source) in commit b01505bec941f06be52ca3ac88d9b6937e1f4c88.