native m2 project environment variables expansion

classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|

native m2 project environment variables expansion

Aleksandar Kostadinov
Hallo,

is anybody seeing that working?

MavenProcessFactory.java:

     public String getMavenOpts() {
         String opts = mms.getMavenOpts();
         if (opts == null)
             return null;

         for (String key : envVars.keySet())
             opts = opts.replace("${" + key + "}", envVars.get(key));

         return opts;
     }


I don't see any options to be expanded using a test maven project. First
it doesn't use the Util.replaceMacro method and then it doesn't do
anything for some reason.

Can I just wipe that out to use the Util method and use variables of the
slave and not master?

Thanks,
Aleksandar

---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

Re: native m2 project environment variables expansion

Kohsuke Kawaguchi
Administrator
Aleksandar Kostadinov wrote:

> Hallo,
>
> is anybody seeing that working?
>
> MavenProcessFactory.java:
>
>      public String getMavenOpts() {
>          String opts = mms.getMavenOpts();
>          if (opts == null)
>              return null;
>
>          for (String key : envVars.keySet())
>              opts = opts.replace("${" + key + "}", envVars.get(key));
>
>          return opts;
>      }
>
>
> I don't see any options to be expanded using a test maven project. First
> it doesn't use the Util.replaceMacro method and then it doesn't do
> anything for some reason.
>
> Can I just wipe that out to use the Util method and use variables of the
> slave and not master?
Sounds like a good idea.

--
Kohsuke Kawaguchi
Sun Microsystems                   [hidden email]

smime.p7s (4K) Download Attachment