Welcome Guest! To enable all features please Login or Register.

Notification

Icon
Error
Build failure when MSBuildProjectExtensionsPath is set
Options
Previous Topic Next Topic
yizhou
#1 Posted : Friday, September 23, 2022 4:05:44 AM(UTC)
Rank: Newbie

Groups: Registered
Joined: 9/23/2022(UTC)
Posts: 3
When MSBuildProjectExtensionsPath is set in Directory.Build.props, the nuget props file (nuget.g.props, which defines Pkg* variables) are not read by NCrunch, so references using Pkg* variable are missing.
Seems that NCrunch is using BaseIntermediateOutputPath to read the nuget props file.

Directory.Build.props:
Code:
<Project>
  <PropertyGroup>
    <MSBuildProjectExtensionsPath>.pkgref</MSBuildProjectExtensionsPath>
  </PropertyGroup>
</Project>

csproj:
Code:
<Project Sdk="Microsoft.NET.Sdk">
  <PropertyGroup>
    <TargetFramework>net472</TargetFramework>
  </PropertyGroup>
  <ItemGroup>
    <PackageReference Include="Microsoft.WindowsAzure.SDK" Version="2.9.0" ExcludeAssets="all" GeneratePathProperty="true" />
    <Reference Include="Microsoft.WindowsAzure.ServiceRuntime">
      <HintPath>$(PkgMicrosoft_WindowsAzure_SDK)\lib\Microsoft.WindowsAzure.ServiceRuntime.dll</HintPath>
    </Reference>
  </ItemGroup>
</Project>

cs:
Code:
public class Class1
{
    public bool Method1() => Microsoft.WindowsAzure.ServiceRuntime.RoleEnvironment.IsAvailable;
}


csproj in NCrunch workspace: (The reference is missing because PkgMicrosoft_WindowsAzure_SDK is not recognized by NCrunch.)
Code:
<Project DefaultTargets="Build;NCrunchOutputDataQuery" Sdk="Microsoft.NET.Sdk">
  <PropertyGroup>
    <TargetFramework>net472</TargetFramework>
  </PropertyGroup>
  <ItemGroup>
    <PackageReference Include="Microsoft.WindowsAzure.SDK" Version="2.9.0" ExcludeAssets="all" GeneratePathProperty="true" />
  </ItemGroup>
  <Target Name="NCrunchOutputDataQuery" DependsOnTargets="">
    <Message Text="#Expression_3f62eece29104339bc4528ab9a1f216f=$(OutDir)$(TargetName)$(TargetExt)" />
    <Message Text="#Expression_c5432365aba34c2597b01269efa1232a=$(Configuration)" />
    <Message Text="#Expression_25a377558861465a82d14d000b75fedf=$(TargetName)" />
    <Message Text="#$TargetFrameworkDirectory=;$(TargetFrameworkDirectory)" />
    <Message Text="#ReferencePath=;@(ReferencePath)" />
    <Message Text="#ReferenceCopyLocalPaths=;@(ReferenceCopyLocalPaths)" />
    <Message Text="#TargetPathWithTargetPlatformMoniker=;@(TargetPathWithTargetPlatformMoniker)" />
    <Message Text="#CopiedFiles_SourceItemsToCopyToOutputDirectoryAlways=@(_SourceItemsToCopyToOutputDirectoryAlways)|@(_SourceItemsToCopyToOutputDirectoryAlways->'$(OutDir)%(TargetPath)')" Condition="'@(_SourceItemsToCopyToOutputDirectoryAlways->'$(OutDir)%(TargetPath)')'!=''" />
    <Message Text="#CopiedFiles_SourceItemsToCopyToOutputDirectory=@(_SourceItemsToCopyToOutputDirectory)|@(_SourceItemsToCopyToOutputDirectory->'$(OutDir)%(TargetPath)')" Condition="'@(_SourceItemsToCopyToOutputDirectory->'$(OutDir)%(TargetPath)')'!=''" />
    <Message Text="#CopiedFilesAllItemsFullPathWithTargetPath=@(AllItemsFullPathWithTargetPath)|@(AllItemsFullPathWithTargetPath->'$(OutDir)%(TargetPath)')" Condition="'@(AllItemsFullPathWithTargetPath->'$(OutDir)%(TargetPath)')'!=''" />
  </Target>
  <Target Name="NCrunchPrepareForBuild" DependsOnTargets="">
  </Target>
</Project>

Build error in NCrunch:
Quote:
Class1.cs (3, 30): The type or namespace name 'WindowsAzure' does not exist in the namespace 'Microsoft' (are you missing an assembly reference?)
Back to top
Remco
#2 Posted : Friday, September 23, 2022 11:28:17 PM(UTC)
Rank: NCrunch Developer

Groups: Administrators
Joined: 4/16/2011(UTC)
Posts: 6,974

Thanks: 929 times
Was thanked: 1256 time(s) in 1169 post(s)
Hi, thanks for sharing this.

