granite can't be uninstalled

[Cris Dywan]

So I installed granite. Then I built a new version. So I went to uninstall the previous one like this:

$ sudo make uninstall
make: *** No rule to make target `uninstall'. Stop.

[ammonkey (am-monkeyd)]

make uninstall is not in the spirit of cmake .(http://www.vtk.org/Wiki/CMake_FAQ#Can_I_do_.22make_uninstall.22_with_CMake.3F)
The list of installed files is stored in install_manifest.txt in the build directory.

As the faq point out to remove the installed files you could use the following command:
xargs rm < install_manifest.txt

[Cris Dywan (kalikiana)]

I ended up removing files by hand, which is hardly safer than a supported command. And passing rm onto a random list of files is not better either.

I propose 'make install' bares exactly the same danger and should therefore also be omitted.

[Avi Romanoff (aroman)]

Yeah, I agree with Christian that there's no reason not to have an uninstall
function.

At the *very* least there should be a note about it in the README, INSTALL,
or (possibly) an UNINSTALL file.

On Sun, Aug 14, 2011 at 9:56 PM, Christian Dywan <email address hidden>wrote:

> I ended up removing files by hand, which is hardly safer than a
> supported command. And passing rm onto a random list of files is not
> better either.
>
> I propose 'make install' bares exactly the same danger and should
> therefore also be omitted.
>
> --
> You received this bug notification because you are a member of
> elementary desktop team, which is the registrant for Granite.
> https://bugs.launchpad.net/bugs/826462
>
> Title:
> granite can't be uninstalled
>
> Status in Granite Framework:
> New
>
> Bug description:
> So I installed granite. Then I built a new version. So I went to
> uninstall the previous one like this:
>
> $ sudo make uninstall
> make: *** No rule to make target `uninstall'. Stop.
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/granite/+bug/826462/+subscriptions
>
> --
> Mailing list: https://launchpad.net/~elementaryart
> Post to : <email address hidden>
> Unsubscribe : https://launchpad.net/~elementaryart
> More help : https://help.launchpad.net/ListHelp
>

[Jaap Broekhuizen (jaapz-b)]

I think we can just add the uninstall target, because i personally dont find the "We do not want "make uninstall" to remove useful files from the system." reasoning very convincing. It's pretty logical that people sometimes want to uninstall stuff, and the most logical way is using the make uninstall target. So i vote we include the code given at the FAQ.

[xapantu (xapantu)]

@Jaap: as said above, it doesn't come with cmake, so, we forget it, we can easily see the installed fiels on the system during the installation process (there is a clear list). ammonkey wrote a command above, it should be sufficient.