-
Notifications
You must be signed in to change notification settings - Fork 574
Plugin doesn't work on Windows - wrong path handling #68
Comments
remove targetPath tag |
@arghya88 your suggestion worked great for me |
If you want to copy your resources to a specific folder in the docker image you will need targetPath. In this case it is still a problem. |
What happens if you use backslashes in the targetPath? |
I'm running into this issue too on version 0.2.11. I've tried: Any helpers or work arounds would be great. |
I have not tested it yet, but it looks like someone has made a fix. It's waiting in a PR |
I've now tested oberthur's fork and it works for me on windows. The change is in #65 |
In case you can't wait for that PR, here is a workaround I came up with:
Hope this helps. |
@BrettFieber this worked for me, thanks |
This should be resolved by https://github.com/spotify/docker-maven-plugin/pull/121/commits |
I just tried this with 0.3.5 on Windows. It's still broken. |
Not that it helps for the spotify plugin, but one I've found that works on windows is https://github.com/rhuss/docker-maven-plugin Good luck. |
Reopening this based on reports that it doesn't work. We welcome PR's but are unlikely to fix this at Spotify, since nobody has a Windows setup to test it out with. |
I ran into this same issue, and have created a PR that works around the issue (its actually a bug in the Windows implementation of Paths.get) |
@javacody do you still have issues with 0.3.9-SNAPSHOT? If so can you tell us the output of running your maven commands with the |
I was facing the same issue. Upgraded to 0.3.9 version and no longer facing the issue. |
@sunieldalal thanks for the confirmation! |
Running the plugin on Windows (yes...) ends with following error (stacktrace enabled):
Plugin configuration used for that project is as follows:
The text was updated successfully, but these errors were encountered: