diff mbox series

[bug#71950] services: web: Return success from nginx-action.

Message ID 400feb0255e86efee23f561e97bc53c6818c3bb0.1720191689.git.~@wolfsden.cz
State New
Headers show
Series [bug#71950] services: web: Return success from nginx-action. | expand

Commit Message

Tomas Volf July 5, 2024, 3:01 p.m. UTC
Previous value of #f lead to `head reload nginx' returning exit code 1,
complicating usage from scripts (and other actions).  Returning #t fixes that.
Any actual failures are still covered by the invoke call above.

* gnu/services/web.scm (nginx-shepherd-service)[nginx-action]<-s>: Return #t.

Change-Id: I4ce1645798a85baddeb0f5bd702a2567db3c7aaa
---
 gnu/services/web.scm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff mbox series

Patch

diff --git a/gnu/services/web.scm b/gnu/services/web.scm
index b73cd30fe7..af4d6de658 100644
--- a/gnu/services/web.scm
+++ b/gnu/services/web.scm
@@ -831,7 +831,7 @@  (define (nginx-shepherd-service config)
              #~(lambda _
                  (invoke #$nginx-binary "-c" #$config-file #$@args)
                  (match '#$args
-                   (("-s" . _) #f)
+                   (("-s" . _) #t)
                    (_
                     ;; When FILE is true, we cannot be sure that PID-FILE will
                     ;; be created, so assume it won't show up.  When FILE is