Skip to content

[catch2] Don't install docs#24046

Merged
strega-nil-ms merged 2 commits intomicrosoft:masterfrom
Thomas1664:catch2
Apr 11, 2022
Merged

[catch2] Don't install docs#24046
strega-nil-ms merged 2 commits intomicrosoft:masterfrom
Thomas1664:catch2

Conversation

@Thomas1664
Copy link
Copy Markdown
Contributor

Describe the pull request

  • What does your PR fix?

    Disable installing docs

  • Which triplets are supported/not supported? Have you updated the CI baseline?

    unchanged

  • Does your PR follow the maintainer guide?

Yes

  • If you have added/updated a port: Have you run ./vcpkg x-add-version --all and committed the result?

    Yes

If you are still working on the PR, open it as a Draft: https://github.blog/2019-02-14-introducing-draft-pull-requests/

@FrankXie05 FrankXie05 added category:port-feature The issue is with a library, which is requesting new capabilities that didn’t exist requires:author-response labels Apr 11, 2022
@Thomas1664
Copy link
Copy Markdown
Contributor Author

@FrankXie05 Why requires:author-response?

@FrankXie05
Copy link
Copy Markdown
Contributor

@Thomas1664 Could you please explain why want to turn off the option to download documents? :)

@Thomas1664
Copy link
Copy Markdown
Contributor Author

@Thomas1664 Could you please explain why want to turn off the option to download documents? :)

Our policy is to not install docs by default.
Copying docs takes time (not much here, though)

@FrankXie05 FrankXie05 added the info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this. label Apr 11, 2022
@strega-nil-ms
Copy link
Copy Markdown
Contributor

Thanks!

@strega-nil-ms strega-nil-ms merged commit 6f9687b into microsoft:master Apr 11, 2022
@Thomas1664 Thomas1664 deleted the catch2 branch April 11, 2022 15:50
ekilmer added a commit to ekilmer/vcpkg that referenced this pull request Apr 12, 2022
* master: (139 commits)
  [dstorage] Add port for Microsoft.Direct3D.DirectStorage NuGet (microsoft#24063)
  [vcpkg] Refactor toolchain & generator selection (microsoft#23846)
  [icu] update to 70.1 (microsoft#23445)
  [vcpkg] Update android usage documentation (microsoft#23690)
  [LMDB] update to 0.9.29 (microsoft#24045)
  [catch2] Don't install docs (microsoft#24046)
  [harfbuff] fix arm64 osx build (microsoft#24055)
  [openxr-loader] remove from CI baseline (microsoft#24057)
  [imath] Update to 3.1.5 (microsoft#24059)
  [openssl] Fix dynamic builds on UNIX (microsoft#24061)
  [c-ares] update to 1.18.1 (microsoft#24062)
  [igraph] update to 0.9.8 (microsoft#24065)
  [cmake-user] Fix library check (microsoft#24070)
  [openxr-loader] fix ci.baseline.txt (microsoft#24073)
  [tinycbor] Fix file conflicts with libcbor (microsoft#24056)
  [graphviz,libslirp] Limit msys to windows (microsoft#24032)
  [bdwgc] Don't build docs (microsoft#24025)
  [capstone] update to 5.0.0-rc2 (microsoft#23979)
  [clockutils] Fix x64-windows-static-md (microsoft#23965)
  [braft] New port (microsoft#23830)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category:port-feature The issue is with a library, which is requesting new capabilities that didn’t exist info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants