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.

Sign up for GitHub

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

Jump to bottom

[docs] document VCPKG_INSTALLED_DIR variable #22695

Merged
merged 1 commit into from
Feb 4, 2022
Merged

[docs] document VCPKG_INSTALLED_DIR variable #22695

merged 1 commit into from
Feb 4, 2022

Conversation

strega-nil
Copy link
Contributor

fixes #22669

@Osyotr
Copy link
Contributor

Osyotr commented Jan 23, 2022

By the way, why is default folder ${CMAKE_BINARY_DIR}? With default triplet we get a copy of debug libraries for release config and vise versa. This is a waste of space and time.

@dg0yt
Copy link
Contributor

dg0yt commented Jan 23, 2022

And you get another copy when you use vcpkg in the source dir at the command line...

@JonLiu1993 JonLiu1993 assigned JonLiu1993 and JackBoosY and unassigned JonLiu1993 Jan 24, 2022
@JackBoosY JackBoosY added category:documentation To resolve the issue, documentation will need to be updated info:internal This PR or Issue was filed by the vcpkg team. labels Jan 24, 2022
@BillyONeal BillyONeal merged commit 94c43b8 into microsoft:master Feb 4, 2022
ekilmer added a commit to ekilmer/vcpkg that referenced this pull request Feb 5, 2022
@ekilmer
Merge branch 'master' into more-lief
8d7ebb2 
* master: (221 commits)
  [vcpkg-tool] update to 2022-02-03 (microsoft#22924)
  [opencv4] Disable building cpufeatures since it conflict with libwebp (microsoft#22844)
  [rhasheq] New port (microsoft#22905)
  [sfml] Add arm64 patch to allow SFML to compile on apple silicon (microsoft#22937)
  [popsift] Fix missing Thrust include, already merged upstream. (microsoft#22929)
  [python3][python2] Use MKDIR_P to create directories to avoid race conditions (microsoft#22902)
  Added libe57format port (microsoft#22909)
  update polyhook2 (microsoft#22906)
  [botan] Fix debug info (microsoft#22911)
  [opentelemetry-cpp] update version to v1.2.0 (microsoft#22925)
  [docs] document VCPKG_INSTALLED_DIR variable (microsoft#22695)
  [c89stringutils] New port (microsoft#22904)
  [randomstr] New port (microsoft#22921)
  [docs] Add Authoring-script-ports.md (microsoft#22396)
  [vcpkg_fail_port_install] Deprecate function (microsoft#21489)
  [vcpkg-cmake-config] add missing TOOLS_PATH (microsoft#22863)
  Ace Build Windows - Missing files in include package (microsoft#22880)
  [opendnp3] Disable FetchContent in favor of predownloading (microsoft#22894)
  libraqm update to 0.9.0 (microsoft#22907)
  [google-cloud-cpp] update to latest release (v1.36.0) (microsoft#22897)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ras0219-msft ras0219-msft ras0219-msft approved these changes

@github-actions GitHub Actions github-actions approved these changes

Assignees

@JackBoosY JackBoosY

Labels
category:documentation To resolve the issue, documentation will need to be updated info:internal This PR or Issue was filed by the vcpkg team.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

CMake integration:VCPKG_INSTALLED_DIR is not documented.
7 participants
@strega-nil @Osyotr @dg0yt @ras0219-msft @BillyONeal @JackBoosY @JonLiu1993