-
Notifications
You must be signed in to change notification settings - Fork 102
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
propose to fix #266 by moving installed jar #438
Conversation
If it works after you do clean and build. Restart IDE and do it again. Then... it is maybe fine? We can also do a test release at some point to see if it worked or not. Some small fixes already accumulated after last release? |
It didn't. Needs investigation... I'll keep trying though. Something made it work. |
Yeah, for me it was like that too. Works when you do it. But.... somehow not permanently. |
If it bleeds, we can kill it. |
So it actually did work, I had just misspelled the path in build.gradle. I don't find it optimal to change it, but I just can't make it work with the "release/libs/" path. |
I did a little bit of further testing with the release/libs path, to no avail. I don't understand why this works but not the other. But unless there are objections, I'll probably merge this some time during the weekend. |
If it works, I'm good with it. I suspect anyway if we get the Maven done. This is obsolete. Maybe. |
Oh, did you by the way test the test project? Doesn't it use the test data as well on some of the tests? |
Which project would that be? |
Ah, the JmeTests project. No I didn't, and of course it didn't work when I did. |
fix here: #440 |
This is essentially #417
I don't know enough about the installation process to know if it actually fixes the problem, but I noticed that if I used #417 and then moved the lib to the folder it actually points to, it worked.
Fixes #266