-
Notifications
You must be signed in to change notification settings - Fork 6.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Adding libraries needed to build Krita #11861
Conversation
5a9eba6
to
104e047
Compare
@hellozee Also need to look into the failures on CI tests. |
@NancyLi1013 ahh yes, I was working on that too, it is just the number of ports is a bit hard to keep track of. Could I get a look in error logs? Cause on my side they build fine on both Windows and Linux, unfortunately I don't have a Mac to test it out. |
@hellozee
As for the failure on x64-windows-static for |
@NancyLi1013 ahh thanks a lot, that cleared a lot of doubts for me, though from first look, it might take a bit of time from my side to get them all resolved, feel free to avoid this till I request a review, 😃 |
@NancyLi1013 so, I couldn't understand why the build for x64-windows-static is failing, I poked around the azure pipelines and found out the artifacts. The windows build is fine, I don't know why it is complaining, nothing fails. For the linux one, I can't reproduce that locally for now. |
As for the failure on x64-windows-static:
We need to add this to portfile.cmake.
|
Okay, seemingly both linux and osx fail with the same logs. I think would try to replicate that in a clean box then, :) |
@NancyLi1013, tried both with a native linux install and the new wsl2, can't reproduce, probably the CI is doing something which I am not quite aware of, 😫 |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
Hi @hellozee
I checked the source code and found these codes in CMakeLists.txt
Since |
Okay, so probably ubuntu comes with pkg-config preinstalled. Dang, now I feel stupid to not even notice that it calls pkgconfig, nonetheless it seems to be buildling fine now, 😄 |
The osx port still fails to build, but it is something else this time. |
LGTM now. Thanks for your hard work. @hellozee |
What does your PR fix?
This PR tries to package the libraries into vcpkg that we use to build Krita. I am still pretty new to packaging libraries for vcpkg, so still learning. I took the KF5 portfiles from More KF5 ports added #4652 and [kf5] recover from #4652 #7075.
Which triplets are supported/not supported? Have you updated the CI baseline?
For now I am testing this on x64-windows but I have to check on other systems for compatibility. Yes I have updated the CI baseline.
Does your PR follow the maintainer guide?
Yes