Browse Source

Merge topic 'patch-GenerateExportHeader'

319dd29ef0 GenerateExportHeader: Use lowercase command names

Acked-by: Kitware Robot <[email protected]>
Acked-by: buildbot <[email protected]>
Merge-request: !10176
Brad King 9 months ago
parent
commit
7ea82512ca
2 changed files with 25 additions and 19 deletions
  1. 1 1
      Help/release/3.11.rst
  2. 24 18
      Modules/GenerateExportHeader.cmake

+ 1 - 1
Help/release/3.11.rst

@@ -161,7 +161,7 @@ Modules
 
 
 * A :module:`FindIconv` module was added to locate iconv support.
 * A :module:`FindIconv` module was added to locate iconv support.
 
 
-* The :module:`GenerateExportHeader` module ``GENERATE_EXPORT_HEADER`` command
+* The :module:`GenerateExportHeader` module ``generate_export_header`` command
   gained an ``INCLUDE_GUARD_NAME`` option to change the name of the include
   gained an ``INCLUDE_GUARD_NAME`` option to change the name of the include
   guard symbol written to the generated export header.
   guard symbol written to the generated export header.
   Additionally, it now adds a comment after the closing ``#endif`` on the
   Additionally, it now adds a comment after the closing ``#endif`` on the

+ 24 - 18
Modules/GenerateExportHeader.cmake

@@ -5,18 +5,21 @@
 GenerateExportHeader
 GenerateExportHeader
 --------------------
 --------------------
 
 
-Function for generation of export macros for libraries
-
-This module provides the function ``GENERATE_EXPORT_HEADER()``.
+This module provides the ``generate_export_header()`` function to generate
+export macros for libraries.
 
 
 .. versionadded:: 3.12
 .. versionadded:: 3.12
   Added support for C projects.  Previous versions supported C++ project only.
   Added support for C projects.  Previous versions supported C++ project only.
 
 
-The ``GENERATE_EXPORT_HEADER`` function can be used to generate a file
-suitable for preprocessor inclusion which contains EXPORT macros to be
-used in library classes::
+.. command:: generate_export_header
+
+  The ``generate_export_header()`` function can be used to generate a file
+  suitable for preprocessor inclusion which contains EXPORT macros to be
+  used in library classes:
+
+  .. code-block:: cmake
 
 
-   GENERATE_EXPORT_HEADER( LIBRARY_TARGET
+    generate_export_header(LIBRARY_TARGET
              [BASE_NAME <base_name>]
              [BASE_NAME <base_name>]
              [EXPORT_MACRO_NAME <export_macro_name>]
              [EXPORT_MACRO_NAME <export_macro_name>]
              [EXPORT_FILE_NAME <export_file_name>]
              [EXPORT_FILE_NAME <export_file_name>]
@@ -28,7 +31,7 @@ used in library classes::
              [DEFINE_NO_DEPRECATED]
              [DEFINE_NO_DEPRECATED]
              [PREFIX_NAME <prefix_name>]
              [PREFIX_NAME <prefix_name>]
              [CUSTOM_CONTENT_FROM_VARIABLE <variable>]
              [CUSTOM_CONTENT_FROM_VARIABLE <variable>]
-   )
+    )
 
 
 The target properties :prop_tgt:`CXX_VISIBILITY_PRESET <<LANG>_VISIBILITY_PRESET>`
 The target properties :prop_tgt:`CXX_VISIBILITY_PRESET <<LANG>_VISIBILITY_PRESET>`
 and :prop_tgt:`VISIBILITY_INLINES_HIDDEN` can be used to add the appropriate
 and :prop_tgt:`VISIBILITY_INLINES_HIDDEN` can be used to add the appropriate
