Skip to content
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

[depend-info] Fix bugs, add --sort, --show-depth and --max-recurse options #7643

Merged
merged 8 commits into from
Aug 14, 2019

Conversation

vicroms
Copy link
Member

@vicroms vicroms commented Aug 11, 2019

This PR changes the following:

  • Fix issue where vpckg depend-info will not list the dependencies added by Default-Features.
  • Make vcpkg depend-info follow the same syntax as vcpkg install.
    • Add support for triplets.
  • Show features required on dependencies.
  • Add option to sort output --sort=sort_mode, where sort-mode is one of the following:
    • lexicographical: sorts packages lexicographically,
    • topological: (default) required install order to satisfy all dependencies,
    • reverse: reverse topological (requested port first, then direct dependencies, then depth 2 dependencies and so on...)
  • Add option to set recursion limit (--max-recurse=n).
    • --max-recurse=n where n is a positive integer, sets the max recursion depth.
    • --max-recurse=0 is equivalent to the now removed --no-recurse option.
    • --max-recurse=-1 for no recursion limit.
  • Add option to show recursion depth (--show-depth).

TODO:

  • Restore --no-recurse, or replace with new option: --max-recurse.
  • Restore --dot option.
  • Restore --dgml option.

Example (now with colors!):

depend-info-with-colors

@vicroms vicroms added category:vcpkg-bug The issue is with the vcpkg system (including helper scripts in `scripts/cmake/`) wip info:internal This PR or Issue was filed by the vcpkg team. labels Aug 11, 2019
@vicroms vicroms self-assigned this Aug 11, 2019
@vicroms vicroms changed the title Pr/depend info [vcpkg_depend_info] List dependencies added by Default-Features Aug 11, 2019
@vicroms vicroms changed the title [vcpkg_depend_info] List dependencies added by Default-Features [vcpkg depend-info] List dependencies added by Default-Features Aug 11, 2019
@vicroms vicroms added category:vcpkg-feature The issue is a new capability of the tool that doesn’t already exist and we haven’t committed and removed category:vcpkg-bug The issue is with the vcpkg system (including helper scripts in `scripts/cmake/`) labels Aug 12, 2019
@vicroms vicroms marked this pull request as ready for review August 14, 2019 22:36
@vicroms vicroms changed the title [vcpkg depend-info] List dependencies added by Default-Features [depend-info] Fix bugs, add --sort, --show-depth and --max-recurse options Aug 14, 2019
@vicroms vicroms merged commit edaf3bf into microsoft:master Aug 14, 2019
@vicroms vicroms deleted the pr/depend-info branch October 27, 2020 06:14
strega-nil pushed a commit to strega-nil/vcpkg that referenced this pull request May 5, 2021
…se` options (microsoft#7643)

* [depend-info] Follow same rules as vcpkg install

* [depend-info] Add --max-depth and --sort options

* [depend-info] Improve output readability (a tiny bit)

* [depend-info] Add --show-depth option

* [depend-info] Fix build on VS 2015

* [depend-info] Fix output of --dot and --dgml
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:vcpkg-feature The issue is a new capability of the tool that doesn’t already exist and we haven’t committed info:internal This PR or Issue was filed by the vcpkg team.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant