]> git.ipfire.org Git - thirdparty/libtool.git/commitdiff
libtool: Handle unset $multilib for Windows targets.
authorMarkus Mützel <markus.muetzel@gmx.de>
Mon, 14 Oct 2024 15:25:07 +0000 (17:25 +0200)
committerIleana Dumitrescu <ileanadumitrescu95@gmail.com>
Tue, 22 Oct 2024 17:00:47 +0000 (20:00 +0300)
If `multilib` is unset (which it apparently is by default), the output
from running a `configure` script contains lines like the following:
```
checking dynamic linker characteristics... ../configure: line 26791: test: yes: unary operator expected
Win32 ld.exe
```

The additional output is not a fatal error. But it could be irritating.

Avoid the unexpected output from `test` by making sure `=` is used as a
binary operator.

* m4/libtool.m4: Prepend "x" on both sides of a comparison with `test`
and the operator `=`.
* NEWS: Update for bug fix.

See also: https://github.com/msys2/MINGW-packages/pull/21905

NEWS
m4/libtool.m4

diff --git a/NEWS b/NEWS
index c83ad75bbd601d44df80c15dd8ed3639476ea114..2e87652a1e20f295f64aa2048c73a2fbb563d6d5 100644 (file)
--- a/NEWS
+++ b/NEWS
@@ -12,6 +12,8 @@ NEWS - list of user-visible changes between releases of GNU Libtool
   - Fix bug where the constructed rpath omit directories, instead of
     appending them to the end.
 
+  - Fix configure error for when variable 'multlib' is unset.
+
 
 * Noteworthy changes in release 2.5.3 (2024-09-25) [stable]
 
index 5f701b702e91b64ce39cca86bba667d52fd7644e..070cb29d7c3e1dc636ef6c9f82ee91d9b6d01f50 100644 (file)
@@ -2584,7 +2584,7 @@ cygwin* | mingw* | windows* | pw32* | cegcc*)
     # If user builds GCC with mulitlibs enabled,
     # it should just install on $(libdir)
     # not on $(libdir)/../bin or 32 bits dlls would override 64 bit ones.
-    if test yes = $multilib; then
+    if test xyes = x$multilib; then
     postinstall_cmds='base_file=`basename \$file`~
       dlpath=`$SHELL 2>&1 -c '\''. $dir/'\''\$base_file'\''i; echo \$dlname'\''`~
       dldir=$destdir/`dirname \$dlpath`~