| Message ID | 20170502201748.27846-1-aconole@redhat.com |
|---|---|
| State | Accepted |
| Headers | show |
On Tue, May 2, 2017 at 4:17 PM, Aaron Conole <aconole@redhat.com> wrote: > When commit d0c961a99f57 ("lib/automake.mk: don't install > runtime directories") landed, it broke RPM based builds since > the requisite directories were no longer available. This commit > adds those directories back when making RPMs so that the package > manager can see them. > > Fixes: d0c961a99f57 ("lib/automake.mk: don't install runtime directories") > Reported-by: Lance Richardson <lrichard@redhat.com> > Tested-by: Lance Richardson <lrichard@redhat.com> > Signed-off-by: Aaron Conole <aconole@redhat.com> applied to master, thanks!
diff --git a/rhel/openvswitch-fedora.spec.in b/rhel/openvswitch-fedora.spec.in index 5d53284..6f67413 100644 --- a/rhel/openvswitch-fedora.spec.in +++ b/rhel/openvswitch-fedora.spec.in @@ -230,6 +230,8 @@ make -f %{_datadir}/selinux/devel/Makefile rm -rf $RPM_BUILD_ROOT make install DESTDIR=$RPM_BUILD_ROOT +install -d -m 0755 $RPM_BUILD_ROOT%{_rundir}/openvswitch +install -d -m 0755 $RPM_BUILD_ROOT%{_localstatedir}/log/openvswitch install -d -m 0755 $RPM_BUILD_ROOT%{_sysconfdir}/openvswitch install -p -D -m 0644 \
When commit d0c961a99f57 ("lib/automake.mk: don't install runtime directories") landed, it broke RPM based builds since the requisite directories were no longer available. This commit adds those directories back when making RPMs so that the package manager can see them. Fixes: d0c961a99f57 ("lib/automake.mk: don't install runtime directories") Reported-by: Lance Richardson <lrichard@redhat.com> Tested-by: Lance Richardson <lrichard@redhat.com> Signed-off-by: Aaron Conole <aconole@redhat.com> --- NOTE: If there is a required corresponding change for Debian, please let me know. rhel/openvswitch-fedora.spec.in | 2 ++ 1 file changed, 2 insertions(+)