Bug#886544: redis-server: Replacing set -e in sysvinit script

2018-08-19 Thread SZÉPE Viktor
Idézem/Quoting Chris Lamb : Chris Lamb wrote: > Please see the original report citing why "set -e" is not something to > be considered in an init script. I have re-read this and do not understand the rationale. In my experienece, `set - e` ensures that the script is not full of error checking

Bug#886544: redis-server: Replacing set -e in sysvinit script

2018-08-19 Thread Chris Lamb
Chris Lamb wrote: > > Please see the original report citing why "set -e" is not something to > > be considered in an init script. > > I have re-read this and do not understand the rationale. In my > experienece, `set - e` ensures that the script is not full of error > checking code that is, in-

Bug#886544: redis-server: Replacing set -e in sysvinit script

2018-07-22 Thread Chris Lamb
SZÉPE Viktor wrote: > Please see the original report citing why "set -e" is not something to > be considered in an init script. I have re-read this and do not understand the rationale. In my experienece, `set - e` ensures that the script is not full of error checking code that is, in- itself, p

Bug#886544: redis-server: Replacing set -e in sysvinit script

2018-07-22 Thread SZÉPE Viktor
Thank you for your answer. Idézem/Quoting Chris Lamb : The only purpose of this bug is to have a non-"set -e" redis package in Debian. I think I'm missing something here. I don't understand the rationale or motivation behind wanting to remove set -e. The scripts work fine as they are, unless

Bug#886544: redis-server: Replacing set -e in sysvinit script

2018-07-22 Thread Chris Lamb
tags 886544 + moreinfo thanks Hi, > The only purpose of this bug is to have a non-"set -e" redis package > in Debian. I think I'm missing something here. I don't understand the rationale or motivation behind wanting to remove set -e. The scripts work fine as they are, unless you are reporting

Bug#886544: redis-server: Replacing set -e in sysvinit script

2018-07-22 Thread SZÉPE Viktor
Idézem/Quoting Chris Lamb : Chris Lamb wrote: I don't want to play "severity wars" but I'm not sure what the exact problem it is you are reporting here. […] Gentle ping on this (perhaps non-Debian?) bug Hello Chris! The only purpose of this bug is to have a non-"set -e" redis package in

Bug#886544: redis-server: Replacing set -e in sysvinit script

2018-07-22 Thread Chris Lamb
Chris Lamb wrote: > I don't want to play "severity wars" but I'm not sure what the exact > problem it is you are reporting here. […] Gentle ping on this (perhaps non-Debian?) bug Regards, -- ,''`. : :' : Chris Lamb `. `'` la...@debian.org / chris-lamb.co.uk `-

Bug#886544: redis-server: Replacing set -e in sysvinit script

2018-01-08 Thread Chris Lamb
severity 886544 wishlist thanks Hi Viktor, First, thanks for the bug report and for forwarding it from GitHub! > Severity: important I don't want to play "severity wars" but I'm not sure what the exact problem it is you are reporting here. Perhaps using "set -e" is good or bad, but are you hit

Bug#886544: redis-server: Replacing set -e in sysvinit script

2018-01-07 Thread Viktor Szépe
Package: redis-server Version: 2:3.2.11-1~dotdeb+8.1 Severity: important Was https://github.com/lamby/pkg-redis/issues/14 Please consider replacing `set -e` with step-by-step error checking in SysVInit scripts. > Be careful of using set -e in init.d scripts https://www.debian.org/doc/debian-po