[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