Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: ltspswapd - Daemon that uses nbd to provide swap space for LTSP thin clients https://bugzilla.redhat.com/show_bug.cgi?id=331531 ------- Additional Comments From pertusus@xxxxxxx 2007-10-24 12:19 EST ------- You should remove the dependency on /etc/ltsp5/ltsp.conf such that the package may be installed on its own. And in my opinion the comment in sysconfig/ltspswapd should be along # # the LTSP_SWAP_DIR variable is preferably set in /etc/ltsp5/ltsp.conf # to have a central config file. If you don't want to use /etc/ltsp5/ltsp.conf # you can also set this variable in this file # %{_localstatedir}/lib is already owned by the filesystem package. You could also add LTSP_SWAP_DIR in a comment in sysconfig/ltspswapd, like # LTSP_SWAP_DIR=/var/lib/ltsp5/swapfiles/ The following lines are not useful if the comment is not there: sed -e 's;LTSPSWAPD_FILEPATH=.*;LTSPSWAPD_FILEPATH=%{swapdir};' %{SOURCE1} > ${RPM_BUILD_ROOT}%{_sysconfdir}/sysconfig/ltspswapd touch -r %{SOURCE1} ${RPM_BUILD_ROOT}%{_sysconfdir}/sysconfig/ltspswapd And if the comment is there, LTSPSWAPD_FILEPATH should be changed to LTSP_SWAP_DIR -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug, or are watching someone who is. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review