Attachment 37460 Details for Bug 13896 – [PASSED QA] Bug 13896: Fix pid file creation, use proper directory instead of /tmp/

[patch] [PASSED QA] Bug 13896: Fix pid file creation, use proper directory instead of /tmp/

PASSED-QA-Bug-13896-Fix-pid-file-creation-use-prop.patch (text/plain), 1.59 KB, created by Kyle M Hall (khall) on 2015-04-03 17:07:51 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Kyle M Hall (khall)

Created: 2015-04-03 17:07:51 UTC

Size: 1.59 KB

patch

obsolete

>From 24b16cddc711aff993537560de753cf67240514e Mon Sep 17 00:00:00 2001
>From: Attila Kinali <attila@kinali.ch>
>Date: Mon, 23 Mar 2015 22:58:22 +0100
>Subject: [PATCH] [PASSED QA] Bug 13896: Fix pid file creation, use proper directory instead of /tmp/
>
>daemon uses by default /tmp/ for pid files, if they are unspecified.
>Pass the right directory to daemon, so it knows where to put the pid file.
>
>Signed-off-by: Attila Kinali <attila@kinali.ch>
>
>TEST PLAN
>---------
>1) Roll a distribution and confirm it works.
>
>OR
>
>1) Hack all the /var directories to point to git directories
>2) Add a dummy user for the instance name
>3) sudo the script with the faked instance name.
>   -- the daemon should be running and the pid file should be in
>      the expected directory.
>
>The first way is the proper way to test, I did the latter. :)
>
>Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
>
>Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
>---
> debian/scripts/koha-start-zebra |    1 +
> 1 files changed, 1 insertions(+), 0 deletions(-)
>
>diff --git a/debian/scripts/koha-start-zebra b/debian/scripts/koha-start-zebra
>index 185a650..ce6c469 100755
>--- a/debian/scripts/koha-start-zebra
>+++ b/debian/scripts/koha-start-zebra
>@@ -40,6 +40,7 @@ start_zebra_instance()
>         "/var/log/koha/$instancename/zebra-output.log"
>     daemon \
>         --name="$instancename-koha-zebra" \
>+        --pidfiles="/var/run/koha/$instancename/" \
>         --errlog="/var/log/koha/$instancename/zebra-error.log" \
>         --stdout="/var/log/koha/$instancename/zebra.log" \
>         --output="/var/log/koha/$instancename/zebra-output.log" \
>-- 
>1.7.2.5

Actions: View | Diff | Splinter Review

Attachments on bug 13896: 37164 | 37434 | 37460 | 37589