While looking into [0], I noticed that main() still only checks for the
--fork prefix, but IIUC commit aafc05d removed all --fork* options except
for --forkchild.  I've attached a patch to strengthen the check in main().
This is definitely just a nitpick.

[0] 
https://postgr.es/m/CAKAnmmJkZtZAiSryho%3DgYpbvC7H-HNjEDAh16F3SoC9LPu8rqQ%40mail.gmail.com

-- 
nathan
diff --git a/src/backend/main/main.c b/src/backend/main/main.c
index bfd0c5ed65..4672aab837 100644
--- a/src/backend/main/main.c
+++ b/src/backend/main/main.c
@@ -185,7 +185,7 @@ main(int argc, char *argv[])
        else if (argc > 1 && strcmp(argv[1], "--boot") == 0)
                BootstrapModeMain(argc, argv, false);
 #ifdef EXEC_BACKEND
-       else if (argc > 1 && strncmp(argv[1], "--fork", 6) == 0)
+       else if (argc > 1 && strncmp(argv[1], "--forkchild", 11) == 0)
                SubPostmasterMain(argc, argv);
 #endif
        else if (argc > 1 && strcmp(argv[1], "--describe-config") == 0)

Reply via email to