[PATCH v3 0/1] MR10876: wine: Show all dlopen() error messages instead of just the last one.

[Sven Baars (＠sbaars)]

I ran into this multiple times while working on arm64 builds, and the actually useful error is always one of the first ones, not the last one.

--
  v3: wine: Show all dlopen() error messages instead of just the last one.
 
https://gitlab.winehq.org/wine/wine/-/merge_requests/10876

[PATCH v3 1/1] wine: Show all dlopen() error messages instead of just the last one.

[Sven Baars]

From: Sven Baars <[e-mail]>

---
 tools/wine/wine.c | 20 ++++++++++++++------
 1 file changed, 14 insertions(+), 6 deletions(-)

diff --git a/tools/wine/wine.c b/tools/wine/wine.c
index ea756088cf8..502ba43b2be 100644
--- a/tools/wine/wine.c
+++ b/tools/wine/wine.c
@@ -27,28 +27,36 @@
 static const char *bindir;
 static const char *libdir;
 
+static void *try_dlopen( const char *path, struct strarray *errors )
+{
+    void *handle = dlopen( path, RTLD_NOW );
+    if (!handle) strarray_add( errors, xstrdup( dlerror() ) );
+    return handle;
+}
+
 static void *load_ntdll(void)
 {
     const char *arch_dir = get_arch_dir( get_default_target() );
-    struct strarray dllpath;
+    struct strarray dllpath, errors = { 0 };
     void *handle;
 
     if (bindir && strendswith( bindir, "/tools/wine" ) &&
-        ((handle = dlopen( strmake( "%s/../../dlls/ntdll/ntdll.so", bindir ), RTLD_NOW ))))
+        ((handle = try_dlopen( strmake( "%s/../../dlls/ntdll/ntdll.so", bindir ), &errors ))))
         return handle;
 
-    if ((handle = dlopen( strmake( "%s/wine%s/ntdll.so", libdir, arch_dir ), RTLD_NOW )))
+    if ((handle = try_dlopen( strmake( "%s/wine%s/ntdll.so", libdir, arch_dir ), &errors )))
         return handle;
 
     dllpath = strarray_frompath( getenv( "WINEDLLPATH" ));
     STRARRAY_FOR_EACH( dir, &dllpath )
     {
-        if ((handle = dlopen( strmake( "%s%s/ntdll.so", dir, arch_dir ), RTLD_NOW )))
+        if ((handle = try_dlopen( strmake( "%s%s/ntdll.so", dir, arch_dir ), &errors )))
             return handle;
-        if ((handle = dlopen( strmake( "%s/ntdll.so", dir ), RTLD_NOW )))
+        if ((handle = try_dlopen( strmake( "%s/ntdll.so", dir ), &errors )))
             return handle;
     }
-    fprintf( stderr, "wine: could not load ntdll.so: %s\n", dlerror() );
+    fprintf( stderr, "wine: could not load ntdll.so:\n" );
+    STRARRAY_FOR_EACH( error, &errors ) fprintf( stderr, "  %s\n", error );
     exit(1);
 }
 
-- 
GitLab

https://gitlab.winehq.org/wine/wine/-/merge_requests/10876

[Sven Baars (＠sbaars)]

On Tue May 12 15:59:21 2026 +0000, Jinoh Kang wrote:
> Having a dlopen wrapper that appends dlerror() on error would be nice.
> Sure, that also works. Thanks!

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/10876#note_139594