This is an automated email from Gerrit.

"zapb <d...@zapb.de>" just uploaded a new patch set to Gerrit, which you can 
find at https://review.openocd.org/c/openocd/+/7129

-- gerrit

commit 550e40d7b2150e78fa557d9a706555487f848f18
Author: Marc Schink <d...@zapb.de>
Date:   Sun May 22 16:15:00 2022 +0200

    Deprecate libjaylink Git submodule
    
    libjaylink was integrated into OpenOCD as a Git submodule, because at
    that time there was no official release and no packages for it.
    Today there are libjaylink packages for most popular distributions [1].
    
    Removing libjaylink from OpenOCD reduces build complexity in both
    projects and makes them more flexible with respect to the build system,
    for example.
    
    Disable the libjaylink submodule by default and announce it as
    deprecated feature that will be removed in the future. This gives
    package maintainers time to package libjaylink if not already done.
    
    [1] https://repology.org/project/libjaylink/versions
    
    Change-Id: I6166ba4757aee5c89a0506de867072f58fa5ec4b
    Signed-off-by: Marc Schink <d...@zapb.de>

diff --git a/configure.ac b/configure.ac
index 7d0cc24698..604fa21b1b 100644
--- a/configure.ac
+++ b/configure.ac
@@ -376,9 +376,9 @@ AC_ARG_ENABLE([jimtcl-maintainer],
   [use_internal_jimtcl_maintainer=$enableval], 
[use_internal_jimtcl_maintainer=no])
 
 AC_ARG_ENABLE([internal-libjaylink],
-  AS_HELP_STRING([--disable-internal-libjaylink],
-  [Disable building internal libjaylink]),
-  [use_internal_libjaylink=$enableval], [use_internal_libjaylink=yes])
+  AS_HELP_STRING([--enable-internal-libjaylink],
+  [Enable building internal libjaylink]),
+  [use_internal_libjaylink=$enableval], [use_internal_libjaylink=no])
 
 AC_ARG_ENABLE([remote-bitbang],
   AS_HELP_STRING([--enable-remote-bitbang], [Enable building support for the 
Remote Bitbang jtag driver]),
@@ -713,7 +713,7 @@ AS_IF([test "x$enable_jlink" != "xno"], [
       AX_CONFIG_SUBDIR_OPTION([src/jtag/drivers/libjaylink],
                [--enable-subproject-build])
     ], [
-      AC_MSG_ERROR([Internal libjaylink not found, run either 'git submodule 
init' and 'git submodule update' or disable internal libjaylink with 
--disable-internal-libjaylink.])
+      AC_MSG_ERROR([Internal libjaylink not found, run 'git submodule init' 
and 'git submodule update'.])
     ])
   ])
 ])
@@ -823,6 +823,12 @@ AC_CONFIG_FILES([
 ])
 AC_OUTPUT
 
+AS_IF([test "x$enable_jlink" != "xno"], [
+  AS_IF([test "x$use_internal_libjaylink" = "xyes"], [
+    AC_MSG_WARN([Using the internal libjaylink is deprecated and will not be 
possible in the future.])
+  ]])
+)
+
 echo
 echo
 echo OpenOCD configuration summary

-- 

Reply via email to