[exim-cvs] use ifdefs for the routers-supported -bV rather t…

Top Page
Delete this message
Reply to this message
Author: Exim Git Commits Mailing List
Date:  
To: exim-cvs
Subject: [exim-cvs] use ifdefs for the routers-supported -bV rather than the info table
Gitweb: https://git.exim.org/exim.git/commitdiff/86489f8a56992071665242707cb52a7800677a0e
Commit:     86489f8a56992071665242707cb52a7800677a0e
Parent:     bec360ab3e1060187724480d109884e56f912b0a
Author:     Jeremy Harris <jgh146exb@???>
AuthorDate: Tue Aug 13 10:31:05 2024 +0100
Committer:  Jeremy Harris <jgh146exb@???>
CommitDate: Wed Aug 14 09:01:27 2024 +0100


    use ifdefs for the routers-supported -bV rather than the info table
---
 src/src/drtables.c | 81 ++++++++++++++++++++++++++++++++++++++++++------------
 1 file changed, 63 insertions(+), 18 deletions(-)


diff --git a/src/src/drtables.c b/src/src/drtables.c
index 2782323c3..987846bac 100644
--- a/src/src/drtables.c
+++ b/src/src/drtables.c
@@ -363,24 +363,6 @@ router_info routers_available_oldarray[] = {
   .ri_flags =        0
   },
 #endif
-#ifdef ROUTER_QUERYPROGRAM
-  {
-  .drinfo = {
-    .driver_name =    US"queryprogram",
-    .options =        queryprogram_router_options,
-    .options_count =    &queryprogram_router_options_count,
-    .options_block =    &queryprogram_router_option_defaults,
-    .options_len =    sizeof(queryprogram_router_options_block),
-    .init =        queryprogram_router_init,
-# if ROUTER_QUIRYPROGRAM==2
-    .dynamic =        TRUE,
-# endif
-    },
-  .code =        queryprogram_router_entry,
-  .tidyup =        NULL,     /* no tidyup entry */
-  .ri_flags =        0
-  },
-#endif
 #ifdef ROUTER_REDIRECT
   {
   .drinfo = {
@@ -518,10 +500,69 @@ return string_cat(g, US"\n");
 gstring *
 route_show_supported(gstring * g)
 {
+gstring * b = NULL, * d = NULL;
+
+#ifdef old
 g = string_cat(g, US"Routers:");
 /*XXX these run off the static list as we want them before the conf is read */
+/*XXX lookup_show_supported is in exim.c and just works off the #defines, with hardoded strings */
 for (router_info * rr = routers_available_oldarray; rr->drinfo.driver_name[0]; rr++)
            g = string_fmt_append(g, " %s", rr->drinfo.driver_name);
+#else
+
+#ifdef ROUTER_ACCEPT
+# if ROUTER_ACCEPT!=2
+  b = string_cat(b, US" accept");
+# else
+  d = string_cat(d, US" cdb");
+# endif
+#endif
+#ifdef ROUTER_DNSLOOKUP
+# if ROUTER_DNSLOOKUP!=2
+  b = string_cat(g, US" dnslookup");
+# else
+  d = string_cat(g, US" dnslookup");
+# endif
+#endif
+# ifdef ROUTER_IPLITERAL
+# if ROUTER_IPLITERAL!=2
+  b = string_cat(g, US" ipliteral");
+# else
+  d = string_cat(g, US" ipliteral");
+# endif
+#endif
+#ifdef ROUTER_IPLOOKUP
+# if ROUTER_IPLOOKUP!=2
+  b = string_cat(g, US" iplookup");
+# else
+  d = string_cat(g, US" iplookup");
+# endif
+#endif
+#ifdef ROUTER_MANUALROUTE
+# if ROUTER_MANUALROUTE!=2
+  b = string_cat(g, US" manualroute");
+# else
+  d = string_cat(g, US" manualroute");
+# endif
+#endif
+#ifdef ROUTER_REDIRECT
+# if ROUTER_REDIRECT!=2
+  b = string_cat(g, US" redirect");
+# else
+  d = string_cat(g, US" redirect");
+# endif
+#endif
+#ifdef ROUTER_QUERYPROGRAM
+# if ROUTER_QUERYPROGRAM!=2
+  b = string_cat(g, US" queryprogram");
+# else
+  d = string_cat(g, US" queryprogram");
+# endif
+#endif
+
+if (b) g = string_fmt_append(g, "Routers (built-in):%Y\n", b);
+if (d) g = string_fmt_append(g, "Routers (dynamic): %Y\n", d);
+#endif
 return string_cat(g, US"\n");
 }


@@ -774,6 +815,10 @@ addlookupmodule(NULL, &testdb_lookup_module_info);
addlookupmodule(NULL, &whoson_lookup_module_info);
#endif

+/* This is a custom expansion, and not available as either
+a list-syntax lookup or a lookup expansion. However, it is
+implemented by a lookup module. */
+
addlookupmodule(NULL, &readsock_lookup_module_info);

#ifdef LOOKUP_MODULE_DIR

--
## subscription configuration (requires account):
## https://lists.exim.org/mailman3/postorius/lists/exim-cvs.lists.exim.org/
## unsubscribe (doesn't require an account):
## exim-cvs-unsubscribe@???
## Exim details at http://www.exim.org/
## Please use the Wiki with this list - http://wiki.exim.org/