Skip to content

Commit 8f98e72

Browse files
committed
Fix accidental cleanup of Native project files during clean build of managed components. Also fixed the path where CMake generated Runtime.vcxproj would drop runtime.lib for ReproNativeCpp and ReproNative projects to pickup.
1 parent d491257 commit 8f98e72

File tree

7 files changed

+13
-13
lines changed

7 files changed

+13
-13
lines changed

build.cmd

+2-2
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,7 @@ echo.
4848

4949
:: Set the remaining variables based upon the determined build configuration
5050
set "__BinDir=%__RootBinDir%\Product\%__BuildOS%.%__BuildArch%.%__BuildType%"
51-
set "__IntermediatesDir=%__RootBinDir%\obj\%__BuildOS%.%__BuildArch%.%__BuildType%"
51+
set "__IntermediatesDir=%__RootBinDir%\obj\Native\%__BuildOS%.%__BuildArch%\
5252
5353
:: Generate path to be set for CMAKE_INSTALL_PREFIX to contain forward slash
5454
set "__CMakeBinDir=%__BinDir%"
@@ -60,7 +60,7 @@ echo Doing a clean build
6060
echo.
6161

6262
:: MSBuild projects would need a rebuild
63-
set __MSBCleanBuildArgs=/t:rebuild
63+
set __MSBCleanBuildArgs=/t:rebuild /p:CleanedTheBuild=1
6464

6565
:: Cleanup the previous output for the selected configuration
6666
if exist "%__BinDir%" rd /s /q "%__BinDir%"

build.proj

+2-2
Original file line numberDiff line numberDiff line change
@@ -32,9 +32,9 @@
3232
<!-- Override RestorePackages from dir.traversal.targets and do a batch restore -->
3333
<Target Name="RestorePackages" DependsOnTargets="BatchRestorePackages" />
3434

35-
<!-- Override clean from dir.traversal.targets and just remove the full BinDir -->
35+
<!-- Override clean from dir.traversal.targets and just remove the full BinDir unless build.[cmd|sh] has already done the cleaning -->
3636
<Target Name="Clean">
37-
<RemoveDir Directories="$(ObjDir)" />
37+
<RemoveDir Condition="'$(CleanedTheBuild)' != '1'" Directories="$(ObjDir)" />
3838
</Target>
3939

4040
</Project>

build.sh

+2-2
Original file line numberDiff line numberDiff line change
@@ -118,7 +118,7 @@ build_managed_corert()
118118
__buildlog=$__scriptpath/msbuild.log
119119

120120
# TODO: Renable running tests
121-
MONO29679=1 ReferenceAssemblyRoot=$__referenceassemblyroot mono $__msbuildpath "$__buildproj" /nologo /verbosity:minimal "/fileloggerparameters:Verbosity=normal;LogFile=$__buildlog" /t:Build /p:SkipTests=true /p:TestNugetRuntimeId=$__TestNugetRuntimeId /p:ToolNugetRuntimeId=$__ToolNugetRuntimeId /p:OSEnvironment=Unix /p:OSGroup=$__BuildOS /p:Configuration=$__BuildType /p:Platform=$__BuildArch /p:UseRoslynCompiler=true /p:COMPUTERNAME=$(hostname) /p:USERNAME=$(id -un) "$@"
121+
MONO29679=1 ReferenceAssemblyRoot=$__referenceassemblyroot mono $__msbuildpath "$__buildproj" /nologo /verbosity:minimal "/fileloggerparameters:Verbosity=normal;LogFile=$__buildlog" /t:Build /p:CleanedTheBuild=$(__CleanBuild) /p:SkipTests=true /p:TestNugetRuntimeId=$__TestNugetRuntimeId /p:ToolNugetRuntimeId=$__ToolNugetRuntimeId /p:OSEnvironment=Unix /p:OSGroup=$__BuildOS /p:Configuration=$__BuildType /p:Platform=$__BuildArch /p:UseRoslynCompiler=true /p:COMPUTERNAME=$(hostname) /p:USERNAME=$(id -un) "$@"
122122
BUILDERRORLEVEL=$?
123123

124124
echo
@@ -232,7 +232,7 @@ BUILDERRORLEVEL=0
232232

