MacOS X: Clarify doc string and fix test-dynload.

MacOS X uses .dylib for shared libraries and .so for modules. Add that
to gnc-module.c and make test-dynload also check for libgncmodule.dylib.


git-svn-id: svn+ssh://svn.gnucash.org/repo/gnucash/branches/reshuffle-modules@15315 57a11ea4-9604-0410-9ed3-97b8803252fd

Author: andi5

src/gnc-module/gnc-module.c

@@ -207,8 +207,11 @@ gnc_module_system_refresh(void)
 
       while ((dent = readdir(d)) != NULL)
       {
-        /* is the file a .la shared lib? */
-        /* FIXME: remove/#ifdef hardcoded .dylib suffix */
+        /* is the file a loadable module? */
+
+        /* Gotcha: On MacOS, G_MODULE_SUFFIX is defined as "so", but if we do
+         * not build clean libtool modules with "-module", we get dynamic
+         * libraries ending on .dylib */
         if (g_str_has_suffix(dent->d_name, "." G_MODULE_SUFFIX) ||
             g_str_has_suffix(dent->d_name, ".dylib"))
         {

src/gnc-module/test/test-dynload.c

@@ -17,6 +17,11 @@ guile_main(void *closure, int argc, char ** argv)
 
   printf("  test-dynload.c: testing dynamic linking of libgncmodule ...");
   gmodule = g_module_open("libgncmodule", 0);
+
+  /* Maybe MacOS? */
+  if (!gmodule)
+    gmodule = g_module_open("libgncmodule.dylib", 0);
+
   if (gmodule) {
     gpointer ptr;
     if (g_module_symbol(gmodule, "gnc_module_system_init", &ptr)) {