diff options
| author | bfredl <bjorn.linse@gmail.com> | 2023-03-07 15:00:51 +0100 |
|---|---|---|
| committer | bfredl <bjorn.linse@gmail.com> | 2023-03-07 20:16:52 +0100 |
| commit | 30632dd21ab243bb2c4eb115819a32fdfd9155dc (patch) | |
| tree | ebe91e82ac53a5b36a5512432b98fb2b5f510484 /cmake | |
| parent | 706bcab75eaad2c370d61bf828531054439d3a3e (diff) | |
| download | rneovim-30632dd21ab243bb2c4eb115819a32fdfd9155dc.tar.gz rneovim-30632dd21ab243bb2c4eb115819a32fdfd9155dc.tar.bz2 rneovim-30632dd21ab243bb2c4eb115819a32fdfd9155dc.zip | |
refactor(build): make installation of runtime/ more effective
Currently files to install in runtime/ is detected by recursive glob
pattern which has two problems:
- cmake needs to do a of work at config time and
build/runtime/cmake_install.cmake becomes HUGE (2.5MB, biggest config file)
- we need to explicitly specify each file suffix used in the entire
runtime, which is duplication of information.
These globs specify every single file in a subdirectory.
Thus, we can just install every runtime/ subdirectory as a single
install command. Furthermore, at the top-level, only .vim and .lua files
need to be installed.
Further possible refactor: we could move files which does not belong
in $PREFIX/runtime out of $REPO/runtime. Then runtime could be installed
with a single install_helper(DIRECTORY ...) command.
Diffstat (limited to 'cmake')
| -rw-r--r-- | cmake/InstallHelpers.cmake | 9 |
1 files changed, 0 insertions, 9 deletions
diff --git a/cmake/InstallHelpers.cmake b/cmake/InstallHelpers.cmake index 3786c4177f..49d8692aae 100644 --- a/cmake/InstallHelpers.cmake +++ b/cmake/InstallHelpers.cmake @@ -167,12 +167,3 @@ function(glob_wrapper outvar) endif() set(${outvar} ${${outvar}} PARENT_SCOPE) endfunction() - -function(globrecurse_wrapper outvar root) - if(${CMAKE_VERSION} VERSION_LESS 3.12) - file(GLOB_RECURSE ${outvar} RELATIVE ${root} ${ARGN}) - else() - file(GLOB_RECURSE ${outvar} CONFIGURE_DEPENDS RELATIVE ${root} ${ARGN}) - endif() - set(${outvar} ${${outvar}} PARENT_SCOPE) -endfunction() |