-
-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
Adapt localization.gradle to python 3 #3415
Conversation
I just checkout this branch, ran
So this seems to not to be working yet. Oh and here is the exception:
|
Well, do you have python3 installed?I know, the solution before with jython was more elegant.The problem is: it doesn't support python3, meaning that it must be pre installed on the user's machine. If this is an infeasible option, I would have to Port it back to a jython-supported version.
|
Ok, I wasn't aware of that. In that case, I am strongly for reverting the changes and going back to the old localization script. Forcing people to install additional dependencies defeats the whole point of using a build system. Everything should be doable with only Java installed so that you can run gradle. This would also make contributions for new users so much harder. @JabRef/developers We need a decision on this. |
Check out #3420 in favor of this. |
Fixes #3414