> On Dez. 11, 2014, 5:02 nachm., Aleix Pol Gonzalez wrote: > > modules/ECMAddAppIcon.cmake, line 36 > > <https://git.reviewboard.kde.org/r/121448/diff/1/?file=332663#file332663line36> > > > > I wouldn't take WINCE into account, its support hasn't been ported into > > kf5. > > > > Also missing indentation.
> WINCE sure, will update the patch >Also missing indentation. To which area of the file are you refering ? Looking on other files in the module subdirectory I see 2 or 4 spaces indention for cmake code. The script uses 4 spaces. On Dez. 11, 2014, 5:02 nachm., Ralf Habacker wrote: > > Regarding patterns, why not just using a list of the icons we need? > > patterns+cmake have weird effects... > > Would it be possible to include a test? It's reasonably easy to create ecm > > tests nowadays and very useful afterwards. You'll see some examples in the > > repository. > > > > By the way, thanks for looking into this, really needed! > Regarding patterns pattern has be introduced in kde4 times because applications provides many different combinations of resolutions, which has otherwise to be maintained by hand. You cannot run simply "ls hi*-app-<app-name>.png" and paste the result into this macro because the windows icon format supports only a subset of resolutions. Umbrello for example returns hi128-app-umbrello.png hi16-app-umbrello.png hi22-app-umbrello.png hi32-app-umbrello.png hi48-app-umbrello.png hi64-app-umbrello.png from which you need to exclude 'hi22-app-umbrello.png' by hand. The script maintains this for you by using the following form: ecm_app_app_icon(myapp_SRCS "pics/cr*-myapp.png") single files are also possible ecm_app_app_icon(myapp_SRCS "pics/cr16-myapp.png;pics/cr32-myapp.png") >Would it be possible to include a test? You mean to create a simple test app and CMakeLists.txt including this script ? sure, on windows this would require to have the png2ico tool installed for example from the kdewin-tool binary package or build from kdewin repo >You'll see some examples in the repository. which repo are your refering ? extra-cmake-modules do not contain any example - Ralf ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/121448/#review71815 ----------------------------------------------------------- On Dez. 11, 2014, 4:40 nachm., Ralf Habacker wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/121448/ > ----------------------------------------------------------- > > (Updated Dez. 11, 2014, 4:40 nachm.) > > > Review request for Extra Cmake Modules, KDE Frameworks and Laurent Navet. > > > Repository: extra-cmake-modules > > > Description > ------- > > This module, which has been migrated from the related KDE4 macto > kde4_app_app_icon, > supports platform specific application icon for Windows and Mac OSX. > > On Windows this function depends on the external tool png2ico, which is > provided by the kdewin-tools binary package. Sources are available at > https://projects.kde.org/projects/kdesupport/kdewin. > > > Diffs > ----- > > modules/ECMAddAppIcon.cmake PRE-CREATION > > Diff: https://git.reviewboard.kde.org/r/121448/diff/ > > > Testing > ------- > > > Thanks, > > Ralf Habacker > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel