xartigas pushed a commit to branch master.

http://git.enlightenment.org/core/efl.git/commit/?id=36c70ee93295d4b5040c0f8a20a2fdb70a0cd646

commit 36c70ee93295d4b5040c0f8a20a2fdb70a0cd646
Author: Xavi Artigas <[email protected]>
Date:   Mon Nov 18 15:20:38 2019 +0100

    docs: Copy all images to output folder
    
    Summary:
    Doxygen only copies to the output folder ("html") images included through
    the \image tag. However, we have several images included using \htmlonly
    blocks with <img> tags inside, which also need copying.
    Old makefiles included code to manually copy all pngs, and this patch does
    the same for meson.
    I apologize for needing an external script just to run "cp -rf", it looks
    like meson does not allow wildcards passed to "cp".
    Files not in the output folder won't be included in the tarball and won't
    be distributed.
    
    Test Plan:
    Generated documentation now includes images for pages like:
    `group__Eina__List__Group.html#details`
    
    Reviewers: bu5hm4n, lauromoura
    
    Reviewed By: bu5hm4n
    
    Subscribers: cedric, #reviewers, #committers, myoungwoon
    
    Tags: #efl
    
    Differential Revision: https://phab.enlightenment.org/D10690
---
 doc/efl_copy.sh |  1 +
 doc/meson.build | 10 +++++++++-
 2 files changed, 10 insertions(+), 1 deletion(-)

diff --git a/doc/efl_copy.sh b/doc/efl_copy.sh
new file mode 100755
index 0000000000..af975fbd8f
--- /dev/null
+++ b/doc/efl_copy.sh
@@ -0,0 +1 @@
+cp $@
diff --git a/doc/meson.build b/doc/meson.build
index 81c0e34910..18b9aeecf4 100644
--- a/doc/meson.build
+++ b/doc/meson.build
@@ -104,6 +104,7 @@ widget_preview_eps = 
custom_target('widget_preview_prefs_epc',
 
 shot_sh = find_program('shot.sh')
 tar = find_program('tar')
+efl_copy = find_program('efl_copy.sh')
 
 foreach text_filter_property : text_filter_properties
   text = text_filter_property[0]
@@ -181,10 +182,17 @@ doc_target += custom_target('doxygen',
   build_by_default: false
 )
 
+# This is not pretty but meson does not seem to allow wildcards in plain cp 
commands
+copy_images = custom_target('documentation images',
+  command: [efl_copy, '-rf', join_paths(meson.current_source_dir(), 'img', 
'*.png'), 'html'],
+  output: ['empty_img_copy'],
+  build_by_default: false
+)
+
 compress_target = custom_target('package_doc_tar',
   command: [tar, '-C', meson.build_root(), '--xz',  '-cf', 
'efl-'+meson.project_version()+'-doc.tar.xz', 'html', 'man'],
   output: 'efl-'+meson.project_version()+'-doc.tar.xz',
-  depends: doc_target,
+  depends: [doc_target, copy_images],
   build_by_default: false
 )
 

-- 


Reply via email to