1) It seems you decided to stick with /etc/rc.d/init.d for now and 
make /etc/init.d just a link, so it should say

 /etc/rc.d/init.d/<servicename>
instead of
 /etc/init.d/<servicename>

since all the rpms use directly /etc/rc.d/init.d in their %files and not 
the link.


2) It should document "condrestart", that you added to all of the init 
scripts to conditionally restart services. 


3) The doc makes the example of

case $1 in

	start)
		<all the stuff here>


but all of your latest init scripts use:

start () {
	<all the stuff here>.
}


case $1 in
	start) 
	start
...


Which is the preferred way if any?