[ovs-dev] [PATCH] rhel: fix ovn-common rpm installation failure

Ben Pfaff blp at ovn.org
Tue Nov 22 23:05:33 UTC 2016


On Mon, Nov 21, 2016 at 05:47:13PM -0500, Lance Richardson wrote:
> The directory /usr/lib/ocf/ does not exist if the pacemaker
> package has not been installed, which causes installation of the
> ovn-common rpm to fail on "mkdir /usr/lib/ocf/resource.d/ovn".
> Allow for the possibility that /usr/lib/ocf does not exist by
> using "mkdir -p".
> 
> Fixes: a4245b7869c8 ("ovn: Add ovn db servers ocf script in fedora packager")
> Signed-off-by: Lance Richardson <lrichard at redhat.com>
> ---
>  rhel/openvswitch-fedora.spec.in | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/rhel/openvswitch-fedora.spec.in b/rhel/openvswitch-fedora.spec.in
> index d2cb3cd..d1f33af 100644
> --- a/rhel/openvswitch-fedora.spec.in
> +++ b/rhel/openvswitch-fedora.spec.in
> @@ -311,7 +311,7 @@ rm -rf $RPM_BUILD_ROOT
>  %endif
>  
>  %post ovn-common
> -mkdir /usr/lib/ocf/resource.d/ovn
> +mkdir -p /usr/lib/ocf/resource.d/ovn

I'm a little surprised to see an empty directory getting created in a
script.  I would have expected it to appear in the RPM file archive.  At
least, that's how I'd do it in Debian packaging; I don't know RPMs very
well.


More information about the dev mailing list