Replies: 3 comments
-
+1 ^ |
Beta Was this translation helpful? Give feedback.
0 replies
-
Hi, I saw that there was changes in scripts/buildsystems/msbuild/applocal.ps1 to the logic that looks for dumpbin. With the new changes, I think it would be preferrable if applocal.ps1 finds dumpbin.exe in a Visual Studio installation using the same logic that is used by bootstrap-vcpkg.bat. |
Beta Was this translation helpful? Give feedback.
0 replies
-
@georg-emg Do you still have the PR you mentioned? If so, can you provide a link to it. Thanks |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
Is your feature request related to a problem? Please describe.
When creating a cmake toolchain under Windows, all the relevant executables can be specified using cmake variables like
CMAKE_CXX_COMPILER
CMAKE_RC_COMPILER
CMAKE_MT
etc. This allows for writing a toolchain file that can be run from a regular shell without callingvcvarsall.bat
, which makes writing CI pipelines a lot easier. These toolchain files can include thevcpkg.cmake
toolchain file to take advantage of the advanced vcpkg features. When trying to use theVCPKG_APPLOCAL_DEPS
feature, however, the scriptapplocal.ps1
will attempt to calldumpbin
without a path. This results in an error ifdumbin
is not in the PATH, i.e. ifvcvarsall.bat
has not been called.Proposed solution
The
vcpkg.cmake
toockain file could provide a cache variableVCPKG_DUMPBIN
by callingfind_program(VCPKG_DUMPBIN, dumpbin)
, and pass the result on toapplocal.ps1
as a parameter. This would allow for specifying the path todumpbin
from the cmake command line, or from a toolchain file.Describe alternatives you've considered
The two possible workarounds are:
vcvarsall.bat
to be called before building a project that uses vcpkg. This kind of defeats the purpose of a cmake toolchain file, though, which should describe the whole build environment.VCPKG_APPLOCAL_DEPS
toOFF
, which would be a pity.Additional context
This is just a small change to
scripts/buildsystems/vcpkg.cmake
andscripts/buildsystems/msbuild/applocal.ps1
each. I have a working patch and could submit a PR if there is interest.Beta Was this translation helpful? Give feedback.
All reactions