Merge lp:~jontai/openvista-gtm-integration/bug366576 into lp:openvista-gtm-integration

Proposed by Jon Tai
Status: Merged
Merged at revision: not available
Proposed branch: lp:~jontai/openvista-gtm-integration/bug366576
Merge into: lp:openvista-gtm-integration
Diff against target: None lines
To merge this branch: bzr merge lp:~jontai/openvista-gtm-integration/bug366576
Reviewer Review Type Date Requested Status
jeff.apple Approve
Review via email: mp+5895@code.launchpad.net
To post a comment you must log in.
Revision history for this message
jeff.apple (jeff-apple) wrote :

I'm not familiar with this construct, but I assume that the : is basically a no-op, so "|| :" succeed or do nothing.
Since this package requires httpd, how is it that httpd can be missing? Just curious.

review: Approve
Revision history for this message
Jon Tai (jontai) wrote :

> I'm not familiar with this construct, but I assume that the : is basically a
> no-op, so "|| :" succeed or do nothing.

Exactly.

> Since this package requires httpd, how is it that httpd can be missing? Just
> curious.

In the case that you have installed httpd and openvista-webdav-config, then you later want to remove httpd, you must also remove openvista-webdav-config, since it's a dependency. While RPM will force you to remove openvsita-webdav-config, it may not necessarily uninstall it first. So by the time the postuninstall scriptlet runs, the httpd init script is gone already.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'packages/rpm/openvista-webdav-config.spec'
2--- packages/rpm/openvista-webdav-config.spec 2009-04-22 04:16:31 +0000
3+++ packages/rpm/openvista-webdav-config.spec 2009-04-24 20:40:45 +0000
4@@ -47,10 +47,10 @@
5 %dir /var/www/html/webdav
6
7 %post
8-/sbin/service httpd reload > /dev/null 2>&1
9+/sbin/service httpd reload > /dev/null 2>&1 || :
10
11 %postun
12 if [ $1 = 0 ]; then
13- /sbin/service httpd reload > /dev/null 2>&1
14+ /sbin/service httpd reload > /dev/null 2>&1 || :
15 fi
16

Subscribers

People subscribed via source and target branches