@@ -37,7 +40,7 @@ and the convenience variables
 :variable:`CMAKE_CXX_VISIBILITY_PRESET <CMAKE_<LANG>_VISIBILITY_PRESET>` and
 :variable:`CMAKE_CXX_VISIBILITY_PRESET <CMAKE_<LANG>_VISIBILITY_PRESET>` and
 :variable:`CMAKE_VISIBILITY_INLINES_HIDDEN`.
 :variable:`CMAKE_VISIBILITY_INLINES_HIDDEN`.
 
 
-By default ``GENERATE_EXPORT_HEADER()`` generates macro names in a file
+By default ``generate_export_header()`` generates macro names in a file
 name determined by the name of the library.  This means that in the
 name determined by the name of the library.  This means that in the
 simplest case, users of ``GenerateExportHeader`` will be equivalent to:
 simplest case, users of ``GenerateExportHeader`` will be equivalent to:
 
 
@@ -181,19 +184,22 @@ Generates the macros ``VTK_SOMELIB_EXPORT`` etc.
 .. versionadded:: 3.11
 .. versionadded:: 3.11
   Added the ``INCLUDE_GUARD_NAME`` option.
   Added the ``INCLUDE_GUARD_NAME`` option.
 
 
-::
+.. command:: add_compiler_export_flags
+
+  .. code-block:: cmake
+
+    add_compiler_export_flags([<output_variable>])
 
 
-   ADD_COMPILER_EXPORT_FLAGS( [<output_variable>] )
+  .. deprecated:: 3.0
 
 
-.. deprecated:: 3.0
-  Set the target properties
-  :prop_tgt:`CXX_VISIBILITY_PRESET <<LANG>_VISIBILITY_PRESET>` and
-  :prop_tgt:`VISIBILITY_INLINES_HIDDEN` instead.
+    Set the target properties
+    :prop_tgt:`CXX_VISIBILITY_PRESET <<LANG>_VISIBILITY_PRESET>` and
+    :prop_tgt:`VISIBILITY_INLINES_HIDDEN` instead.
 
 
-The ``ADD_COMPILER_EXPORT_FLAGS`` function adds ``-fvisibility=hidden`` to
+The ``add_compiler_export_flags()`` function adds ``-fvisibility=hidden`` to
 :variable:`CMAKE_CXX_FLAGS <CMAKE_<LANG>_FLAGS>` if supported, and is a no-op
 :variable:`CMAKE_CXX_FLAGS <CMAKE_<LANG>_FLAGS>` if supported, and is a no-op
 on Windows which does not need extra compiler flags for exporting support.
 on Windows which does not need extra compiler flags for exporting support.
-You may optionally pass a single argument to ``ADD_COMPILER_EXPORT_FLAGS``
+You may optionally pass a single argument to ``add_compiler_export_flags()``
 that will be populated with the ``CXX_FLAGS`` required to enable visibility
 that will be populated with the ``CXX_FLAGS`` required to enable visibility
 support for the compiler/architecture in use.
 support for the compiler/architecture in use.
 #]=======================================================================]
 #]=======================================================================]
@@ -343,7 +349,7 @@ macro(_DO_GENERATE_EXPORT_HEADER TARGET_LIBRARY)
     "${_GEH_PREFIX_NAME}${BASE_NAME_UPPER}_NO_DEPRECATED")
     "${_GEH_PREFIX_NAME}${BASE_NAME_UPPER}_NO_DEPRECATED")
 
 
   if(_GEH_UNPARSED_ARGUMENTS)
   if(_GEH_UNPARSED_ARGUMENTS)
-    message(FATAL_ERROR "Unknown keywords given to GENERATE_EXPORT_HEADER(): \"${_GEH_UNPARSED_ARGUMENTS}\"")
+    message(FATAL_ERROR "Unknown keywords given to generate_export_header(): \"${_GEH_UNPARSED_ARGUMENTS}\"")
   endif()
   endif()
 
 
   if(_GEH_EXPORT_MACRO_NAME)
   if(_GEH_EXPORT_MACRO_NAME)