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: zvbi - Raw VBI, Teletext and Closed Caption decoding library
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=204694
------- Additional Comments From packages@amiga-hardware.com 2006-09-01 12:37 EST -------
W: zvbi no-reload-entry /etc/rc.d/init.d/zvbid In your init script (/etc/rc.d/init.d/your_file), you don't have a 'reload' entry, which is necessary for good functionality.
W: zvbi incoherent-init-script-name zvbid The init script name should be the same as the package name in lower case.
all above warnings have their descriptions given about how to solve them so follow that.
I'm in agreement with Rex on this one, I think calling the init script zvbi in this case doesn't make much sense, particularly when the daemon is really a small subset of the whole zvbi package, as opposed to being the primary function. It would require more than a simple namechange as the init script would need to be patched so that the 'subsystems' had matching names. Overkill I think for simply dropping the trailing 'd'.
With wrt the reload option as it's considered optional I would rather not add it. IIRC primary reason for a reload option is to tell the daemon to reload its config files without quitting and starting again which is very useful for daemons that don't instantly stop or start such as squid, but in this case zvbid doesn't load configs.