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

[vcpkg_acquire_msys] Error when SOURCE_PATH is not defined #9729

Closed
JackBoosY opened this issue Jan 17, 2020 · 1 comment
Closed

[vcpkg_acquire_msys] Error when SOURCE_PATH is not defined #9729

JackBoosY opened this issue Jan 17, 2020 · 1 comment
Assignees
@vicroms @JackBoosY
Labels
category:vcpkg-bug The issue is with the vcpkg system (including helper scripts in `scripts/cmake/`)

Comments

@JackBoosY
Copy link
Contributor

Describe the bug
When we use vcpkg_acquire_msys, if SOURCE_PATH is not defined, we will encounter an error:

CMake Error at scripts/cmake/vcpkg_execute_required_process.cmake:50 (_execute_process):
  _execute_process called with no value for WORKING_DIRECTORY.
Call Stack (most recent call first):
  scripts/cmake/vcpkg_acquire_msys.cmake:119 (vcpkg_execute_required_process)

Related code:

vcpkg/scripts/cmake/vcpkg_acquire_msys.cmake

Line 122 in bf3e071

WORKING_DIRECTORY ${SOURCE_PATH}

@JackBoosY JackBoosY added the category:vcpkg-bug The issue is with the vcpkg system (including helper scripts in `scripts/cmake/`) label Jan 17, 2020
@JackBoosY JackBoosY self-assigned this Jan 17, 2020
@JackBoosY JackBoosY mentioned this issue Jan 17, 2020
Closed
@LilyWangLL
Copy link
Contributor

LilyWangLL commented Jun 9, 2022 •
edited
Loading

Fixed by PR #11810

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@vicroms vicroms

@JackBoosY JackBoosY

Labels
category:vcpkg-bug The issue is with the vcpkg system (including helper scripts in `scripts/cmake/`)
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@vicroms @JackBoosY @LilyWangLL