D22490: Copy the packages into the build directory so they can be found without installign

Vlad Zagorodniy <[email protected]>
Newsgroups gmane.comp.kde.devel.kwin
Message-ID <[email protected]>
zzag added inline comments.

INLINE COMMENTS

> CMakeLists.txt:134
>  # scripted effects
> -kpackage_install_package(dialogparent/package kwin4_effect_dialogparent effects kwin)
> -kpackage_install_package(dimscreen/package kwin4_effect_dimscreen effects kwin)
> -kpackage_install_package(eyeonscreen/package kwin4_effect_eyeonscreen effects kwin)
> -kpackage_install_package(fade/package kwin4_effect_fade effects kwin)
> -kpackage_install_package(fadedesktop/package kwin4_effect_fadedesktop effects kwin)
> -kpackage_install_package(fadingpopups/package kwin4_effect_fadingpopups effects kwin)
> -kpackage_install_package(frozenapp/package kwin4_effect_frozenapp effects kwin)
> -kpackage_install_package(login/package kwin4_effect_login effects kwin)
> -kpackage_install_package(logout/package kwin4_effect_logout effects kwin)
> -kpackage_install_package(maximize/package kwin4_effect_maximize effects kwin)
> -kpackage_install_package(morphingpopups/package kwin4_effect_morphingpopups effects kwin)
> -kpackage_install_package(scale/package kwin4_effect_scale effects kwin)
> -kpackage_install_package(squash/package kwin4_effect_squash effects kwin)
> -kpackage_install_package(translucency/package kwin4_effect_translucency effects kwin)
> -kpackage_install_package(windowaperture/package kwin4_effect_windowaperture effects kwin)
> +function(install_effect name)
> +    kpackage_install_package(${name}/package kwin4_effect_${name} effects kwin)

install_effect is a pretty generic name, it can be applied to builtin effects too.

add/install_scripted_effect is a better name.

REPOSITORY
  R108 KWin

REVISION DETAIL
  https://phabricator.kde.org/D22490

To: apol, #kwin
Cc: zzag, kwin, LeGast00n, fmonteiro, sbergeron, jraleigh, fbampaloukas, GB_2, mkulinski, ragreen, jackyalcine, Pitel, iodelay, crozbo, bwowk, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, mart
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.