Cannot specify a special PATH to run node from

Question

Cannot specify a special PATH to run node from

Closed this issue 8 years ago · 8 comments

Hi,

thanks for this maven plugin, it's very useful to us.

Sadly, we have a small issue with it in our CI environment. In this environment we don't have node/npm installed, but we use the maven-frontend-plugin to install node/npm upon build.

Is it possible to add an option to add something to the PATH by redefining : org.jsweet.util.ProcessUtil.EXTRA_PATH ?

Because it seems to do exactly what I need : (https://github.com/cincheo/jsweet/blob/40cb6d3597ef57ed318cf36dd212c5497a493dec/candy-generator/src/main/java/org/jsweet/util/ProcessUtil.java#L74)

I can't modify the PATH outside of maven's run so that's not an option for me sadly.

I can provide the pull request if you accept such change.

Answer 1 · 2017-03-24T08:39:14.000Z

Hello, I think this is a really good idea.
Indeed we need a suitable solution for CI, especially when Node can't be installed on the build environment, which will IMO occur often.

I will take a look later today.

Which version of the jsweet-maven-plugin are you running?

Thanks for your suggestion!

Answer 2 · 2017-03-24T13:20:36.000Z

Hi, thanks

We are currently running the version 1.1.1 of the plugin

Answer 3 · 2017-03-24T13:37:13.000Z

Ok, you will probably have to migrate to 1.2.0-SNAPSHOT, which is not a bad thing because there were several fixes included in 1.2

Answer 4 · 2017-03-24T13:57:11.000Z

@lgrignon you mean 1.2.0 and not 1.2.0-SNAPSHOT right?
That raises the question of how to version an update of the maven plugin on a given version of JSweet...

Answer 5 · 2017-03-24T14:45:06.000Z

Sure, I have no problem migrating to 1.2.0 when it's out.

Answer 6 · 2017-03-24T14:56:45.000Z

@renaudpawlak As we discussed, it will be 1.2.1-SNAPSHOT

@onigoetz If you want to migrate, 1.2.0 is out for a long time, I gave you the wrong next version, it is:
1.2.1-SNAPSHOT
Please do not worry about the SNAPSHOT for this version, there are no development in progress except for the enhancement you proposed. The actual next version in preparation is 2.0.0 ;)

Answer 7 · 2017-03-25T16:40:36.000Z

@onigoetz Could you please try, I just deployed the jsweet-maven-plugin 1.2.2-SNAPSHOT
The extraPath option should do the trick:
https://github.com/lgrignon/jsweet-maven-plugin/tree/1.2.1

We should integrate it better in the 2.0.0 version.

Thanks again for your suggestion, please close if it solves your issue.

Answer 8 · 2017-03-27T16:09:02.000Z

Hello, it seems it works fine with the latest code.

Thanks for the quick response.