#258 TclMagick requires tk even when configure --without-tk

[Benjamin Brink]

This case on Ubuntu 12.04 LTS shows tcl built as part of naviserver/openacs. However, building TclMagick fails:

root@...:/usr/local/src/GraphicsMagick-1.3.19/TclMagick# ./configure
--with-magick=/usr/local/src/GraphicsMagick-1.3.19/wand/GraphicsMagickWand-config
checking for a BSD-compatible install... /usr/bin/install -c
...
checking for Tk configuration... configure: WARNING: Can't find Tk
configuration definitions

Details in thread:
https://sourceforge.net/p/graphicsmagick/mailman/graphicsmagick-tclmagick/?viewmonth=201401&viewday=9&style=threaded

[Tobias Mark]

Hi; I've seen your call for help and that seemed like something i could do.

I've create a merge Request for this:
https://sourceforge.net/p/graphicsmagick/code/merge-requests/16/

However ... i stumbled upon additional problems after that.
make install failed for me
after fixing make install loading this in tcl fails
* also tests/ all fail to load the package
Seems strange that this did not happen before so maybe I'am doing something wrong? Because the make compiles all under generic/ not under unix/ like install/tests seem to expect
Seems that after all this is still nothing for me to help :/

For me fixing the install would be:

--- a/TclMagick/unix/m4/tcl.m4  Fri Jan 14 11:53:57 2022 +0100
+++ b/TclMagick/unix/m4/tcl.m4  Fri Jan 14 12:06:43 2022 +0100
@@ -3340,7 +3340,7 @@ 
     AC_PROG_CC
     AC_PROG_CPP
·
-    INSTALL='$(SHELL) $(srcdir)/config/install-sh -c'
+    INSTALL='$(SHELL) $(top_srcdir)/unix/config/install-sh -c'
     INSTALL_DATA_DIR='${INSTALL} -d -m 755'
     INSTALL_DATA='${INSTALL} -m 644'
     INSTALL_PROGRAM='${INSTALL}'

And fixing the loading is:

---a/TclMagick/pkgIndex.tcl 2021-05-25 15:36:16.312617911 +0200
+++ b/TclMagick/pkgIndex.tcl    2022-01-14 12:36:12.972626122 +0100
@@ -4,9 +4,17 @@
 if { ! [info exists ::env(MAGICK_CONFIGURE_PATH)] } {
 [list set ::env(MAGICK_CONFIGURE_PATH) [file join $dir config]]
 }
-[list load [file join $dir TclMagick[info sharedlibextension]]]"
+[list load [file join $dir [switch $::tcl_platform(platform) {
+   windows { set f TclMagick.dll }
+   unix { set f libTclMagick[info sharedlibextension] }
+    }]
+]]"

 package ifneeded TkMagick 0.46 "
 package require Tk
 package require TclMagick
-[list load [file join $dir TkMagick[info sharedlibextension]]]"
+[list load [file join $dir [switch $::tcl_platform(platform) {
+   windows { set f TkMagick.dll }
+   unix { set f libTkMagick[info sharedlibextension] }
+    }]
+]]"

[Tobias Mark]

Possible patch from merge request attached. Closed merge request because of hg problems.