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
[icu] Add reminder to install autoconf-archive #38019
[icu] Add reminder to install autoconf-archive #38019
Conversation
I canceled the CI Pipeline run because it only added a reminder and did not change the build process. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Disapproval.
When you need autoconf-archive, you also need additional packages.
So do you think it's better to keep the current status or add other additional packages to the reminder? Or is there another plan? Looking forward to your reply. |
I don't think the current status is acceptable. There is even a guideline that ports should print such hints. I wonder why it took so long to see action. It has been reported several times IIRC.
Yes, please. There is prior art. For example python3.
There is no plan. However, I still think that the current way of adding these warnings is immature. The ports should just declare dependencies and delegate actions to a shared facility. But what we get is each port carrying a (different) copy of explicit instructions for installing dependencies. |
Again. |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
Fixes #38005.
By user request, add a reminder to install
autoconf-archive
on non-windows machines.SHA512s are updated for each updated download.The "supports" clause reflects platforms that may be fixed by this new version.Any patches that are no longer applied are deleted from the port's directory../vcpkg x-add-version --all
and committing the result.