From e05c3507c02f3ced42a88b4019e766ee5cbf2d4c Mon Sep 17 00:00:00 2001 From: Chen Qi Date: Fri, 25 Oct 2024 11:32:27 +0800 Subject: [PATCH] graphviz: remove obsolete and problematic patch Remove 0001-Set-use_tcl-to-be-empty-string-if-tcl-is-disabled.patch. This patch is obsolete and not needed because the current graphviz configure.ac has correct logic of checking use_tcl. This use_tcl variable needs to be set when '--disable-tcl' is set, otherwise, things will behave as if no option is supplied and the configure process will check tcl automatically. This patch is problematic because its logic against the current version is wrong. The recipe has already explicitly set '--disable-tcl', so the configure process should not do automatic checking for tcl at do_configure. This patch fixes do_configure error when host has tcl8.6-dev installed. The error is like below: QA Issue: This autoconf log indicates errors, it looked at host include and/or library paths while determining system capabilities. Rerun configure task after fixing this. [configure-unsafe] Signed-off-by: Chen Qi Signed-off-by: Khem Raj --- ...o-be-empty-string-if-tcl-is-disabled.patch | 42 ------------------- .../graphviz/graphviz_12.1.2.bb | 4 -- 2 files changed, 46 deletions(-) delete mode 100644 meta-oe/recipes-graphics/graphviz/graphviz/0001-Set-use_tcl-to-be-empty-string-if-tcl-is-disabled.patch diff --git a/meta-oe/recipes-graphics/graphviz/graphviz/0001-Set-use_tcl-to-be-empty-string-if-tcl-is-disabled.patch b/meta-oe/recipes-graphics/graphviz/graphviz/0001-Set-use_tcl-to-be-empty-string-if-tcl-is-disabled.patch deleted file mode 100644 index dda1d4350f..0000000000 --- a/meta-oe/recipes-graphics/graphviz/graphviz/0001-Set-use_tcl-to-be-empty-string-if-tcl-is-disabled.patch +++ /dev/null @@ -1,42 +0,0 @@ -From 62193e52edfc71283b7bbcd6f590d63251290b3e Mon Sep 17 00:00:00 2001 -From: Khem Raj -Date: Tue, 6 Aug 2019 17:51:39 -0700 -Subject: [PATCH] Set use_tcl to be empty string if tcl is disabled - -Upstream-Status: Inappropriate [Cross-compile specific] - -Signed-off-by: Khem Raj ---- - Makefile.am | 2 +- - configure.ac | 2 +- - 2 files changed, 2 insertions(+), 2 deletions(-) - -diff --git a/Makefile.am b/Makefile.am -index fe373be..5db5cdc 100644 ---- a/Makefile.am -+++ b/Makefile.am -@@ -11,7 +11,7 @@ pkginclude_HEADERS = $(top_builddir)/graphviz_version.h - dist_man_MANS = graphviz.7 - - # $(subdirs) contains the list from: AC_CONFIG_SUBDIRS --SUBDIRS = $(subdirs) lib plugin cmd tclpkg doc contrib share graphs tests -+SUBDIRS = $(subdirs) lib plugin cmd doc contrib share graphs tests - - .PHONY: doxygen - doxygen: -diff --git a/configure.ac b/configure.ac -index 993a3dd..5e4a1ad 100644 ---- a/configure.ac -+++ b/configure.ac -@@ -1086,7 +1086,7 @@ AC_ARG_ENABLE(tcl, - [], [enable_tcl=yes]) - - if test "$enable_tcl" != "yes"; then -- use_tcl="No (disabled)" -+ use_tcl="" - fi - - if test "$use_tcl" = ""; then --- -2.25.1 - diff --git a/meta-oe/recipes-graphics/graphviz/graphviz_12.1.2.bb b/meta-oe/recipes-graphics/graphviz/graphviz_12.1.2.bb index 1843091aaf..50243da344 100644 --- a/meta-oe/recipes-graphics/graphviz/graphviz_12.1.2.bb +++ b/meta-oe/recipes-graphics/graphviz/graphviz_12.1.2.bb @@ -20,10 +20,6 @@ inherit autotools-brokensep pkgconfig gettext qemu SRC_URI = "https://gitlab.com/api/v4/projects/4207231/packages/generic/${BPN}-releases/${PV}/${BP}.tar.xz \ " -# Use native mkdefs -SRC_URI:append:class-target = "\ - file://0001-Set-use_tcl-to-be-empty-string-if-tcl-is-disabled.patch \ -" SRC_URI:append:class-nativesdk = "\ file://graphviz-setup.sh \ "