Review Request 127445: "make install" sometimes creates a share directory owned by root in the source tree

[Maurizio Paolini]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/127445/
-----------------------------------------------------------

Review request for Extra Cmake Modules.


Repository: extra-cmake-modules


Description
-------

Avoid creation of a "share" directory owned by root in the source location.

At "make install" stage, project icons are installed in their correct place.  However a new folder is created
(owned by root, typically) in the current directory, typically in the home directory.

This is often inconvenient, since the user is then unable to remove such directory without root priviledges, and
could also lead to an error at "make install" stage.
One example of this is when the home directory of the user is an NFS mount from a remote NFS server, in which case
all NFS requests by "root" are remapped on the server to the "nfsnobody" which would typically prevent root from
creating files in the home directory.

The cause seems to be in file modules/ECMInstallIcons.cmake:

        file(WRITE \"${installdir}/${theme}/temp.txt\" \"update\")
        file(REMOVE \"${installdir}/${theme}/temp.txt\")

because the standard value or ${installdir} is the relative path "share/icons"


Diffs
-----

  modules/ECMInstallIcons.cmake 3923636 

Diff: https://git.reviewboard.kde.org/r/127445/diff/


Testing
-------


Thanks,

Maurizio Paolini

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-buildsystem/attachments/20160321/d487b880/attachment.html>