[Craft] [Bug 481455] kiconthemes master failed
https://bugs.kde.org/show_bug.cgi?id=481455 --- Comment #4 from Ben Cooksley --- Building master versions of software with Craft is generally not supported - which is why it fails. The patch will be removed when the next Frameworks release takes place and the Craft cache is rebuilt. -- You are receiving this mail because: You are watching all bug changes.
[Craft] [Bug 481455] kiconthemes master failed
https://bugs.kde.org/show_bug.cgi?id=481455 --- Comment #3 from Laurent Montel --- So you fix only kontact but not kicontheme.cpp where patch will be not apply ? So ok you revert kontact patch but if kicontheme.cpp failed you can't say that bug is fixed. -- You are receiving this mail because: You are watching all bug changes.
[Craft] [Bug 481455] kiconthemes master failed
https://bugs.kde.org/show_bug.cgi?id=481455 Julius Künzel changed: What|Removed |Added Resolution|--- |FIXED Latest Commit||https://invent.kde.org/pim/ ||kontact/-/commit/31d55788aa ||a4ea1869dbb65c487632c3c7f66 ||a79 Status|REPORTED|RESOLVED --- Comment #2 from Julius Künzel --- Git commit 31d55788aaa4ea1869dbb65c487632c3c7f66a79 by Julius Künzel. Committed on 16/03/2024 at 18:44. Pushed by jlskuz into branch 'master'. [craft] Don't build master version for everything This is bad practice as master versions are not cached. Hence this will take a lot of time to build. Maybe even so much that it will timeout (and don't set qtVersion, it is set globally with the correct GitLab template) M +0-4.craft.ini https://invent.kde.org/pim/kontact/-/commit/31d55788aaa4ea1869dbb65c487632c3c7f66a79 -- You are receiving this mail because: You are watching all bug changes.
[Craft] [Bug 481455] kiconthemes master failed
https://bugs.kde.org/show_bug.cgi?id=481455 Ingo Klöcker changed: What|Removed |Added CC||kloec...@kde.org --- Comment #1 from Ingo Klöcker --- kontact master requests master of frameworks and libs via the blueprint settings in its `.craft.ini`: ``` kde/frameworks.version=master kde/pim.version=master kde/libs.version=master ``` Don't do this. On Binary Factory Craft builds always used the latest released versions of all dependencies (with very few exceptions). This isn't going to change with GitLab because compiling everything from master is prohibitively expensive. We have a cache for Craft with the latest stable releases for a reason. The Craft people only update the patches when they switch to a new released version of frameworks. Feel free to submit an MR for the blueprint of kiconthemes which doesn't apply the patch to the master version. Even better: Remove all those `kde/*=master` lines kontact's .craft.ini. -- You are receiving this mail because: You are watching all bug changes.