233233
# Set the various build properties here so that CMake and MSBuild can pick them up
234234
__UnprocessedBuildArgs=
235-
__CleanBuild=false
235+
__CleanBuild=0
236236
__VerboseBuild=false
237237
__ClangMajorVersion=3
238238
__ClangMinorVersion=5

src/ILToNative/reproNative/reproNative.sln

+1-1
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ VisualStudioVersion = 14.0.23107.0
55
MinimumVisualStudioVersion = 10.0.40219.1
66
Project("{8BC9CEB8-8B4A-11D0-8D11-00A0C91BC942}") = "reproNative", "reproNative.vcxproj", "{ECB5D162-A31B-45FF-87C7-2E92BD445F5A}"
77
EndProject
8-
Project("{8BC9CEB8-8B4A-11D0-8D11-00A0C91BC942}") = "Runtime", "..\..\..\bin\obj\Windows_NT.x64.Debug\Runtime\Full\Runtime.vcxproj", "{82CD69C7-FA29-45D3-A87C-2FCA61CE7613}"
8+
Project("{8BC9CEB8-8B4A-11D0-8D11-00A0C91BC942}") = "Runtime", "..\..\..\bin\obj\Native\Windows_NT.x64\Runtime\Full\Runtime.vcxproj", "{82CD69C7-FA29-45D3-A87C-2FCA61CE7613}"
99
EndProject
1010
Global
1111
GlobalSection(SolutionConfigurationPlatforms) = preSolution

src/ILToNative/reproNative/reproNative.vcxproj

+2-2
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@
5959
<Link>
6060
<SubSystem>Console</SubSystem>
6161
<GenerateDebugInformation>true</GenerateDebugInformation>
62-
<AdditionalDependencies>kernel32.lib;user32.lib;gdi32.lib;winspool.lib;comdlg32.lib;advapi32.lib;shell32.lib;ole32.lib;oleaut32.lib;uuid.lib;odbc32.lib;odbccp32.lib;%(AdditionalDependencies);..\..\..\bin\Product\Windows_NT.x64.Debug\lib\Runtime.lib</AdditionalDependencies>
62+
<AdditionalDependencies>kernel32.lib;user32.lib;gdi32.lib;winspool.lib;comdlg32.lib;advapi32.lib;shell32.lib;ole32.lib;oleaut32.lib;uuid.lib;odbc32.lib;odbccp32.lib;%(AdditionalDependencies);..\..\..\bin\obj\Native\Windows_NT.x64\Runtime\Debug\Runtime.lib</AdditionalDependencies>
6363
</Link>
6464
</ItemDefinitionGroup>
6565
<ItemDefinitionGroup Condition="'$(Configuration)|$(Platform)'=='Release|x64'">
@@ -80,7 +80,7 @@
8080
<GenerateDebugInformation>true</GenerateDebugInformation>
8181
<EnableCOMDATFolding>true</EnableCOMDATFolding>
8282
<OptimizeReferences>true</OptimizeReferences>
83-
<AdditionalDependencies>kernel32.lib;user32.lib;gdi32.lib;winspool.lib;comdlg32.lib;advapi32.lib;shell32.lib;ole32.lib;oleaut32.lib;uuid.lib;odbc32.lib;odbccp32.lib;%(AdditionalDependencies);..\..\..\bin\Product\Windows_NT.x64.Release\lib\Runtime.lib</AdditionalDependencies>
83+
<AdditionalDependencies>kernel32.lib;user32.lib;gdi32.lib;winspool.lib;comdlg32.lib;advapi32.lib;shell32.lib;ole32.lib;oleaut32.lib;uuid.lib;odbc32.lib;odbccp32.lib;%(AdditionalDependencies);..\..\..\bin\obj\Native\Windows_NT.x64\Runtime\Release\Runtime.lib</AdditionalDependencies>
8484
</Link>
8585
</ItemDefinitionGroup>
8686
<ItemGroup>

src/ILToNative/reproNative/reproNativeCpp.sln

