-
Notifications
You must be signed in to change notification settings - Fork 950
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
Fix broken distribution #13740
Fix broken distribution #13740
Conversation
In Grails 6 (and earlier), which used the legacy Do we want to continue generating these |
I do not believe we need the SHA's based on the following: |
The distribution builds great. When run, I am seeing
digging down further:
it is this line: grails-core/grails-shell/src/main/groovy/org/grails/cli/boot/GrailsDependencyVersions.groovy Line 51 in 7db9a62
The POM files are published on https://repo.grails.org/ui/repos/tree/PomView/libs-snapshots-local/org/grails/grails-bom/7.0.0-SNAPSHOT/grails-bom-7.0.0-20241014.150731-91.pom This is also causing these 2 test failures: https://github.com/search?q=repo%3Agrails%2Fgrails-core%20bom%3Apom&type=code |
@jamesfredley That's interesting, I manage to run the distribution built on this branch: mattias@mattias-xps:/mnt/c/Users/matti/Projects/github/grails-core/build/distributions$ ./grails-7.0.0-SNAPSHOT/bin/grails -version
| Grails Version: 7.0.0-SNAPSHOT
| JVM Version: 17.0.12
mattias@mattias-xps:/mnt/c/Users/matti/Projects/github/grails-core/build/distributions$ ./grails-7.0.0-SNAPSHOT/bin/grails
| Starting interactive mode...
| Enter a command name to run. Use TAB for completion:
grails> |
I am still digging into the cause here.
openjdk version "17.0.12" 2024-07-16 LTS |
@jamesfredley Does building and running a distribution from the |
I tested |
# Conflicts: # gradle.properties
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.
@matrei do you want to move Exception: java.lang.NoSuchMethodError thrown from the UncaughtExceptionHandler in thread "main"
over to a new issue and merge this one? If so I approve.
No description provided.