search

cheshirekow / cmake_format

Source code formatter for cmake listfiles.
GNU General Public License v3.0
954 stars 105 forks source link

install kwarg INCLUDES DESTINATION formatted wrong #245

Open blackliner opened 3 years ago

blackliner commented 3 years ago

https://github.com/cheshirekow/cmake_format/blob/904c42da298f7fda932d9ac774517c20e2667fb8/cmakelang/parse/funs/install.py#L58 seems to not work, this:

install(TARGETS ${target}
        EXPORT MyTargets
        RUNTIME DESTINATION new_install/${CMAKE_INSTALL_BINDIR}
        LIBRARY DESTINATION new_install/${CMAKE_INSTALL_LIBDIR}
        ARCHIVE DESTINATION new_install/${CMAKE_INSTALL_LIBDIR}
        INCLUDES DESTINATION new_install/${CMAKE_INSTALL_INCLUDEDIR}
        PUBLIC_HEADER DESTINATION new_install/${CMAKE_INSTALL_INCLUDEDIR}
)

becomes

install(TARGETS ${target}
        EXPORT MyTargets
        RUNTIME DESTINATION new_install/${CMAKE_INSTALL_BINDIR}
        LIBRARY DESTINATION new_install/${CMAKE_INSTALL_LIBDIR}
        ARCHIVE DESTINATION new_install/${CMAKE_INSTALL_LIBDIR}
        INCLUDES
        DESTINATION new_install/${CMAKE_INSTALL_INCLUDEDIR}
        PUBLIC_HEADER DESTINATION new_install/${CMAKE_INSTALL_INCLUDEDIR}
)