Skip to content

Commit 8830cb5

Browse files
author
MarcoFalke
committed
Merge #17741: build: Included test_bitcoin-qt in msvc build
592af5a Moved the include of the system projects to before the build depends on task. Otherwise it doesn't get run. (Aaron Clauson) 6e22151 Included test_bitcoin-qt in msvc build. (Aaron Clauson) Pull request description: This PR includes the `test_bitcoin-qt` project in the msvc build. The project is already in the repo but is not part of the solution and therefore does not get built. The test executable output from this project does not pass successfully on Windows (it may never have). This PR only builds the project and does not add a step to execute the tests. MarcoFalke mentioned the fact that it's missing in #17571. Top commit has no ACKs. Tree-SHA512: ebb8fc7710022b995e842708bbf83b9c0084dac5e1a22fa3f53d9ea472faabe96d75ad53150cc915ded5f782515a896f2b60840390bde38c1d0f80d0fd56a4ea
2 parents b931f61 + 592af5a commit 8830cb5

File tree

2 files changed

+16
-9
lines changed

2 files changed

+16
-9
lines changed

build_msvc/bitcoin.sln

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -46,6 +46,8 @@ Project("{8BC9CEB8-8B4A-11D0-8D11-00A0C91BC942}") = "bitcoin-qt", "bitcoin-qt\bi
4646
EndProject
4747
Project("{8BC9CEB8-8B4A-11D0-8D11-00A0C91BC942}") = "libtest_util", "libtest_util\libtest_util.vcxproj", "{868474FD-35F6-4400-8EED-30A33E7521D4}"
4848
EndProject
49+
Project("{8BC9CEB8-8B4A-11D0-8D11-00A0C91BC942}") = "test_bitcoin-qt", "test_bitcoin-qt\test_bitcoin-qt.vcxproj", "{51201D5E-D939-4854-AE9D-008F03FF518E}"
50+
EndProject
4951
Global
5052
GlobalSection(SolutionConfigurationPlatforms) = preSolution
5153
Debug|x64 = Debug|x64
@@ -230,6 +232,14 @@ Global
230232
{868474FD-35F6-4400-8EED-30A33E7521D4}.Release|x64.Build.0 = Release|x64
231233
{868474FD-35F6-4400-8EED-30A33E7521D4}.Release|x86.ActiveCfg = Release|Win32
232234
{868474FD-35F6-4400-8EED-30A33E7521D4}.Release|x86.Build.0 = Release|Win32
235+
{51201D5E-D939-4854-AE9D-008F03FF518E}.Debug|x64.ActiveCfg = Debug|x64
236+
{51201D5E-D939-4854-AE9D-008F03FF518E}.Debug|x64.Build.0 = Debug|x64
237+
{51201D5E-D939-4854-AE9D-008F03FF518E}.Debug|x86.ActiveCfg = Debug|Win32
238+
{51201D5E-D939-4854-AE9D-008F03FF518E}.Debug|x86.Build.0 = Debug|Win32
239+
{51201D5E-D939-4854-AE9D-008F03FF518E}.Release|x64.ActiveCfg = Release|x64
240+
{51201D5E-D939-4854-AE9D-008F03FF518E}.Release|x64.Build.0 = Release|x64
241+
{51201D5E-D939-4854-AE9D-008F03FF518E}.Release|x86.ActiveCfg = Release|Win32
242+
{51201D5E-D939-4854-AE9D-008F03FF518E}.Release|x86.Build.0 = Release|Win32
233243
EndGlobalSection
234244
GlobalSection(SolutionProperties) = preSolution
235245
HideSolutionNode = FALSE

build_msvc/test_bitcoin-qt/test_bitcoin-qt.vcxproj

Lines changed: 6 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,9 @@
11
<?xml version="1.0" encoding="utf-8"?>
2-
<Project DefaultTargets="Build" ToolsVersion="15.0" xmlns="http://schemas.microsoft.com/developer/msbuild/2003">
2+
<Project DefaultTargets="Build" xmlns="http://schemas.microsoft.com/developer/msbuild/2003">
33
<Import Project="..\common.init.vcxproj" />
44
<Import Project="..\common.qt.init.vcxproj" />
55
<PropertyGroup Label="Globals">
66
<ProjectGuid>{51201D5E-D939-4854-AE9D-008F03FF518E}</ProjectGuid>
7-
</PropertyGroup>
8-
<PropertyGroup Label="Configuration">
97
<ConfigurationType>Application</ConfigurationType>
108
<OutDir>$(SolutionDir)$(Platform)\$(Configuration)\</OutDir>
119
</PropertyGroup>
@@ -66,12 +64,15 @@
6664
</ProjectReference>
6765
</ItemGroup>
6866

67+
<Import Project="$(VCTargetsPath)\Microsoft.Cpp.props" />
68+
<Import Project="$(VCTargetsPath)\Microsoft.Cpp.targets" />
69+
6970
<ItemDefinitionGroup Condition="'$(Configuration)|$(Platform)'=='Release|x64'">
7071
<ClCompile>
7172
<AdditionalIncludeDirectories>..\libbitcoin_qt\$(GeneratedFilesOutDir)\..\;$(QtIncludeDir)\QtTest;$(QtIncludes);%(AdditionalIncludeDirectories)</AdditionalIncludeDirectories>
7273
</ClCompile>
7374
<Link>
74-
<AdditionalDependencies>$(QtReleaseLibaries);%(AdditionalDependencies)</AdditionalDependencies>
75+
<AdditionalDependencies>$(QtLibraryDir)\Qt5Test.lib;$(QtReleaseLibraries);%(AdditionalDependencies)</AdditionalDependencies>
7576
</Link>
7677
</ItemDefinitionGroup>
7778

@@ -83,12 +84,10 @@
8384
<AdditionalDependencies>$(QtDebugLibraries);%(AdditionalDependencies)</AdditionalDependencies>
8485
</Link>
8586
</ItemDefinitionGroup>
86-
8787
<ItemGroup>
8888
<MocTestFiles Include="..\..\src\qt\test\addressbooktests.h" />
8989
<MocTestFiles Include="..\..\src\qt\test\apptests.h" />
9090
<MocTestFiles Include="..\..\src\qt\test\compattests.h" />
91-
<MocTestFiles Include="..\..\src\qt\test\paymentservertests.h" />
9291
<MocTestFiles Include="..\..\src\qt\test\rpcnestedtests.h" />
9392
<MocTestFiles Include="..\..\src\qt\test\uritests.h" />
9493
<MocTestFiles Include="..\..\src\qt\test\wallettests.h" />
@@ -106,8 +105,6 @@
106105
<RemoveDir Directories="$(GeneratedFilesOutDir)\moc\*" />
107106
<RemoveDir Directories="$(GeneratedFilesOutDir)\moc" />
108107
</Target>
109-
<Import Project="$(VCTargetsPath)\Microsoft.Cpp.props" />
110-
<Import Project="$(VCTargetsPath)\Microsoft.Cpp.targets" />
111108
<PropertyGroup>
112109
<BuildDependsOn>
113110
moccode;
@@ -120,4 +117,4 @@
120117
$(CleanDependsOn);
121118
</CleanDependsOn>
122119
</PropertyGroup>
123-
</Project>
120+
</Project>

0 commit comments

Comments
 (0)