-
Notifications
You must be signed in to change notification settings - Fork 27
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
Upgrade to grails 3.3.1 #27
Comments
Installing grails version 2.3.11 on Ubuntu 17.04 with Java 8 etc. or on Ubuntu 16.04 LTS does not work. Grails exists with return value 1 and a cleared terminal screen. Building with grails 3.3.1, which does run on Ubuntu 17.04, requires application.properties file to support that version of grails. |
Note to self: apparently the directory structure has to be refactored for Grails version 3.3.1. See also http://docs.grails.org/3.2.7/guide/upgrading.html Will try to do so. Hope core devs can help out by reviewing the PR. |
I will commit a branch today with a first attempt but see that it will need some serious review and improvements by a Grails developer (which I am not). Hopefully, somebody could bring it to a next stage this weekend. |
Please review this commit and improve on it PanderMusubi@ed54f07 @danielnaber, probably good to get this branch also in top-level repo so other people can work on it more easily. |
In short, for upgrade needs to support latest stable version of Grails on Ubuntu 18.04, Ubuntu LTS, Debian 9.4 and Debian LTS. Grails may be installed from upstream or from OS distribution repo. |
Trying Grails 3.3.1 with your |
@danielnaber My branch was an attempt to upgrade. I hoped you could fix where the branch gets stuck. Since I'm lacking the needed Grails upgrade skills, a bounty has been added via https://www.bountysource.com/issues/49810378-upgrade-to-grails-3-3-1 |
Is this still waiting for a resolution? |
Yes, please. @danielnaber, you are not working on this now? |
No, I'm not working on this. |
I'll take a look at this soon. I've been swamped. |
I've started working on it too, but had some compile issues and havent commited anything yet. I also looked at your branch @PanderMusubi and you did at least one critical mistake. |
That is exactly where my very limited knowledge of Grails prevented me to continue and when it was time to call in the experts. As far as the rules for the bounty, they go to the person which successfully solves this issue first. We cannot split it, as it is paid via BountySource. We do not want to disappoint one or both of you. So if you both want to work on it, perhaps better discuss this first. Of course, we want the best fix, without personal disappointment, but we also value an indepent review of the result. |
Upon reviewing the guide for upgrading from Grails 2.3.x, it appears that the plugins used by openthesaurus will need to upgraded first. There appears to be 8 distinct plugin references in the existing repository. Some of them appear to have already been ported to Grails 3, but several appear to be unmaintained, such as feeds and jquery. How do you want to handle these? The easiest solution might be to drop them, but I don't know what they actually do here. |
jQuery is used in the front-end, but I see no need to use it as a plugin. feeds creates the Atom feed at https://www.openthesaurus.de/feed I think. |
Sounds like they're important to keep then. I'll see if I can port these plugins. Since some are unmaintained, where should their source go? Hm. |
This is now done thanks to @kstuart. |
Thanks! I will try to install it next days and figure out how he can collect the bounty. |
Yes, I have been able to configure, build and deploy it with Grails 3.3.1 on Ubuntu 18.04. Thank you. @kstuart Let me know if you need help claiming the bounty. |
Hi @PanderMusubi, good to know it's working well. Yes, now the work is complete and openthesaurus has been running on the main site for a while it would be great to get the bounty released, you'll need to accept the claim on BountySource, Stichting OpenTaal Could you do the same? Thanks! |
Done |
Please upgrade application.properties to grails version 3.3.1, see also https://grails.org/download.html
The text was updated successfully, but these errors were encountered: