-
Notifications
You must be signed in to change notification settings - Fork 2
Write out argsFile #85
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
Conversation
|
Thanks for contributing this @koppor! You can publish locally by running with Alternatively, you can add |
|
Thank you for the hints @jjohannes. I think, I got that part working. Now, I am stuck at Is this JDK or which tool is executing that statement?
UPDATE - seems to be another issue |
|
@jjohannes Thank you for the hint - seems to work -> JabRef/jabref#14124 |
|
Error in the tests - only on Windows - maybe, someone has a quick idea how to fix |
|
The failure on Windows is: |
jjohannes
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the change is good as it is. As the existing tests show that this does not break any functionality, I don't think we need to add additional tests.
I was wondering if there should be other conditions for not using the argsFile file (like if the modulePathAsPath is short).
But I also like to avoid such complexity in the code if it is not necessary.
src/main/java/org/gradlex/javamodule/packaging/tasks/Jpackage.java
Outdated
Show resolved
Hide resolved
…java Co-authored-by: Jendrik Johannes <jendrik.johannes@gmail.com>
jjohannes
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The build is working now. I assume the weird message we got on Windows (see comment above) was caused by the broken fallback if an IOExcpetion occurs. (And the exception itself was caused by the flakiness in the test.)

Fixes #69
Please guide how to test
results in