You are correct. Changing this property will break NCrunch. To enable NCrunch to work with MSBuild, we need to override certain things and make adjustments inside the build. This does mean there are certain things we simply can't support (like changing this property).

I'm not sure what you're trying to achieve, but does suppressing the property declaration resolve this for you?

Code:
<Project>
  <PropertyGroup Condition="'$(NCrunch)' != '1'">
    <MSBuildProjectExtensionsPath>.pkgref</MSBuildProjectExtensionsPath>
  </PropertyGroup>
</Project>
Back to top
 |  Edit by user
yizhou
#3 Posted : Saturday, September 24, 2022 3:20:23 AM(UTC)
Rank: Newbie

Groups: Registered
Joined: 9/23/2022(UTC)
Posts: 3
Thanks Remco.
Our build system relies on setting this MSBuildProjectExtensionsPath property.
I've tried suppressing it for NCrunch before, but I got this error that obj\project.assets.json is not found. The generated files are in .pkgref folder as we set in MSBuildProjectExtensionsPath.
Could we let NCrunch run nuget restore for itself?
Quote:
System.Exception: An exception was thrown in the remote environment: System.Reflection.TargetInvocationException: Exception has been thrown by the target of an invocation. ---> nCrunch.Common.UserException: Errors occurred while trying to load the project file:
Assets file 'C:\Users\yizmo\source\repos\NCrunchBuildFailure\NCrunchBuildFailure\obj\project.assets.json' not found. Run a NuGet package restore to generate this file.
at nCrunch.Compiler.ComponentAnalysisContext.()
at nCrunch.Compiler.ComponentAnalysisContext.AnalyseComponentBuild()
at nCrunch.Compiler.RemoteBuildRunner.AnalyseComponentBuild(ComponentLoadParameters parameters)
--- End of inner exception stack trace ---
at System.RuntimeMethodHandle.InvokeMethod(Object target, Object[] arguments, Signature sig, Boolean constructor)
at System.Reflection.RuntimeMethodInfo.UnsafeInvokeInternal(Object obj, Object[] parameters, Object[] arguments)
at System.Reflection.RuntimeMethodInfo.Invoke(Object obj, BindingFlags invokeAttr, Binder binder, Object[] parameters, CultureInfo culture)
at nCrunch.TaskRunner.Ipc.IpcMessageProcessor.(CallMethodMessage )
at nCrunch.TaskRunner.Ipc.IpcMessageProcessor.ProcessMessageReturningResult(Byte[] data)
at nCrunch.TaskRunner.Ipc.RemoteInstance.(Byte[] )
at nCrunch.TaskRunner.Ipc.RemoteInstance.Invoke(IMessage msg)
at System.Runtime.Remoting.Proxies.RealProxy.PrivateInvoke(MessageData& msgData, Int32 type)
at nCrunch.Compiler.IRemoteBuildRunner.AnalyseComponentBuild(ComponentLoadParameters parameters)
at nCrunch.Core.BuildManagement.BuildProcessLauncher..(IRemoteBuildRunner )
at nCrunch.Core.BuildManagement.BuildProcessLauncher.(Action`1 , FilePath , String , ExternalProcess )
at nCrunch.Core.BuildManagement.BuildProcessLauncher.(Action`1 , EffectiveProcessorArchitecture , GridClientId , BuildSystemParameters , IList`1 , Nullable`1 , GridAddress , DotNetCoreSdkVersion  )
at nCrunch.Core.BuildManagement.BuildProcessLauncher.AnalyseComponentBuildInExternalProcess(ComponentLoadParameters parameters, IList`1 customEnvironmentVariables)
at nCrunch.Client.ComponentLoader.SnapshotComponentLoader.(EffectiveProcessorArchitecture , String )
at nCrunch.Client.ComponentLoader.SnapshotComponentLoader.CreateComponentFromXml(FilePath projectFilePath, ParsedBuildXml projectXml, FilePath solutionFilePath, String[] additionalFilesToIncludeAtSolutionLevel, Boolean isLoadedFromFile, IDEVersion vsVersion, ComponentUniqueName componentName, TaskSettings componentTaskSettings, Exception parseException, String targetFrameworkForMsbuild, String visibleTargetFramework)
Back to top
Remco
#4 Posted : Saturday, September 24, 2022 11:08:37 AM(UTC)
Rank: NCrunch Developer

Groups: Administrators
Joined: 4/16/2011(UTC)
Posts: 6,974

Thanks: 929 times
Was thanked: 1256 time(s) in 1169 post(s)
yizhou;16280 wrote:
Thanks Remco.
Our build system relies on setting this MSBuildProjectExtensionsPath property.
I've tried suppressing it for NCrunch before, but I got this error that obj\project.assets.json is not found. The generated files are in .pkgref folder as we set in MSBuildProjectExtensionsPath.
Could we let NCrunch run nuget restore for itself?


I was worried about this. This is, unfortunately, one area of NCrunch where we have little to no flexibility. The problem here is rooted in the way VS handles the Nuget restore step. Basically, we can't do the restore ourselves in a stable way without interfering with VS. When MS introduced the automatic package restore step, our initial builds did actually do this, but we had to take it out because MSBuild simply can't handle two processes performing the step concurrently in the same place.

So we're reliant on VS to perform this restore step for us, then we need to be reliant on standard build conventions when finding these files, because MSBuild has its own expectations and constraints when we call into it.

From the side of someone engineering a build system, overriding this particular property (and certain others like it) can seem like a perfectly reasonable thing to do. But from the perspective of NCrunch's build integration, it's basically like pulling out the foundation under a stack of cards. In almost every other area of the build system there are workarounds we can introduce, or things we can override to bend it to our will. But this part is really baked together. The package restore step and location of its derived files might as well be set in stone as far as a maintainable NCrunch is concerned.

This is not to say that is impossible to make such a system work, but rather that it would involve hacking the build system in ways that would introduce unforeseeable side-effects, making our build integration impossible to maintain.

Sorry, you need to avoid changing this property if you want to use NCrunch on this solution.
Back to top
yizhou
#5 Posted : Monday, September 26, 2022 5:39:47 AM(UTC)
Rank: Newbie

Groups: Registered
Joined: 9/23/2022(UTC)
Posts: 3
Thanks, Remco, for the information.
Currently our workaround is to import the nuget props explicitly in csproj for NCrunch.

Code:
<Project Sdk="Microsoft.NET.Sdk">
  <PropertyGroup>
    <TargetFramework>net472</TargetFramework>
  </PropertyGroup>

  <Import Project="$(MSBuildProjectExtensionsPath)OurProjectName.csproj.nuget.g.props" Condition="'$(NCrunch)' == '1'" />

  <ItemGroup>
    <PackageReference Include="Microsoft.WindowsAzure.SDK" Version="2.9.0" ExcludeAssets="all" GeneratePathProperty="true" />
    <Reference Include="Microsoft.WindowsAzure.ServiceRuntime">
      <HintPath>$(PkgMicrosoft_WindowsAzure_SDK)\lib\Microsoft.WindowsAzure.ServiceRuntime.dll</HintPath>
    </Reference>
  </ItemGroup>
</Project>


It works as expected but we're seeking a more generic solution that doesn't need to hard code the project name in each project.
I've tried $(MSBuildProjectFile) but it becomes a temporary file name like nCrunchTemp_c3f9f128-4e6e-4467-a77c-009678083cfa.csproj.
I'm wondering if there's some property that we can use to get the original project name?
Back to top
Remco
#6 Posted : Tuesday, September 27, 2022 12:09:42 AM(UTC)
Rank: NCrunch Developer

Groups: Administrators
Joined: 4/16/2011(UTC)
Posts: 6,974

Thanks: 929 times
Was thanked: 1256 time(s) in 1169 post(s)
yizhou;16282 wrote:
Thanks, Remco, for the information.
Currently our workaround is to import the nuget props explicitly in csproj for NCrunch.

Code:
<Project Sdk="Microsoft.NET.Sdk">
  <PropertyGroup>
    <TargetFramework>net472</TargetFramework>
  </PropertyGroup>

  <Import Project="$(MSBuildProjectExtensionsPath)OurProjectName.csproj.nuget.g.props" Condition="'$(NCrunch)' == '1'" />

  <ItemGroup>
    <PackageReference Include="Microsoft.WindowsAzure.SDK" Version="2.9.0" ExcludeAssets="all" GeneratePathProperty="true" />
    <Reference Include="Microsoft.WindowsAzure.ServiceRuntime">
      <HintPath>$(PkgMicrosoft_WindowsAzure_SDK)\lib\Microsoft.WindowsAzure.ServiceRuntime.dll</HintPath>
    </Reference>
  </ItemGroup>
</Project>



It's great that this works around the issue for you. Nice work!

yizhou;16282 wrote:

It works as expected but we're seeking a more generic solution that doesn't need to hard code the project name in each project.
I've tried $(MSBuildProjectFile) but it becomes a temporary file name like nCrunchTemp_c3f9f128-4e6e-4467-a77c-009678083cfa.csproj.
I'm wondering if there's some property that we can use to get the original project name?


Try using $(ProjectName) or $(ProjectFileName). We force these to the name of the original project file.
Back to top
Users browsing this topic
Guest
Forum Jump  
You cannot post new topics in this forum.
You cannot reply to topics in this forum.
You cannot delete your posts in this forum.
You cannot edit your posts in this forum.
You cannot create polls in this forum.
You cannot vote in polls in this forum.

YAF | YAF © 2003-2011, Yet Another Forum.NET
This page was generated in 0.070 seconds.
Trial NCrunch
Take NCrunch for a spin
Do your fingers a favour and supercharge your testing workflow
Free Download