[Bug 204694] Review Request: zvbi - Raw VBI, Teletext and Closed Caption decoding library

[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: zvbi - Raw VBI, Teletext and Closed Caption decoding library


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





------- Additional Comments From packages at 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.


-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.