----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128526/#review97865 -----------------------------------------------------------
Update: the script of build.kde.org sets BINARY_ICONS_RESOURCE to ON, nevertheless everyone who compiles breeze-icons needs to know it's needed. It's hard to say not buiulding the rcc files gives any important optimization. The cost is like 10MiB for white and 10MiB for dark file while build dir of a single app is measured in GiBs. - Jarosław Staniek On July 26, 2016, 11:35 a.m., Jarosław Staniek wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/128526/ > ----------------------------------------------------------- > > (Updated July 26, 2016, 11:35 a.m.) > > > Review request for Plasma and Andreas Kainz. > > > Repository: breeze-icons > > > Description > ------- > > Make BINARY_ICONS_RESOURCE ON by default. Otherwise packagers will not create > packages with breeze-icons.rcc. Let them possible create split packages - 1 > with icon files and 1 with the rcc file. > > Also our Jenkins fails to create the file, apps have no way to add dependency > on breeze-icons "with rcc", there's just breeze-icons that we can depend on. > Example failure: > https://build.kde.org/job/kexi%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/lastFailedBuild/console > > > Diffs > ----- > > CMakeLists.txt c0623ca > > Diff: https://git.reviewboard.kde.org/r/128526/diff/ > > > Testing > ------- > > breeze-icons now installs the .rcc file. > > > Thanks, > > Jarosław Staniek > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
