Re: Path environment variable not set by SimpleBuildWrapper.Context.env() on Windows slave

2017-07-10 Thread 15 knots
2017-07-09 15:40 GMT+02:00 'Nikolas Falco' via Jenkins Developers : > @15knots this issue is not caused by the envinject plugin (I had try this > when I got the issue). Sorry, I have to disagree here. If I uninstall or disable the envinject plugin, the issue is

Re: Path environment variable not set by SimpleBuildWrapper.Context.env() on Windows slave

2017-07-09 Thread 'Nikolas Falco' via Jenkins Developers
@15knots this issue is not caused by the envinject plugin (I had try this when I got the issue). As you can see in the JIRA bug the issue is a Jenkins Core class (Run.java) and in the sequence that it use to constructs a new EnvVars for the current node. In the chain of enviroments pouring

Re: Path environment variable not set by SimpleBuildWrapper.Context.env() on Windows slave

2017-07-07 Thread Oleg Nenashev
EnvInject Bug for it: https://issues.jenkins-ci.org/browse/JENKINS-26583 I have a plan to fix it, but right now it is not a top priority for me. The plugin is looking for adopters четверг, 6 июля 2017 г., 18:30:07 UTC+2 пользователь 15 knots написал: > > 2017-07-06 13:45 GMT+02:00 15 knots

Re: Path environment variable not set by SimpleBuildWrapper.Context.env() on Windows slave

2017-07-06 Thread 15 knots
2017-07-06 13:45 GMT+02:00 15 knots : > 2017-07-06 0:27 GMT+02:00 Jesse Glick : >> On Wed, Jul 5, 2017 at 3:43 PM, Martin Weber >> wrote: > ... >>> Maybe one of the plugins installed on my production Jenkins interferes

Re: Path environment variable not set by SimpleBuildWrapper.Context.env() on Windows slave

2017-07-06 Thread Jesse Glick
On Thu, Jul 6, 2017 at 7:45 AM, 15 knots wrote: > Found the culprit: Environment Injector Plugin > Should I file a bug report? Ah, yes, I suppose so. One of the unfortunate things about this plugin (at least in my memory) is its tendency to cause behavioral changes

Re: Path environment variable not set by SimpleBuildWrapper.Context.env() on Windows slave

2017-07-06 Thread 15 knots
2017-07-06 0:29 GMT+02:00 Daniel Beck : > >> On 6. Jul 2017, at 00:27, Jesse Glick wrote: >> >> Uh, really? Is this reproducible in a minimal self-contained test >> case? > > FWIW I remember a really old bug report about that, maybe this is "known" >

Re: Path environment variable not set by SimpleBuildWrapper.Context.env() on Windows slave

2017-07-06 Thread 15 knots
2017-07-06 0:27 GMT+02:00 Jesse Glick : > On Wed, Jul 5, 2017 at 3:43 PM, Martin Weber > wrote: ... >> Maybe one of the plugins installed on my production Jenkins interferes here? > > Conceivable. Found the culprit: Environment Injector Plugin

Re: Path environment variable not set by SimpleBuildWrapper.Context.env() on Windows slave

2017-07-05 Thread Daniel Beck
> On 6. Jul 2017, at 00:27, Jesse Glick wrote: > > Uh, really? Is this reproducible in a minimal self-contained test > case? FWIW I remember a really old bug report about that, maybe this is "known" already. -- You received this message because you are subscribed to

Re: Path environment variable not set by SimpleBuildWrapper.Context.env() on Windows slave

2017-07-05 Thread Jesse Glick
On Wed, Jul 5, 2017 at 3:43 PM, Martin Weber wrote: > I tried PATH+SOMETHING` first, but that put in the wrong path-separator > (':' where ';' is needed, when master is on Linux). The EnvVars object in this > case has platform==null, so EnvVars.override() gets it wrong.

Re: Path environment variable not set by SimpleBuildWrapper.Context.env() on Windows slave

2017-07-05 Thread Martin Weber
Am Mittwoch, 5. Juli 2017, 15:19:37 CEST schrieb Jesse Glick: > On Wed, Jul 5, 2017 at 1:43 PM, Martin Weber wrote: > > in my SimpleBuildWrapper.setup(), I want to add an extra directory to > > the PATH-variable by SimpleBuildWrapper.Context.env("Path",). > > I think

Re: Path environment variable not set by SimpleBuildWrapper.Context.env() on Windows slave

2017-07-05 Thread Jesse Glick
On Wed, Jul 5, 2017 at 1:43 PM, Martin Weber wrote: > in my SimpleBuildWrapper.setup(), I want to add an extra directory to > the PATH-variable by SimpleBuildWrapper.Context.env("Path",). I think you meant to use `PATH+SOMETHING` as per the Javadoc. -- You received

Path environment variable not set by SimpleBuildWrapper.Context.env() on Windows slave

2017-07-05 Thread Martin Weber
Hi all, in my SimpleBuildWrapper.setup(), I want to add an extra directory to the PATH-variable by SimpleBuildWrapper.Context.env("Path",). The job is configured to run on a windows slave. Manipulating the PATH works well with Jenkins instances launched by mvn hpi:run on both Linux and windows.