> On Oct. 16, 2015, 12:30 a.m., Friedrich W. H. Kossebau wrote: > > `Q_WS_MAC` is not a cmake var, or? Seems someone accidentally used the c++ > > macro/define here, and noone ever noticed? :) > > > > In Qt5/KF5/ECM worlds ecm_add_app_icon does the respective installation, > > right? So only the `Q_WS_MAC` fix will need porting to the master branch?
No, Q_WS_MAC is not something that cmake provides by default, so I guess you're right. Do calligragemini and calligraauthor get icons on Linux? For the former it seems that that should only be the case if the icon .png file(s) is/are already installed, but if that's the intended behaviour shouldn't my patch be Mac-specific? Calligraauthor's `kde4_add_app_icon` call is commented out, was that intended or should that patch *not* be Mac-specific? I have no idea at the moment what has changed in KF5/ECM. If ECM provides a function `ecm_add_app_icon` then it would make sense that it does all the required work. Note however that the sole use of `kde4_add_app_icon` isn't enough. It generates the icon file, and it adds the corresponding entry in the application's `Info.plist`, but that's not always enough. I haven't yet figured out why certain applications always got an icon, and why others require explicit installation of the icon resource. - René J.V. ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125649/#review86899 ----------------------------------------------------------- On Oct. 15, 2015, 10:55 p.m., René J.V. Bertin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/125649/ > ----------------------------------------------------------- > > (Updated Oct. 15, 2015, 10:55 p.m.) > > > Review request for Calligra and KDE Software on Mac OS X. > > > Repository: calligra > > > Description > ------- > > Builds on OS X currently generate icons for most Calligra applications, but > those are installed only for a happy few (Krita, Braindump and Kexi). The > other applications require an explicit install command of the generated > `.icns` file into the app bundle's Resources directory. > > The attached patch takes care of that. > > In addition, it corrects the picture source directories for calligragemini > and calligraauthor so those applications can have icons on other platforms > too, and replaces the `Q_WS_MACOS` token with the (IMHO) more appropriate > `APPLE` token. > > > Diffs > ----- > > flow/part/CMakeLists.txt 58882f1 > gemini/CMakeLists.txt 85123fa > karbon/CMakeLists.txt b574779 > plan/CMakeLists.txt ad39f57 > sheets/CMakeLists.txt b0cc134 > stage/app/CMakeLists.txt 079bece > words/app/CMakeLists.txt 1e73971 > words/part/CMakeLists.txt 9143176 > > Diff: https://git.reviewboard.kde.org/r/125649/diff/ > > > Testing > ------- > > On OS X 10.9 with KDELibs 4.14.12 and MacPorts 2.3.4 . > > > Thanks, > > René J.V. Bertin > >
_______________________________________________ calligra-devel mailing list [email protected] https://mail.kde.org/mailman/listinfo/calligra-devel