+1-1
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ Project("{8BC9CEB8-8B4A-11D0-8D11-00A0C91BC942}") = "reproNativeCpp", "reproNati
88
{82CD69C7-FA29-45D3-A87C-2FCA61CE7613} = {82CD69C7-FA29-45D3-A87C-2FCA61CE7613}
99
EndProjectSection
1010
EndProject
11-
Project("{8BC9CEB8-8B4A-11D0-8D11-00A0C91BC942}") = "Runtime", "..\..\..\bin\obj\Windows_NT.x64.Debug\Runtime\Portable\PortableRuntime.vcxproj", "{82CD69C7-FA29-45D3-A87C-2FCA61CE7613}"
11+
Project("{8BC9CEB8-8B4A-11D0-8D11-00A0C91BC942}") = "Runtime", "..\..\..\bin\obj\Native\Windows_NT.x64\Runtime\Portable\PortableRuntime.vcxproj", "{82CD69C7-FA29-45D3-A87C-2FCA61CE7613}"
1212
EndProject
1313
Global
1414
GlobalSection(SolutionConfigurationPlatforms) = preSolution

src/ILToNative/reproNative/reproNativeCpp.vcxproj

+3-3
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,7 @@
6262
<Link>
6363
<SubSystem>Console</SubSystem>
6464
<GenerateDebugInformation>true</GenerateDebugInformation>
65-
<AdditionalDependencies>kernel32.lib;user32.lib;gdi32.lib;winspool.lib;comdlg32.lib;advapi32.lib;shell32.lib;ole32.lib;oleaut32.lib;uuid.lib;odbc32.lib;odbccp32.lib;%(AdditionalDependencies);..\..\..\bin\Product\Windows_NT.x64.Debug\lib\PortableRuntime.lib</AdditionalDependencies>
65+
<AdditionalDependencies>kernel32.lib;user32.lib;gdi32.lib;winspool.lib;comdlg32.lib;advapi32.lib;shell32.lib;ole32.lib;oleaut32.lib;uuid.lib;odbc32.lib;odbccp32.lib;%(AdditionalDependencies);..\..\..\bin\obj\Native\Windows_NT.x64\Runtime\Debug\PortableRuntime.lib</AdditionalDependencies>
6666
</Link>
6767
</ItemDefinitionGroup>
6868
<ItemDefinitionGroup Condition="'$(Configuration)|$(Platform)'=='Release|x64'">
@@ -75,15 +75,15 @@
7575
<PreprocessorDefinitions>CPPCODEGEN;WIN32;NDEBUG;_CONSOLE;_LIB;%(PreprocessorDefinitions)</PreprocessorDefinitions>
7676
<SDLCheck>true</SDLCheck>
7777
<PrecompiledHeaderFile>common.h</PrecompiledHeaderFile>
78-
<AdditionalIncludeDirectories>..\gc;..\..\Native\gc\env;..\..\Native\gc\sample</AdditionalIncludeDirectories>
78+
<AdditionalIncludeDirectories>..\..\Native\gc;..\..\Native\gc\env;..\..\Native\gc\sample</AdditionalIncludeDirectories>
7979
<DisableSpecificWarnings>4477</DisableSpecificWarnings>
8080
</ClCompile>
8181
<Link>
8282
<SubSystem>Console</SubSystem>
8383
<GenerateDebugInformation>true</GenerateDebugInformation>
8484
<EnableCOMDATFolding>true</EnableCOMDATFolding>
8585
<OptimizeReferences>true</OptimizeReferences>
86-
<AdditionalDependencies>kernel32.lib;user32.lib;gdi32.lib;winspool.lib;comdlg32.lib;advapi32.lib;shell32.lib;ole32.lib;oleaut32.lib;uuid.lib;odbc32.lib;odbccp32.lib;%(AdditionalDependencies);..\..\..\bin\Product\Windows_NT.x64.Release\lib\PortableRuntime.lib</AdditionalDependencies>
86+
<AdditionalDependencies>kernel32.lib;user32.lib;gdi32.lib;winspool.lib;comdlg32.lib;advapi32.lib;shell32.lib;ole32.lib;oleaut32.lib;uuid.lib;odbc32.lib;odbccp32.lib;%(AdditionalDependencies);..\..\..\bin\obj\Native\Windows_NT.x64\Runtime\Release\PortableRuntime.lib</AdditionalDependencies>
8787
</Link>
8888
</ItemDefinitionGroup>
8989
<ItemGroup>

0 commit comments

Comments
 (0)