Crash while building after update to SmartAssembly 7.4.2.3588

• 
  Addition: This is a .NET Standard 2.0 project; it also fails on other projects of the same type.
  

  clinke May 26, 2020 14:22

  - 0 +
• 
  On my local machine I have the same error. I build the sa project from the SA gui and I send the error report to you.
  

  mrmiagi0101 May 27, 2020 13:48

  - 0 +
• 
  Ok, I found the problem. Somewhere between 7.2.x and 7.4.x Redgate changed the behavior for .NET Standard assemblies. The problem is, if you have the projectname.dep.json file not next to the dll, SmartAssembly is throwing the exception now.
  But this is often the case, because if you build e.g. a WPF exe referencing a .NET Standard project, Visual Studio will never copy the json file from the referenced assembly to output folder of the WPF project. If you now try to build the sa project for the .NET Standard dll located in wpf output folder, you will get the exception. By copying the json manually to this folder, SA is working again.
  redgate: Can build in a fallback if the json is not there to work with old behavior?
  

  mrmiagi0101 May 27, 2020 15:16

  - 0 +
• 
  Hi both!
  
  I unfortunately haven't been able to reproduce this yet, but with regard to your question @mrmiagi0101
  , I know there was some work done as of 7.4.2 so that if the .deps.json not found, SmartAssembly will do further searching to try to locate it. (There is also a feature request SA-2319 which hasn't been implemented yet, but requests that SmartAssembly point out if the file can't be found and prompt you for the correct location- I've +1'd this with your feedback.)
  
  You can though add the following task to the .csproj file which will copy .deps.json file to the output folder as needed:
  
  

  <Target Name="WorkaroundForSmartAssembly_CopyDeps" BeforeTargets="PostBuildEvent">

    <Copy SourceFiles="$(OutDir)$(ProjectName).deps.json" DestinationFolder="$(OutDir)bin" Condition="Exists('$(OutDir)$(ProjectName).deps.json')"/>

  </Target>
  
  

  Then if required, the file can be removed after SmartAssembly build is done:

  <Target Name="WorkaroundForSmartAssembly_RemoveDeps" AfterTargets="BuildWithSmartAssembly">

    <Delete Files="$(OutDir)bin\$(ProjectName).deps.json" Condition="Exists('$(OutDir)bin\$(ProjectName).deps.json')"/>

  </Target>
  
  If you are still running into a different issue though @clinke, is it possible for you to share you assemblies with us to test with locally? You can send it to support@red-gate.com with subject "Files for SmartAssembly issue #162320"

  Jessica R May 28, 2020 19:50

  - 0 +
• 
  "prompt you for the correct location". How should this work without gui? We are building on build server.

  
  

  <Target Name="WorkaroundForSmartAssembly_CopyDeps" BeforeTargets="PostBuildEvent">

    <Copy SourceFiles="$(OutDir)$(ProjectName).deps.json" DestinationFolder="$(OutDir)bin" Condition="Exists('$(OutDir)$(ProjectName).deps.json')"/>

  </Target>

  
  

  Where do we have to place this? In every .net Standard project we are referencing in out executable project? This is not a solution!

  
  

  Why can you not just have a fallback to the old behavior. Or how it was working in Version 7.2.1? With this version everything is fine on build server.
  

  
  

  mrmiagi0101 May 29, 2020 07:24

  - 0 +
• 
  Unfortunately this is no solution for us. Many dependencies of our project are resolved by NuGet-packages. These are not provided with .json-files. So on a multi-assembly solution, providing the .json-file is not an option. As mrmiagi0101 says, we need a fallback.
  

  clinke May 29, 2020 09:10

  - 0 +
• 
  Any news here?
  

  mrmiagi0101 June 24, 2020 19:30

  - 0 +
• 
  Hi all,
  
  So sorry for the delay!
  
  I've escalated the issue but we just want to make sure we are understanding the issue correctly. Would it be possible for either of you to share a reproduction to test with?
  

  Jessica R June 29, 2020 07:33

  - 0 +

Add comment

Please sign in to leave a comment.