-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
[qt/6.4.2] Remove .dll extension as QSystemLibrary unconditionally adds it #16317
Conversation
🤖 Beep Boop! This pull request is making changes to 'recipes/qt//'. 👋 @ericLemanissier @jwillikers @MartinDelille you might be interested. 😉 |
This comment has been minimized.
This comment has been minimized.
I detected other pull requests that are modifying qt/6.x.x recipe:
This message is automatically generated by https://github.com/ericLemanissier/conan-center-conflicting-prs so don't hesitate to report issues/improvements there. |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
@uilianries is it fine to push the change to fix this error or I should wait before triggering the CI for such a big package ? |
This comment has been minimized.
This comment has been minimized.
1a7a5b7
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
|
Since this changed is included in 6.4.3, I could eventually close this in favor of #16584 |
Conan v1 pipeline ❌Failure in build 8 (
Note: To save resources, CI tries to finish as soon as an error is found. For this reason you might find that not all the references have been launched or not all the configurations for a given reference. Also, take into account that we cannot guarantee the order of execution as it depends on CI workload and workers availability. Conan v2 pipeline (informative, not required for merge) ❌
The v2 pipeline failed. Please, review the errors and note this will be required for pull requests to be merged in the near future. See details:Failure in build 8 (
Note: To save resources, CI tries to finish as soon as an error is found. For this reason you might find that not all the references have been launched or not all the configurations for a given reference. Also, take into account that we cannot guarantee the order of execution as it depends on CI workload and workers availability. |
Hooks produced the following warnings for commit 957b6ecqt/6.4.2
qt/6.4.1
|
@ericLemanissier Do you think this PR should be merge or should I wait for qt/6.4.3 that include the patch ? |
Better use 6.4.3 instead of adding a short-lived patch, but the discussion is not relevant, as long as C3I does not solve this disk space issue. Please still my two commits https://github.com/conan-io/conan-center-index/pull/16584/commits by the way. I won't be maintaining qt recipe any more. |
I can understand that maintaining this recipe is a huge amount of work! How do you see the future of qt recipe in CCI ? The Qt company used to offer a conan package distribution but I realized today that I missed this blog post: https://www.qt.io/blog/conan-package-manager-pilot-to-end-in-december |
there are already other contributors submitting fixes to qt recipe, so the future of this recipe is no different than any other recipe on CCI. |
Is there a thread about it somewhere ? |
No, but I feel like a documentation/python script stored next to the recipe is easier to find and use than thread on github. |
Yes of course. My full question was "Is there a thread about the ongoing C3I disk space issue ? |
OK. I don't think there is yet |
I created it here: #16857 |
@MartinDelille instead of a python script, I just wrote a conan extension to do the recipe upgrade => conan-io/conan-extensions#20 |
Close in favor of #16889 |
Specify library name and version: qt/6.4.2
Remove .dll extension as QSystemLibrary unconditionally adds it