[llvm] r283028 - [OCaml] Install .mli (interface) files
Michal Gorny via llvm-commits
llvm-commits at lists.llvm.org
Sat Oct 1 02:26:23 PDT 2016
Author: mgorny
Date: Sat Oct 1 04:26:23 2016
New Revision: 283028
URL: http://llvm.org/viewvc/llvm-project?rev=283028&view=rev
Log:
[OCaml] Install .mli (interface) files
Install the OCaml interface .mli files. Those files were most likely
omitted because they are input files for the compiled .cmi files.
However, installing them is reasonable since -- unlike .cmi files --
they are human-readable.
The issue was originally spotted by @jpdeplaix.
Differential Revision: https://reviews.llvm.org/D25128
Modified:
llvm/trunk/cmake/modules/AddOCaml.cmake
Modified: llvm/trunk/cmake/modules/AddOCaml.cmake
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/cmake/modules/AddOCaml.cmake?rev=283028&r1=283027&r2=283028&view=diff
==============================================================================
--- llvm/trunk/cmake/modules/AddOCaml.cmake (original)
+++ llvm/trunk/cmake/modules/AddOCaml.cmake Sat Oct 1 04:26:23 2016
@@ -180,10 +180,11 @@ function(add_ocaml_library name)
set(install_files)
set(install_shlibs)
- foreach( ocaml_output ${ocaml_outputs} )
+ foreach( ocaml_output ${ocaml_inputs} ${ocaml_outputs} )
get_filename_component(ext "${ocaml_output}" EXT)
if( NOT (ext STREQUAL ".cmo" OR
+ ext STREQUAL ".ml" OR
ext STREQUAL CMAKE_C_OUTPUT_EXTENSION OR
ext STREQUAL CMAKE_SHARED_LIBRARY_SUFFIX) )
list(APPEND install_files "${ocaml_output}")
More information about the llvm-commits
mailing list