On Thu, Dec 22, 2016 at 01:02:20PM -0500, Jarod Wilson wrote: > These are numerous additional little fixups for the Fedora/Red Hat spec > file, stemming from Fedora package review. > > - Per https://fedoraproject.org/wiki/Packaging:Systemd it seems we need to > have systemd as a BuildRequires for %{_unitdir} to evaluate properly. > > - Add explicit dependencies on kmod, initscripts and systemd, as there > are container and vm cases where there might not be a kernel > installed Do we need initscripts? > - Make sure srp_daemon.sh is executable Oops, lets fix that commonly please, can you roll this in and drop the chmod: diff --git a/srp_daemon/CMakeLists.txt b/srp_daemon/CMakeLists.txt index f2752a8e6e1580..acb851bcb8a438 100644 --- a/srp_daemon/CMakeLists.txt +++ b/srp_daemon/CMakeLists.txt @@ -22,7 +22,8 @@ rdma_install_symlink(srp_daemon "${CMAKE_INSTALL_SBINDIR}/ibsrpdm") rdma_install_symlink(srp_daemon "${CMAKE_INSTALL_SBINDIR}/run_srp_daemon") rdma_subst_install(FILES "srp_daemon.sh.in" DESTINATION "${CMAKE_INSTALL_SBINDIR}" - RENAME "srp_daemon.sh") + RENAME "srp_daemon.sh" + PERMISSIONS OWNER_WRITE OWNER_READ GROUP_READ WORLD_READ OWNER_EXECUTE GROUP_EXECUTE WORLD_EXECUTE) install(FILES logrotate-srp_daemon DESTINATION "${CMAKE_INSTALL_SYSCONFDIR}/logrotate.d" RENAME "srp_daemon") install(FILES rsyslog-srp_daemon.conf DESTINATION "${CMAKE_INSTALL_SYSCONFDIR}/rsyslog.d" RENAME "srp_daemon.conf") > Requires: %{name}%{?_isa} = %{version}-%{release} > +Requires: logrotate I don't think we use logrotate for anything unless the srp rsyslog examples are being used? Maybe it would be better to move the logrotate and rsyslog stuff to doc/examples or something? Jason -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html