[Bug 331531] Review Request: ltspswapd - Daemon that uses nbd to provide swap space for LTSP thin clients

[bugzilla at redhat.com]

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 at free.fr  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.