Search Unity

Package dependencies from git

Discussion in 'Package Manager' started by Baste, Jul 29, 2019.

  1. Baste

    Baste

    Joined:
    Jan 24, 2013
    Posts:
    6,338
    I've got a custom fork of the graph-visualizer package.

    Adding this custom fork to my project's manifest.json works as expected:

    Code (csharp):
    1. //manifest.json
    2. "dependencies": {
    3.   ...
    4.   "com.unity.playablegraph-visualizer": "https://github.com/Baste-RainGames/graph-visualizer.git",
    5.   ...
    6. }
    But, when I try to add that package as a dependency for a different package:

    Code (csharp):
    1. //package.json
    2. "dependencies": {
    3.   "com.unity.playablegraph-visualizer": "https://github.com/Baste-RainGames/graph-visualizer.git"
    4. },
    That fails:

    upload_2019-7-29_17-36-11.png

    It seems like the dependency resolution doesn't work with git url's? This is using Unity 2018.4.4f1
     
  2. maximeb_unity

    maximeb_unity

    Unity Technologies

    Joined:
    Mar 20, 2018
    Posts:
    556
    Hi @Baste,

    Unfortunately, that's correct. This is not yet supported, though it's on our roadmap. (I cannot provide any ETA.) As a workaround, you can add that dependency directly to the project itself - I believe I've seen an open-source project which implements a workaround for the limitation, somewhere on this forum.
     
  3. Baste

    Baste

    Joined:
    Jan 24, 2013
    Posts:
    6,338
    Is the fix something that will land in the package manager package once it does, or does it require engine changes? We're sticking to 2018.4 LTS until launch, so if the fix is only going to land in the tech stream, I'll not worry about manual dependency resolution and just embed the visualizer directly in our library.
     
  4. maximeb_unity

    maximeb_unity

    Unity Technologies

    Joined:
    Mar 20, 2018
    Posts:
    556
    This is currently not planned for the 2019.X release cycle.
     
  5. Thaina

    Thaina

    Joined:
    Jul 13, 2012
    Posts:
    1,166