robotology / icub-contrib-common

Meta-package to configure contrib modules and libraries
2 stars 2 forks source link

Add uninstall target #5

Open traversaro opened 8 years ago

traversaro commented 8 years ago

I am working on a system in which icub-contrib-common has been installed to /usr/local, while it should have been installed to another directory (see http://wiki.icub.org/wiki/Compilation_on_the_pc104). Not having the uninstall target complicates the cleaning process.

traversaro commented 8 years ago

For people with the same problem:

sudo rm -r /usr/local/lib/ICUBcontrib/
sudo rm -r /usr/local/share/ICUBcontrib/

cleanup my system.

Note that "ideally" the unistall of icub-contrib would need also to remove any file installed by any third party project using icub-contrib.

pattacini commented 8 years ago

If I'm not mistaken, make uninstall purges the content of the installed directories, leaving the directories themselves untouched. Here, instead, we should remove the directories straight away, given that we do not know in advance their contents (a.k.a created by third party projects).

Does this mean we would need a non-standard uninstall target?

traversaro commented 7 years ago

@pattacini good point. However, I think we should only remove the contents installed by icub-contrib-common, otherwise if the CMAKE_INSTALL_PREFIX of icub-contrib-common is set to something like /usr/local the effect of a "delete everything" would be terrible.

claudiofantacci commented 6 years ago

Why dont we use YCM's AddUninstallTarget.cmake?