[Bug 1322168] Review Request: ibacm - InfiniBand Communication Manager Assistant

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



https://bugzilla.redhat.com/show_bug.cgi?id=1322168



--- Comment #7 from Honggang LI <honli@xxxxxxxxxx> ---
diff -u ibacm.spec.old  ibacm.spec
--- ibacm.spec.old      2016-04-05 03:38:44.606740117 -0400
+++ ibacm.spec  2016-04-05 03:49:53.128820748 -0400
@@ -1,6 +1,6 @@
 Name: ibacm
 Version: 1.2.0
-Release: 2%{?dist}
+Release: 3%{?dist}
 Summary: InfiniBand Communication Manager Assistant
 License: GPLv2 or BSD
 Url: http://www.openfabrics.org/
@@ -31,7 +31,7 @@

 %package devel
 Summary: Headers file needed when building apps to talk directly to ibacm
-Requires: %{name} = %{version}-%{release}
+Requires: %{name}%{?_isa} = %{version}-%{release}

 %description devel
 Most applications do not need to know how to talk directly to the ibacm
@@ -64,8 +64,8 @@
 make DESTDIR=%{buildroot} install
 rm -fr %{buildroot}%{_sysconfdir}/init.d
 find %{buildroot} -name '*.la' -delete
-install -D -m 644 ibacm_opts.cfg
%{buildroot}%{_sysconfdir}/rdma/ibacm_opts.cfg
-install -D -m 644 %{SOURCE1} %{buildroot}%{_unitdir}/ibacm.service
+install -D -p -m 644 ibacm_opts.cfg
%{buildroot}%{_sysconfdir}/rdma/ibacm_opts.cfg
+install -D -p -m 644 %{SOURCE1} %{buildroot}%{_unitdir}/ibacm.service

 %post
 %systemd_post ibacm.service
@@ -81,6 +81,7 @@
 %license COPYING
 %{_bindir}/ib_acme
 %{_sbindir}/ibacm
+%dir %{_sysconfdir}/rdma
 %config(noreplace)%{_sysconfdir}/rdma/ibacm_opts.cfg
 %{_mandir}/man1/*
 %{_mandir}/man7/*
@@ -88,10 +89,16 @@
 %{_libdir}/ibacm

 %files devel
+%dir %{_includedir}/infiniband
 %{_includedir}/infiniband/acm.h
 %{_includedir}/infiniband/acm_prov.h

 %changelog
+* Tue Apr  5 2016 Honggang Li <honli@xxxxxxxxxx> - 1.2.0-3
+- Requiring arch-specific base package.
+- Preserve the timestamp of bacm_opts.cfg and ibacm.service.
+- Own all directories that it creates.
+
 * Fri Apr  1 2016 Honggang Li <honli@xxxxxxxxxx> - 1.2.0-2
 - Improve the spec file based on Fedora review

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
http://lists.fedoraproject.org/admin/lists/package-review@xxxxxxxxxxxxxxxxxxxxxxx




[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]