[Bug 331731] Review Request: ltsp-server-config - LTSP5 server-side configurations and setup scripts

[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: ltsp-server-config - LTSP5 server-side configurations and setup scripts


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





------- Additional Comments From pertusus at free.fr  2007-10-16 04:57 EST -------
ltspfs-insecure should be a config script to be run by the admin
(or part of the main cionfig script) and not an init script.

init.d/ltsp-swapfile-delete shouldn't be automatically started
since there is already the cron.daily script that should take care
of stale files, and therefore the choice should be left to the
user.

The READMES should be in %doc.

ltsp-initialize should be in %_bindir.

I propose adding an /etc/sysconfig/ltsp5 file with 

LTSP_DEV="eth0"
LTSP_DEFAULTIP="192.168.0.254"
LTSP_DEFAULTMASK="255.255.255.0"

and source it in the beginning of ltsp-initialize.

It seems to me that ltsp-dhcpd.conf should be taken from
/usr/share/ltsp5/ by dhcpd-update if there is no 
/etc/ltsp-dhcpd.conf file, that way nothing would be
installed prior from ltsp-initialize being called.
This would solve the generality issue I pointed out
in the ltsp-dhcp review. In that case the comment
in spec file should be shortened.

it would be nice to provide scripts like those used
for other display managers for the 2 other dm of fedora
wdm and slim. Not a blocker, and certainly a job for
the dm packagers (I am the wdm packager...).

-- 
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.