Hi,
On 11.04.2020 20:21, Michael Tremer wrote:
On 11 Apr 2020, at 19:17, Matthias Fischer matthias.fischer@ipfire.org wrote:
Hi,
On 11.04.2020 20:03, Michael Tremer wrote:
Hi,
On 11 Apr 2020, at 17:39, Matthias Fischer matthias.fischer@ipfire.org wrote:
Hi.
On 11.04.2020 17:28, Michael Tremer wrote:
Hello,
Thanks for the patch, but this only solves one of my concerns.
You mean the exit status for the 'stop'-section?
No, everywhere.
-v please.
LOL. Good one :)
:-)
So, you are launching vnstatd by calling loadproc.
Yep.
That function will start the process and print “OK” or “ERROR” in a colour on the console.
I noticed that, but...
Calling evaluate_retval (which should be called in loadproc, too) will then print OK again.
I saw that, too, but didn't think of ALL actions in that script.
Did it not do that when you tested it?
Nope. Doing it that way was a bit unfamiliar but I'll get used to it.
I had to take a look at the other inits and 'functions.pl'.
Now I finally know where the colorful "OKs" come from...you're never too old... ;-))
Does that last version fit your needs?
Current link is: https://git.ipfire.org/?p=people/mfischer/ipfire-2.x.git;a=blob;f=src/initsc...
Best, Matthias
-Michael
'evaluate_retval' is now four times in, status output has been added.
Anything else?
What about printing the colourful exit status more than once?
Done... ;-)
Best, -Michael
On 11 Apr 2020, at 15:38, Matthias Fischer matthias.fischer@ipfire.org wrote:
Removed 'sleep 2'
Signed-off-by: Matthias Fischer matthias.fischer@ipfire.org
src/initscripts/system/vnstat | 2 -- 1 file changed, 2 deletions(-)
diff --git a/src/initscripts/system/vnstat b/src/initscripts/system/vnstat index bcc19c3ab..a21905d75 100755 --- a/src/initscripts/system/vnstat +++ b/src/initscripts/system/vnstat @@ -20,14 +20,12 @@ case "$1" in
boot_mesg "Starting vnstatd..." loadproc /usr/sbin/vnstatd -d --alwaysadd
sleep 2evaluate_retval ;;
stop) boot_mesg "Stopping vnstatd..." killproc /usr/sbin/vnstatd
sleep 2evaluate_retval
umount_ramdisk "${VNSTATLOG}"
-- 2.18.0