Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

JPro Release Maven Powershell issue on windows. #184

Closed
nyzzik opened this issue Aug 28, 2024 · 5 comments
Closed

JPro Release Maven Powershell issue on windows. #184

nyzzik opened this issue Aug 28, 2024 · 5 comments

Comments

@nyzzik
Copy link

nyzzik commented Aug 28, 2024

Hello, I am utilizing the jpro plugin for maven and it seems that whenever I create a release, the powershell script that is created and place into the bin folder uses backslash \ in order to escape the quotes " however in powershell, at least for windows, the backtick ` is used to escape quotes. So in order to allow this powershell script to run on windows, i need to replace all the backslashes with backticks

Another question, unrelated to the previous concern, would it be possible to allow for the jpro release to output a folder with the folders inside of it rather than the zip? I am creating an installer utilizing the jpro release however i do not need it zipped, so for my current use case i have to run mvn jpro:release then unzip the release and then load it into my installer.

@besidev
Copy link

besidev commented Aug 29, 2024

Hello @nyzzik.

We have just released JPro version 2024.3.2 containing some bug fixes regarding paths handling in general in the plugins, especially on Windows. Please give it a try and let us know if the issue still persists.

Regarding the other topic we can consider adding an option to not produce a release zip file when jpro:release is called, but just the folder containing all the stuff inside it. In the meantime easily create a script to unzip it automatically.

@nyzzik
Copy link
Author

nyzzik commented Sep 10, 2024

After upgrading to the version 2024.3.2 the start.ps1 contained in the bin folder after running jpro:release is still using \ to escape quotes rather than `

Regarding the second topic, thats what I am currently doing, it just seems unnecessary hence asking for the feature.

@besidev
Copy link

besidev commented Oct 24, 2024

Hi @nyzzik Josh,

Can you please check the new snapshot build we have just released, JPro version 2024.3.4-SNAPSHOT. We have reviewed and updated the starting script on Windows to use single-quotes and some other improvements. Please give it try and let us know if it works. It will be great to know that we now fixed this issue before publishing the next release.

Thank you,
Besmir

@besidev
Copy link

besidev commented Nov 5, 2024

We just released JPro version 2024.4.0. Give it a try and let us know if the issue is now fixed.

@besidev
Copy link

besidev commented Nov 11, 2024

I'm closing this ticket. I'f the issue still persist, you can re-open it or just create a new one.

@besidev besidev closed this as completed Nov 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants