Add comprehensive logging to library loading process

- Add logging functionality to track library loading workflow
- Improve error handling with try-catch and detailed error messages
- Add debug logging for architecture detection and path resolution
- Log library path existence and loading status
- Break down complex operations for better traceability
- Fix typo in LibraryLoadError message (path -> lib)

This change makes it easier to diagnose library loading issues by providing detailed logs about the loading process, architecture detection, and path resolution steps.

Author: ctoth

libloader/libloader.py

@@ -3,6 +3,10 @@
 import platform
 import os
 import sys
+import logging
+
+# Set up logger
+logger = logging.getLogger(__name__)
 
 
 TYPES = {
@@ -33,29 +37,60 @@ class LibraryLoadError(OSError):
 
 
 def load_library(library, x86_path=".", x64_path=".", *args, **kwargs):
+    logger.info("Attempting to load library: %s", library)
+    logger.debug("x86_path: %s, x64_path: %s", os.path.abspath(x86_path), os.path.abspath(x64_path))
+    
     lib = find_library_path(library, x86_path=x86_path, x64_path=x64_path)
+    logger.info("Resolved library path: %s", lib)
+    
     loaded = _do_load(lib, *args, **kwargs)
     if loaded is not None:
+        logger.info("Successfully loaded library: %s", lib)
         return loaded
+    
+    logger.error("Failed to load library: %s", lib)
     raise LibraryLoadError(
-        "unable to load %r. Provided library path: %r" % (library, path)
+        "unable to load %r. Provided library path: %r" % (library, lib)
     )
 
 
 def _do_load(file, *args, **kwargs):
-    loader = TYPES[platform.system()]["loader"]
-    return loader(file, *args, **kwargs)
+    system = platform.system()
+    logger.debug("Using loader for system: %s", system)
+    loader = TYPES[system]["loader"]
+    try:
+        return loader(file, *args, **kwargs)
+    except Exception as e:
+        logger.error("Error loading %s: %s", file, str(e))
+        return None
 
 
 def find_library_path(libname, x86_path=".", x64_path="."):
-    libname = "%s%s" % (TYPES[platform.system()]["prefix"], libname)
-    if platform.architecture()[0] == "64bit":
-        path = os.path.join(x64_path, libname)
+    system = platform.system()
+    prefix = TYPES[system]["prefix"]
+    libname_with_prefix = "%s%s" % (prefix, libname)
+    logger.debug("Library name with prefix: %s", libname_with_prefix)
+    
+    arch = platform.architecture()[0]
+    logger.debug("Detected architecture: %s", arch)
+    
+    if arch == "64bit":
+        path = os.path.join(x64_path, libname_with_prefix)
+        logger.debug("Using 64-bit path: %s", x64_path)
     else:
-        path = os.path.join(x86_path, libname)
+        path = os.path.join(x86_path, libname_with_prefix)
+        logger.debug("Using 32-bit path: %s", x86_path)
+    
     ext = get_library_extension()
+    logger.debug("Using library extension: %s", ext)
+    
     path = "%s%s" % (path, ext)
-    return os.path.abspath(path)
+    abs_path = os.path.abspath(path)
+    
+    logger.info("Resolved library path: %s", abs_path)
+    logger.debug("Path exists: %s", os.path.exists(abs_path))
+    
+    return abs_path
 
 
 def get_functype():