Merge lp:~townsend/libertine/fix-pkill-maint-scripts into lp:libertine

Proposed by Christopher Townsend
Status: Merged
Approved by: Larry Price
Approved revision: 452
Merged at revision: 452
Proposed branch: lp:~townsend/libertine/fix-pkill-maint-scripts
Merge into: lp:libertine
Diff against target: 25 lines (+2/-2)
2 files modified
debian/libertined.preinst (+1/-1)
debian/libertined.prerm (+1/-1)
To merge this branch: bzr merge lp:~townsend/libertine/fix-pkill-maint-scripts
Reviewer Review Type Date Requested Status
Libertine CI Bot continuous-integration Approve
Larry Price Approve
Review via email: mp+321340@code.launchpad.net

Commit message

Add a '-x' option to pkill so it only kills the libertined process(es) and nothing else.

To post a comment you must log in.
Revision history for this message
Larry Price (larryprice) wrote :

ok makes sense

review: Approve
Revision history for this message
Libertine CI Bot (libertine-ci-bot) wrote :

PASSED: Continuous integration, rev:452
https://jenkins.canonical.com/libertine/job/lp-libertine-ci/494/
Executed test runs:
    SUCCESS: https://jenkins.canonical.com/libertine/job/build/893
    SUCCESS: https://jenkins.canonical.com/libertine/job/test-0-autopkgtest/label=amd64,release=xenial+overlay,testname=default/737
    SUCCESS: https://jenkins.canonical.com/libertine/job/test-0-autopkgtest/label=amd64,release=zesty,testname=default/737
    SUCCESS: https://jenkins.canonical.com/libertine/job/test-0-autopkgtest/label=i386,release=xenial+overlay,testname=default/737
    SUCCESS: https://jenkins.canonical.com/libertine/job/test-0-autopkgtest/label=i386,release=zesty,testname=default/737
    SUCCESS: https://jenkins.canonical.com/libertine/job/build-0-fetch/903
    SUCCESS: https://jenkins.canonical.com/libertine/job/build-2-binpkg/arch=amd64,release=xenial+overlay/894
        deb: https://jenkins.canonical.com/libertine/job/build-2-binpkg/arch=amd64,release=xenial+overlay/894/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/libertine/job/build-2-binpkg/arch=amd64,release=zesty/894
        deb: https://jenkins.canonical.com/libertine/job/build-2-binpkg/arch=amd64,release=zesty/894/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/libertine/job/build-2-binpkg/arch=i386,release=xenial+overlay/894
        deb: https://jenkins.canonical.com/libertine/job/build-2-binpkg/arch=i386,release=xenial+overlay/894/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/libertine/job/build-2-binpkg/arch=i386,release=zesty/894
        deb: https://jenkins.canonical.com/libertine/job/build-2-binpkg/arch=i386,release=zesty/894/artifact/output/*zip*/output.zip

Click here to trigger a rebuild:
https://jenkins.canonical.com/libertine/job/lp-libertine-ci/494/rebuild

review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'debian/libertined.preinst'
--- debian/libertined.preinst 2017-03-06 16:15:27 +0000
+++ debian/libertined.preinst 2017-03-29 17:35:25 +0000
@@ -16,7 +16,7 @@
1616
17if [ "$1" = "upgrade" ]; then17if [ "$1" = "upgrade" ]; then
18 echo "Shutting down the libertined service..."18 echo "Shutting down the libertined service..."
19 pkill libertined19 pkill -x libertined
20fi20fi
2121
22#DEBHELPER#22#DEBHELPER#
2323
=== modified file 'debian/libertined.prerm'
--- debian/libertined.prerm 2017-03-06 16:15:27 +0000
+++ debian/libertined.prerm 2017-03-29 17:35:25 +0000
@@ -16,7 +16,7 @@
1616
17if [ "$1" = "remove" ]; then17if [ "$1" = "remove" ]; then
18 echo "Shutting down the libertined service..."18 echo "Shutting down the libertined service..."
19 pkill libertined19 pkill -x libertined
20fi20fi
2121
22#DEBHELPER#22#DEBHELPER#

Subscribers

People subscribed via source and target branches