Bug#948115: Revise init script Policy based on GR result

2020-01-20 Thread Sean Whitton
control: tag -1 +pending Hello, On Sat 04 Jan 2020 at 05:51PM -05, Sam Hartman wrote: > Russ said off-list he was ready for seconds. > I second his patch with the status being encouraged rather than > recommended change. > In seconding, my primary review criteria was whether I thought the >

Bug#948115: Revise init script Policy based on GR result

2020-01-05 Thread Russ Allbery
Lorenzo Puliti writes: > I wonder if it make sense to encourage or even recommend to use the > interpreter in /lib/init/init-d-script for writing init scripts. > Or maybe just citing that there is an interpreter and say that > it's the preferred way to create init scripts, I don't know.. >

Bug#948115: Revise init script Policy based on GR result

2020-01-05 Thread Lorenzo Puliti
Package: debian-policy Version: 4.4.1.2 Followup-For: Bug #948115 Hi, > Writing the scripts > ~~~ > ... > +The ``start``, ``stop``, ``restart``, and ``force-reload`` options should > +be supported by all init scripts. Supporting ``status`` is recommended but > +not required.

Bug#948115: Revise init script Policy based on GR result

2020-01-04 Thread Sam Hartman
Russ said off-list he was ready for seconds. I second his patch with the status being encouraged rather than recommended change. In seconding, my primary review criteria was whether I thought the change accurately reflected what the GR conclusion was. --Sam signature.asc Description: PGP

Bug#948115: Revise init script Policy based on GR result

2020-01-04 Thread Russ Allbery
Sam Hartman writes: > I've reviewed your patch. > It looks good. > One minor suggestion: > +The ``start``, ``stop``, ``restart``, and ``force-reload`` options should > +be supported by all init scripts. Supporting ``status`` is recommended but > +not required. The ``reload`` and

Bug#948115: Revise init script Policy based on GR result

2020-01-04 Thread Sam Hartman
I've reviewed your patch. It looks good. One minor suggestion: +The ``start``, ``stop``, ``restart``, and ``force-reload`` options should +be supported by all init scripts. Supporting ``status`` is recommended but +not required. The ``reload`` and ``try-restart`` options are optional. How about

Bug#948115: Revise init script Policy based on GR result

2020-01-03 Thread Russ Allbery
Package: debian-policy Version: 4.4.1.2 Severity: important Per recent (non-BTS) discussion, this patch is a first draft at reconciling Policy with the recent GR result. Summary of changes: * Change section headings and anchors to reflect the more general topic * Add recommended naming