build - remove evas-modules option as its rather pointless
authorCarsten Haitzler (Rasterman) <raster@rasterman.com>
Fri, 4 Oct 2019 12:16:02 +0000 (13:16 +0100)
committerJongmin Lee <jm105.lee@samsung.com>
Sun, 6 Oct 2019 21:22:22 +0000 (06:22 +0900)
evas-modules affects evas engine modules if they are static or shared,
but ecore evas modules are still modules... so all in all this doesnt
help much as it still requires modules to be separate from the shared
libs, thus disallowing for statically linking efl into an app anyway
etc. etc. etc. ... so less options to deal with, less complexity.
better.

16 files changed:
.ci/ci-configure.sh
meson.build
meson_options.txt
src/modules/evas/engines/buffer/meson.build
src/modules/evas/engines/drm/meson.build
src/modules/evas/engines/fb/meson.build
src/modules/evas/engines/gl_cocoa/meson.build
src/modules/evas/engines/gl_drm/meson.build
src/modules/evas/engines/gl_generic/meson.build
src/modules/evas/engines/gl_x11/meson.build
src/modules/evas/engines/meson.build
src/modules/evas/engines/software_ddraw/meson.build
src/modules/evas/engines/software_gdi/meson.build
src/modules/evas/engines/software_x11/meson.build
src/modules/evas/engines/wayland_egl/meson.build
src/modules/evas/engines/wayland_shm/meson.build

index 54ed89d..97fcb17 100755 (executable)
@@ -25,7 +25,7 @@ if [ "$DISTRO" != "" ] ; then
   DISABLED_LINUX_COPTS=" -Daudio=false -Davahi=false -Dx11=false -Dphysics=false -Deeze=false \
   -Dopengl=none -Deina-magic-debug=false -Dbuild-examples=false -Dbuild-tests=false \
   -Dcrypto=gnutls -Dglib=false -Dgstreamer=false -Dsystemd=false -Dpulseaudio=false \
-  -Dnetwork-backend=connman -Dxinput2=false -Dtslib=false -Devas-modules=static \
+  -Dnetwork-backend=connman -Dxinput2=false -Dtslib=false \
   -Devas-loaders-disabler=gst,pdf,ps,raw,svg,xcf,bmp,dds,eet,generic,gif,ico,jp2k,json,pmaps,psd,tga,tgv,tiff,wbmp,webp,xpm \
   -Decore-imf-loaders-disabler=xim,ibus,scim  -Demotion-loaders-disabler=gstreamer,gstreamer1,libvlc,xine \
   -Demotion-generic-loaders-disabler=vlc -Dfribidi=false -Dfontconfig=false \
@@ -37,7 +37,7 @@ if [ "$DISTRO" != "" ] ; then
 
   MINGW_COPTS="--cross-file .ci/cross_toolchain.txt -Davahi=false -Deeze=false -Dsystemd=false \
   -Dpulseaudio=false -Dx11=false -Dopengl=none -Dlibmount=false \
-  -Devas-loaders-disabler=json,pdf,ps,raw,svg,rsvg -Devas-modules=static -Dbindings=luajit \
+  -Devas-loaders-disabler=json,pdf,ps,raw,svg,rsvg -Dbindings=luajit \
   -Dharfbuzz=true -Dpixman=true -Dembedded-lz4=false "
 
   if [ "$1" = "options-enabled" ]; then
