EricVoll / ros-sharp

ROS# is a set of open source software libraries and tools in C# for communicating with ROS from .NET applications, in particular Unity3D
Apache License 2.0
36 stars 7 forks source link

Instructions of latest version #8

Closed dnlwbr closed 4 years ago

dnlwbr commented 4 years ago

Dear @EricVoll

first of all, thank you very much for doing all the work, bringing the code up to date. When I wanted to integrate your new version into my project I noticed a few little things in the instructions (Step 2, Option 2):

Even if the first two points are insignificant and result from the context, the last point is irritating and leaves one in doubt whether one is doing everything correctly. Especially because afterwards you emphasize again that "it is important to keep the NewtonSoft.dll in the RosSharp/Plugins folder".

Best, Daniel

EricVoll commented 4 years ago

Hi @dnlwbr

You are totally right! I'll fix that asap. I haven't updated the readme with the newest version of the library... The demo-project is configured correctly though an should work out of the box (at least on a freshly cloned version and clean build it does in my case)

Best Eric

EricVoll commented 4 years ago

Hi @dnlwbr I updated the ReadMe and fixed the issues you mentioned. Could you check if the instructions are now more clear to you? Best, Eric

dnlwbr commented 4 years ago

Hi @EricVoll thanks for updating the documentation. Unfortunately the mentioned file "External/NewtonSoft.Json.dll" still does not exist. Neither in the RosSharp folder nor in the Demo Project. I think you have deleted that file in 26f7c528358eb957c0589cf3aefdd8a2fe5e1408 Apart from that, I think the rest is clearer now. Best Daniel

EricVoll commented 4 years ago

Hi, oh true - I forgot that this one was deleted. In an earlier version it had to be there, but with the newer version this is not the case anymore. I fixed it. Feel free to close the issue if your problems are solved Thanks, Eric

dnlwbr commented 4 years ago

Okay, I get it. Thank you very much.