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

Use java.io.File to concatenate path components #104

Closed
rslemos opened this issue Apr 20, 2021 · 2 comments
Closed

Use java.io.File to concatenate path components #104

rslemos opened this issue Apr 20, 2021 · 2 comments
Assignees

Comments

@rslemos
Copy link

rslemos commented Apr 20, 2021

When concatenating path components, care should be taken to introduce path separators ("/") when needed.

In:

String serverConfigDir = getWlpUsrDir() + "servers/" + containerConfiguration.getServerName();

it should be noted that getWlpUsrDir() must return a directory with / appended to it.

Otherwise it may error out with:

Caused by: java.io.FileNotFoundException: C:\dev-env\wlp-usrservers\myserver\server.xml (The system could not find the specified path.)
@cherylking cherylking self-assigned this Apr 23, 2021
@cherylking
Copy link
Member

This was actually fixed in this PR last year: #70

We just have not released version 1.0.7 yet with the fix. I will work on that and post here when it is available.

@cherylking
Copy link
Member

Release 1.0.7 is available with the fix for this issue: https://github.com/OpenLiberty/liberty-arquillian/releases/tag/arquillian-parent-liberty-1.0.7

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