index 27827c8..581d32b 100644 (file)
@@ -252,21 +252,7 @@ evas_img_modules = ['evas'                       ,[],  true, false, false, false
 evas_engines =     [join_paths('evas', 'engines'),[],  true, false, false, false, false, false, [], []]
 evas_goal =        ['evas_goal'                  ,[], false,  true, false, false, false, false, ['eina', 'efl', 'eo'], []]
 
-if get_option('evas-modules') == 'static'
-  evas_settings = [
-    evas_lib,
-    evas_img_modules,
-    evas_engines,
-    evas_goal,
-  ]
-else
-  evas_settings = [
-    evas_lib,
-    evas_img_modules,
-    evas_goal,
-    evas_engines,
-  ]
-endif
+evas_settings = [ evas_lib, evas_img_modules, evas_goal, evas_engines ]
 
 subprojects = [
 # name              |   option              | mod  | lib  | bin  | bench | tests | examples | pkg-config options | name of static libs
index 44940e8..a57110a 100644 (file)
@@ -192,13 +192,6 @@ option('unmount-path',
   description : 'Path to the unmount binary, if "detect" internal eeze bins are used'
 )
 
-option('evas-modules',
-  type : 'combo',
-  choices : ['shared', 'static'],
-  value : 'shared',
-  description : 'Should modules be shared or statically inlined'
-)
-
 option('evas-loaders-disabler',
   type : 'array',
   description : 'List of modular image/vector loaders to disable in efl',
index ef42fd4..8c6a071 100644 (file)
@@ -13,15 +13,11 @@ install_headers('Evas_Engine_Buffer.h',
   install_dir : join_paths(dir_include, 'evas-'+version_major),
 )
 
-if get_option('evas-modules') == 'shared'
-  evas_engine_buffer_mod = shared_module(mod_full_name, engine_src,
-        include_directories : config_dir + [engine_include_dir],
-        dependencies : [eina, evas] + engine_deps,
-        install : true,
-        install_dir : mod_install_dir,
-        name_suffix : sys_mod_extension
-      )
-      module_files += join_paths(mod_install_dir, 'lib'+mod_full_name + '.' + sys_mod_extension)
-else
-  evas_engine_buffer_mod = []
-endif
+evas_engine_buffer_mod = shared_module(mod_full_name, engine_src,
+  include_directories : config_dir + [engine_include_dir],
+  dependencies : [eina, evas] + engine_deps,
+  install : true,
+  install_dir : mod_install_dir,
+  name_suffix : sys_mod_extension
+)
+module_files += join_paths(mod_install_dir, 'lib'+mod_full_name + '.' + sys_mod_extension)
index 441a09a..fb97b63 100644 (file)
@@ -7,13 +7,11 @@ engine_src = files([
 
 engine_deps = [ecore_drm2, libdrm]
 
-if get_option('evas-modules') == 'shared'
-  shared_module(mod_full_name, engine_src,
-        include_directories : config_dir + [engine_include_dir],
-        dependencies : [eina, evas] + engine_deps,
-        install : true,
-        install_dir : mod_install_dir,
-        name_suffix : sys_mod_extension
-      )
-      module_files += join_paths(mod_install_dir, 'lib'+mod_full_name + '.' + sys_mod_extension)
-endif
+shared_module(mod_full_name, engine_src,
+  include_directories : config_dir + [engine_include_dir],
+  dependencies : [eina, evas] + engine_deps,
+  install : true,
+  install_dir : mod_install_dir,
+  name_suffix : sys_mod_extension
+)
+module_files += join_paths(mod_install_dir, 'lib'+mod_full_name + '.' + sys_mod_extension)
index c857e83..625edd5 100644 (file)
@@ -9,13 +9,11 @@ engine_src = files([
 
 engine_deps = [ecore_fb]
 
-if get_option('evas-modules') == 'shared'
-  shared_module(mod_full_name, engine_src,
-        include_directories : config_dir + [engine_include_dir],
-        dependencies : [eina, evas] + engine_deps,
-        install : true,
-        install_dir : mod_install_dir,
-        name_suffix : sys_mod_extension
-      )
-      module_files += join_paths(mod_install_dir, 'lib'+mod_full_name + '.' + sys_mod_extension)
-endif
+shared_module(mod_full_name, engine_src,
+  include_directories : config_dir + [engine_include_dir],
+  dependencies : [eina, evas] + engine_deps,
+  install : true,
+  install_dir : mod_install_dir,
+  name_suffix : sys_mod_extension
+)
+module_files += join_paths(mod_install_dir, 'lib'+mod_full_name + '.' + sys_mod_extension)
index a175c3c..a0bd765 100644 (file)
@@ -7,13 +7,11 @@ engine_src = files([
 
 engine_deps = [ecore_cocoa, gl_deps]
 
-if get_option('evas-modules') == 'shared'
-  shared_module(mod_full_name, engine_src,
-        include_directories : config_dir + [engine_include_dir],
-        dependencies : [eina, evas] + engine_deps,
-        install : true,
-        install_dir : mod_install_dir,
-        name_suffix : sys_mod_extension
-      )
-      module_files += join_paths(mod_install_dir, 'lib'+mod_full_name + '.' + sys_mod_extension)
-endif
+shared_module(mod_full_name, engine_src,
+  include_directories : config_dir + [engine_include_dir],
+  dependencies : [eina, evas] + engine_deps,
+  install : true,
+  install_dir : mod_install_dir,
+  name_suffix : sys_mod_extension
+)
+module_files += join_paths(mod_install_dir, 'lib'+mod_full_name + '.' + sys_mod_extension)
index 28988d5..8acf823 100644 (file)
@@ -7,13 +7,11 @@ engine_src = files([
 
 engine_deps = [ecore_drm2, libdrm, gl_deps, dependency('gbm')]
 
-if get_option('evas-modules') == 'shared'
-  shared_module(mod_full_name, engine_src,
-        include_directories : config_dir + [engine_include_dir],
-        dependencies : [eina, evas] + engine_deps,
-        install : true,
-        install_dir : mod_install_dir,
-        name_suffix : sys_mod_extension
-      )
-      module_files += join_paths(mod_install_dir, 'lib'+mod_full_name + '.' + sys_mod_extension)
-endif
+shared_module(mod_full_name, engine_src,
+  include_directories : config_dir + [engine_include_dir],
+  dependencies : [eina, evas] + engine_deps,
+  install : true,
+  install_dir : mod_install_dir,
+  name_suffix : sys_mod_extension
+)
+module_files += join_paths(mod_install_dir, 'lib'+mod_full_name + '.' + sys_mod_extension)
index e705915..b2620c8 100644 (file)
@@ -86,14 +86,12 @@ engine_deps = [gl_deps]
 
 engine_include_dir = include_directories(join_paths('..','software_generic'), join_paths('..', 'gl_common'))
 
-if get_option('evas-modules') == 'shared'
-  shared_module(mod_full_name, engine_src,
-        include_directories : config_dir + [engine_include_dir],
-        dependencies : [eina, evas] + engine_deps,
-        install : true,
-        install_dir : mod_install_dir,
-        c_args: ['-DEVAS_GL_RENDER_THREAD_COMPILE_FOR_GL_GENERIC',],
-        name_suffix : sys_mod_extension
-      )
-      module_files += join_paths(mod_install_dir, 'lib'+mod_full_name + '.' + sys_mod_extension)
-endif
+shared_module(mod_full_name, engine_src,
+  include_directories : config_dir + [engine_include_dir],
+  dependencies : [eina, evas] + engine_deps,
+  install : true,
+  install_dir : mod_install_dir,
+  c_args: ['-DEVAS_GL_RENDER_THREAD_COMPILE_FOR_GL_GENERIC',],
+  name_suffix : sys_mod_extension
+)
+module_files += join_paths(mod_install_dir, 'lib'+mod_full_name + '.' + sys_mod_extension)
index 7aec198..0bac841 100644 (file)
@@ -7,13 +7,11 @@ engine_src = files([
 
 engine_deps = [ecore_x_deps, gl_deps]
 
-if get_option('evas-modules') == 'shared'
-  shared_module(mod_full_name, engine_src,
-        include_directories : config_dir + [engine_include_dir],
-        dependencies : [eina, evas] + engine_deps,
-        install : true,
-        install_dir : mod_install_dir,
-        name_suffix : sys_mod_extension
-      )
-      module_files += join_paths(mod_install_dir, 'lib'+mod_full_name + '.' + sys_mod_extension)
-endif
+shared_module(mod_full_name, engine_src,
+  include_directories : config_dir + [engine_include_dir],
+  dependencies : [eina, evas] + engine_deps,
+  install : true,
+  install_dir : mod_install_dir,
+  name_suffix : sys_mod_extension
+)
+module_files += join_paths(mod_install_dir, 'lib'+mod_full_name + '.' + sys_mod_extension)
index 8dcbaee..7456dc3 100644 (file)
@@ -74,37 +74,6 @@ foreach engine_conf : engines
 
     subdir(engine)
 
-    if get_option('evas-modules') == 'static'
-      if module == 'gl_generic'
-        tmp = static_library(mod_full_name, engine_src,
-          include_directories : config_dir + [engine_include_dir],
-          dependencies : [eina, evas_pre] + engine_deps,
-          c_args: ['-DEVAS_GL_RENDER_THREAD_COMPILE_FOR_GL_GENERIC',]
-        )
-      else
-        tmp = static_library(mod_full_name, engine_src,
-          include_directories : config_dir + [engine_include_dir],
-          dependencies : [eina, evas_pre] + engine_deps,
-        )
-      endif
-
-      evas_static_list += declare_dependency(
-        include_directories: [include_directories('.')] + config_dir + [engine_include_dir],
-        link_with: tmp,
-        dependencies : [eina, evas_pre] + engine_deps,
-        sources : engine_src
-      )
-      if engine == 'gl_generic'
-        #special case, see evas_module.c
-        config_h.set('EVAS_STATIC_BUILD_GL_COMMON', '1')
-      else
-        config_h.set('EVAS_STATIC_BUILD_'+engine.to_upper(), '1')
-      endif
-    else
-      #nothing here shared building is handled directly on the engine configuration side
-      #reason for this is that the .so files have to be placed in the correct directory in order
-      # to make them discoverable by evas module code
-    endif
     config_h.set('BUILD_ENGINE_'+engine.to_upper(), '1')
   endif
 endforeach
index 404fe7c..9b74275 100644 (file)
@@ -7,14 +7,12 @@ engine_src = files([
 
 engine_deps = [cc.find_library('ddraw')]
 
-if get_option('evas-modules') == 'shared'
-  shared_module(mod_full_name, engine_src,
-        include_directories : config_dir + [engine_include_dir],
-        cpp_args: ['-fno-rtti', '-fno-exceptions'],
-        dependencies : [eina, evas] + engine_deps,
-        install : true,
-        install_dir : mod_install_dir,
-        name_suffix : sys_mod_extension
-      )
-      module_files += join_paths(mod_install_dir, 'lib'+mod_full_name + '.' + sys_mod_extension)
-endif
+shared_module(mod_full_name, engine_src,
+  include_directories : config_dir + [engine_include_dir],
+  cpp_args: ['-fno-rtti', '-fno-exceptions'],
+  dependencies : [eina, evas] + engine_deps,
+  install : true,
+  install_dir : mod_install_dir,
+  name_suffix : sys_mod_extension
+)
+module_files += join_paths(mod_install_dir, 'lib'+mod_full_name + '.' + sys_mod_extension)
index e492e00..ef7a0aa 100644 (file)
@@ -7,13 +7,11 @@ engine_src = files([
 
 engine_deps = [cc.find_library('gdi32')]
 
-if get_option('evas-modules') == 'shared'
-  shared_module(mod_full_name, engine_src,
-        include_directories : config_dir + [engine_include_dir],
-        dependencies : [eina, evas] + engine_deps,
-        install : true,
-        install_dir : mod_install_dir,
-        name_suffix : sys_mod_extension
-      )
-      module_files += join_paths(mod_install_dir, 'lib'+mod_full_name + '.' + sys_mod_extension)
-endif
+shared_module(mod_full_name, engine_src,
+  include_directories : config_dir + [engine_include_dir],
+  dependencies : [eina, evas] + engine_deps,
+  install : true,
+  install_dir : mod_install_dir,
+  name_suffix : sys_mod_extension
+)
+module_files += join_paths(mod_install_dir, 'lib'+mod_full_name + '.' + sys_mod_extension)
index b540956..c2f8f54 100644 (file)
@@ -28,13 +28,11 @@ config_h.set('BUILD_ENGINE_SOFTWARE_XLIB', '1')
 #it looks weird but is right, gl_deps is needed for evas_x_egl.c
 engine_deps = [x11, ecore_x, ecore_x_deps, gl_deps]
 
-if get_option('evas-modules') == 'shared'
-  shared_module(mod_full_name, engine_src,
-        include_directories : config_dir + [engine_include_dir],
-        dependencies : [eina, evas] + engine_deps,
-        install : true,
-        install_dir : mod_install_dir,
-        name_suffix : sys_mod_extension
-      )
-      module_files += join_paths(mod_install_dir, 'lib'+mod_full_name + '.' + sys_mod_extension)
-endif
+shared_module(mod_full_name, engine_src,
+  include_directories : config_dir + [engine_include_dir],
+  dependencies : [eina, evas] + engine_deps,
+  install : true,
+  install_dir : mod_install_dir,
+  name_suffix : sys_mod_extension
+)
+module_files += join_paths(mod_install_dir, 'lib'+mod_full_name + '.' + sys_mod_extension)
index 2f3a950..1d68058 100644 (file)
@@ -17,14 +17,11 @@ engine_deps = [
 
 engine_include_dir = include_directories(join_paths('..','wayland_common'))
 
-
-if get_option('evas-modules') == 'shared'
-  shared_module(mod_full_name, engine_src,
-        include_directories : config_dir + [engine_include_dir],
-        dependencies : [eina, evas] + engine_deps,
-        install : true,
-        install_dir : mod_install_dir,
-        name_suffix : sys_mod_extension
-      )
-      module_files += join_paths(mod_install_dir, 'lib'+mod_full_name + '.' + sys_mod_extension)
-endif
+shared_module(mod_full_name, engine_src,
+  include_directories : config_dir + [engine_include_dir],
+  dependencies : [eina, evas] + engine_deps,
+  install : true,
+  install_dir : mod_install_dir,
+  name_suffix : sys_mod_extension
+)
+module_files += join_paths(mod_install_dir, 'lib'+mod_full_name + '.' + sys_mod_extension)
index 2945384..2a181ce 100644 (file)
@@ -8,13 +8,11 @@ engine_deps = [ecore_wl2]
 
 engine_include_dir = include_directories(join_paths('..','wayland_common'))
 
-if get_option('evas-modules') == 'shared'
-  shared_module(mod_full_name, engine_src,
-        include_directories : config_dir + [engine_include_dir],
-        dependencies : [eina, evas] + engine_deps,
-        install : true,
-        install_dir : mod_install_dir,
-        name_suffix : sys_mod_extension
-      )
-      module_files += join_paths(mod_install_dir, 'lib'+mod_full_name + '.' + sys_mod_extension)
-endif
+shared_module(mod_full_name, engine_src,
+  include_directories : config_dir + [engine_include_dir],
+  dependencies : [eina, evas] + engine_deps,
+  install : true,
+  install_dir : mod_install_dir,
+  name_suffix : sys_mod_extension
+)
+module_files += join_paths(mod_install_dir, 'lib'+mod_full_name + '.' + sys_mod_extension)