[Merge] lp:~stgraber/upstart/upstart-inherit-env-no-override into lp:upstart

[Stéphane Graber]

Stéphane Graber has proposed merging lp:~stgraber/upstart/upstart-inherit-env-no-override into lp:upstart.

Requested reviews:
  Upstart Reviewers (upstart-reviewers)

For more details, see:
https://code.launchpad.net/~stgraber/upstart/upstart-inherit-env-no-override/+merge/148362

When preparing the test PPA for user session, I noticed that I had a problem
with the new environment inheriting feature.

In my test case, I had DBUS_SESSION_BUS_ADDRESS set in the parent environment,
so upstart with --inherit-env stored that value.
I then used initctl set-env to set it to another value, but this would never be
propagated to the new jobs.

The reason for this is because the inherited environment was appended after any
other environment variables and so would effectively override the "right"
variable as was defined through initctl.


This change simply changes the logic to always apply the inherited environment
first, then have any other environment variables override it.


NOTE: This change is already part of my current PPA packages.
-- 
https://code.launchpad.net/~stgraber/upstart/upstart-inherit-env-no-override/+merge/148362
Your team Upstart Reviewers is requested to review the proposed merge of lp:~stgraber/upstart/upstart-inherit-env-no-override into lp:upstart.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: review-diff.txt
Type: text/x-diff
Size: 720 bytes
Desc: not available
URL: <https://lists.ubuntu.com/archives/upstart-devel/attachments/20130214/88fa3a6b/attachment.diff>