Discussion:
[win-pv-devel] [xenvkbd PATCH] Conditionally package DPInst
Owen Smith
2018-08-14 13:09:33 UTC
Permalink
Since DPInst.exe is not shipped with the Windows Driver Kit 10, an
environment variable must point to local copies. Make the inclusion of
DPInst conditional on DPINST_REDIST being defined and that path
existing. This simplifies building packages which do not require DPInst
for installation, and removes a required step to create a working build.

Signed-off-by: Owen Smith <***@citrix.com>
---
vs2015/package/package.vcxproj | 2 +-
vs2017/package/package.vcxproj | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/vs2015/package/package.vcxproj b/vs2015/package/package.vcxproj
index d2baa4c..db5c2e0 100644
--- a/vs2015/package/package.vcxproj
+++ b/vs2015/package/package.vcxproj
@@ -39,7 +39,7 @@
<Project>{2BFAC7E6-3420-47A5-A092-BDC5C9D78A44}</Project>
</ProjectReference>
</ItemGroup>
- <ItemGroup>
+ <ItemGroup Condition="Exists('$(DPINST_REDIST)')">
<FilesToPackage Include="$(DPINST_REDIST)\x86\dpinst.exe" Condition="'$(Platform)'=='Win32'" />
<FilesToPackage Include="$(DPINST_REDIST)\x64\dpinst.exe" Condition="'$(Platform)'=='x64'" />
</ItemGroup>
diff --git a/vs2017/package/package.vcxproj b/vs2017/package/package.vcxproj
index 3827ea4..f32de24 100644
--- a/vs2017/package/package.vcxproj
+++ b/vs2017/package/package.vcxproj
@@ -39,7 +39,7 @@
<Project>{2BFAC7E6-3420-47A5-A092-BDC5C9D78A44}</Project>
</ProjectReference>
</ItemGroup>
- <ItemGroup>
+ <ItemGroup Condition="Exists('$(DPINST_REDIST)')">
<FilesToPackage Include="$(DPINST_REDIST)\x86\dpinst.exe" Condition="'$(Platform)'=='Win32'" />
<FilesToPackage Include="$(DPINST_REDIST)\x64\dpinst.exe" Condition="'$(Platform)'=='x64'" />
</ItemGroup>
--
2.16.2.windows.1
Loading...