-
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
vcpkg export with feature does not work anymore #22913
Comments
Yeah, but we may end up deprecating including features because the body of the export is a port instead of a feature. |
@ras0219-msft Is this changes by design? |
Because export does not install or remove packages, the set of features has always had no substantial effect except perhaps failing if the features were not currently installed. Therefore, if you had a configuration before that worked, the workaround is simply to remove the feature list from the packages list. All currently installed features of the selected packages will be exported. However, we'll still need to look into this as a scripting regression and perhaps issue a warning instead that features on the cli have no effect for the export command. Thanks for posting this issue! |
If you consider a scripting environment where the same string is used to install and export the ports, it makes perfectly sens to have the list of features also during export. The alternative would be some magic string handling to remove the features, or using --x-all-installed. The later might be export deprecated packages that have been installed earlier. |
This situation is unsatisfying. The latest version of For OpenCV, a workaround is to patch But that defies the whole point of |
This is a workaround for a upstream regression tracked in microsoft#22913
See daschuer/vcpkg@eba695e and microsoft/vcpkg#22913. Co-authored-by: Daniel Schürmann <[email protected]>
Not very nice workaround, it either requires editing a bunch of build configuration, or doing some not really needed scripting.
The problem appears in scripting environment. It is also related to the package dependencies. So I think the logic here could look like:
|
Any update on this? I rely on |
AFAIU the answer was that features don't make sense for the export command: #22913 (comment) There was the idea of adding a more tolerant behaviour, but maybe this tolerance is no longer relevant after almost two years. |
Thanks. I had misread that comment. The advice in my case would be to use (I had been thinking of |
This is an automated message. Per our repo policy, stale issues get closed if there has been no activity in the past 180 days. The issue will be automatically closed in 14 days. If you wish to keep this issue open, please add a new comment. |
Describe the bug
vcpkg export
fails when using features. This recently broke (probably microsoft/vcpkg-tool#298?).Environment
To Reproduce
Steps to reproduce the behavior:
.\vcpkg.exe install 'abseil[cxx17]'
.\vcpkg export abseil[cxx17] --nuget
.\vcpkg export abseil --nuget
Expected behavior
Exporting a port with feature works as before.
The text was updated successfully, but these errors were encountered: