search

JKISoftware / JKI-JSON-Serialization

JSON Serialization & Deserialization Library for LabVIEW
http://jki.net/tools#json
BSD 3-Clause "New" or "Revised" License
25 stars 10 forks source link

The latest package has dependency to Caraya Unit Test Framework #50

Open Bas-vE opened 2 months ago

Bas-vE commented 2 months ago

version 2.1.0 has a dependency to caraya which can also be seen on VIPM https://www.vipm.io/package/jki_lib_json_serialization/ It feels like this is not by design.

jimkring commented 2 months ago

Oh! I'll take a look. You're right @Bas-vE, it's not by design.

CC: @nate-moehring

nate-moehring commented 2 months ago

When I allow VIPM to scan for dependencies, this is all that comes up: image

@jimkring, I'm guessing you or I mistakenly added Caraya to the package dependencies manually, though I don't know why. You did create that VI Tree in the src folder, which does have a static call to run_tests.vi, but neither is included in the package source files list.

I think we just need to remove it from package dependency list and rebuild it.

jimkring commented 1 month ago

@nate-moehring @Bas-vE I’ll take a look into this one later this weekend and see if I can do a clean rebuild of the package and what the final dependencies look like.

francois-normandin commented 1 month ago

The VI Tree references run_tests.vi, which pulls Caraya as a dependency.