[Merge] lp:~jamesodhunt/upstart/remove-initctl-job+instance-options into lp:upstart

James Hunt james.hunt at canonical.com
Sat Feb 2 16:22:19 UTC 2013


James Hunt has proposed merging lp:~jamesodhunt/upstart/remove-initctl-job+instance-options into lp:upstart.

Requested reviews:
  Upstart Reviewers (upstart-reviewers)

For more details, see:
https://code.launchpad.net/~jamesodhunt/upstart/remove-initctl-job+instance-options/+merge/146272

This branch removes the ability for:

- a user to modify an already running job instances environment table.
- a job to modify _another jobs_ running environment table.

The ability to do that was originally added since it simply reflects the full available D-Bus interface (which we do not wish to change) (*).

The rationale for removing these initctl options are:

1) There is no obvious use-case for such a feature.
2) Any use of this could lead to unexpected behaviour and should thus be avoided.

(*) - Removing these initctl options does not remove the ability to use such a feature, but it does make it marginally more difficult, and thus should be perceived as a good thing (tm).
-- 
https://code.launchpad.net/~jamesodhunt/upstart/remove-initctl-job+instance-options/+merge/146272
Your team Upstart Reviewers is requested to review the proposed merge of lp:~jamesodhunt/upstart/remove-initctl-job+instance-options into lp:upstart.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: review-diff.txt
Type: text/x-diff
Size: 12185 bytes
Desc: not available
URL: <https://lists.ubuntu.com/archives/upstart-devel/attachments/20130202/b5df9334/attachment.diff>


More information about the upstart-devel mailing list