strands-project/strands_3d_mapping

Rename to scitos_3d_mapping

Closed this issue · 11 comments

Is renaming this OK?

Seems good to me!

@marc-hanheide are you pulling this repo down automatically somewhere?

I like it!

Also, delete the line referring to 3d_mapping. Will this automatically delete the package locally when we run wstool merge and wstool update?

@bfalacerda, right! It will do for the jenkins automated testing (as that is a fresh checkout anyway). Not sure if it would in general, but I vaguely remember it worked...

I'll confirm it when everything is updated

Looks like it doesn't delete the repos in .rosinstall. It just adds the new one. What happens now is:

  • 3d_mapping update gives error because it does not exist anymore
  • scitos_3d_mapping_and_navigation works because it is redirected to scitos_3d_mapping
  • scitos_3d_mapping works as expected.

So now we have 2 extra packages locally that are not needed, one that doesn't update and gives an error (3d_mapping) and one that is equal to the new scitos_3d_mapping (scitos_3d_mapping_and_navigation). This can be solved my manually editing the .rosinstall file and deleting the old packages locally, but this needs to be done by every person that already had a previous install. How to warn them all in an easy way? (I guess many won't read this :) )

In these cases I think it might be worth sending emails to strands@cs.bham.ac.uk with a link to this discussion which should include clear instructions to resolve the problem...

This repository was renamed to strands_3d_mapping. We can close this issue.