Search Unity

Burst Conflict error after update 1.0.0 preview1 to preview2

Discussion in 'Burst' started by echeg, Mar 13, 2019.

  1. echeg

    echeg

    Joined:
    Aug 1, 2012
    Posts:
    90
    Hello I update Burst from version 1.0.0 preview1 to preview2
    And I get two conflict errors with Newtonsoft

    First
    Code (CSharp):
    1. Library\PackageCache\com.unity.burst@1.0.0-preview.2\Runtime\BurstCompilerOptions.cs(184,16): error CS0433: The type 'PropertyChangingEventHandler' exists in both 'Newtonsoft.Json, Version=8.0.0.0, Culture=neutral, PublicKeyToken=null' and 'netstandard, Version=2.0.0.0, Culture=neutral, PublicKeyToken=cc7b13ffcd2ddd51'
    Second
    Code (CSharp):
    1. Library\PackageCache\com.unity.burst@1.0.0-preview.2\Runtime\BurstCompilerOptions.cs(309,38): error CS0433: The type 'PropertyChangingEventArgs' exists in both 'Newtonsoft.Json, Version=8.0.0.0, Culture=neutral, PublicKeyToken=null' and 'netstandard, Version=2.0.0.0, Culture=neutral, PublicKeyToken=cc7b13ffcd2ddd51'
     
  2. eizenhorn

    eizenhorn

    Joined:
    Oct 17, 2016
    Posts:
    2,684
  3. Carpet_Head

    Carpet_Head

    Joined:
    Nov 27, 2014
    Posts:
    258
    Also getting this issue, I guess it only happens if you use newtonsoft.json?
     
  4. Jinnan87

    Jinnan87

    Joined:
    Jul 21, 2014
    Posts:
    1
    Upgrade 'Newtonsoft.Json' to 12.0.1
     
  5. Carpet_Head

    Carpet_Head

    Joined:
    Nov 27, 2014
    Posts:
    258
  6. xoofx

    xoofx

    Unity Technologies

    Joined:
    Nov 5, 2016
    Posts:
    417
    Where does this Newtonsoft.Json version comes from? I'm unable to find in the official version this type, so I suspect that the version you are using has been tampered?

    We will remove the usage of PropertyChangingEventHandler, as it is used internally and is not critical.
     
    FROS7 and eizenhorn like this.
  7. xoofx

    xoofx

    Unity Technologies

    Joined:
    Nov 5, 2016
    Posts:
    417
    This should be fixed in the upcoming burst 1.0.0-preview.5 that will be released early next week
     
    echeg and FROS7 like this.
  8. Carpet_Head

    Carpet_Head

    Joined:
    Nov 27, 2014
    Posts:
    258
    It is in the package I linked
    https://assetstore.unity.com/packages/tools/input-management/json-net-for-unity-11347

    It is fairly popular

    Is there any reason you would recommend against using it?
     
    echeg likes this.
  9. echeg

    echeg

    Joined:
    Aug 1, 2012
    Posts:
    90
  10. xoofx

    xoofx

    Unity Technologies

    Joined:
    Nov 5, 2016
    Posts:
    417
    > Is there any reason you would recommend against using it?

    I'm not against it, but they are doing something quite wrong (adding PropertyChangingEventHandler for example is really bad) and it is not the official Newtonsoft.Json, so it can be very confusing.

    Anyway, we are pushing a new version 1.0.0-preview.5 of burst that should be available later today/tomorrow.