On 13/08/15 11:30, Lisandro Damián Nicanor Pérez Meyer wrote:
> On Thursday 13 August 2015 15:21:54 Niels Thykier wrote:
> [snip]
> > I have been in contact with Felix Geyer and Lisandro.  As I understand
> > them, cmake should be handling this part itself and debhelper should not
> > try to change anything.  Notably, it sets LIBDIR to
> > lib/$MULTIARCH_TRIPLET, which should "just work(tm)" assuming upstream
> > uses LIBDIR instead of hardcoding "lib".
>
> Indeed this is true and I want to emphasize Niel's reference:
>
> [snip]
> > References:
> >  *
> > https://anonscm.debian.org/cgit/collab-maint/cmake.git/tree/Modules/GNUInsta
> > llDirs.cmake
>
> Basically the CMakeLists.txt should be using this variables and things should
> just work. Not doing it it's most probably a bug.

Well, yes, but a few minor notes here:

  * the upstream has to use GNUInstallDirs module
  * it will not work for arch-dependent include dir, because it is set
    to "include" always
  * the final variables are called CMAKE_INSTALL_* where asterisk
    is one of many names (LIBDIR, INCLUDEDIR, etc.)

Cheers,
Tomasz

Attachment: signature.asc
Description: Digital signature

Reply via email to