-
Notifications
You must be signed in to change notification settings - Fork 8
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
Core20 and Java 17 #34
Conversation
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.
LGTM!
BGI trigger didn't seem to work - at least no updates to builds since 2021-04-16-11:05:25. This COULD be due to an old version of Java used by the previous snacraft.yaml.
Using same method to grab version from git. May not be ideal, but more consistent and less taxing on build servers. Also removed `unzip` from build-packages.
Clean up snapcraft yaml and change build procedures
Using same method to grab version from git. May not be ideal, but more consistent and less taxing on build servers. Also removed `unzip` from build-packages.
Build version settings
I've got a "final" rev for the yaml. Local build is running here on my end. |
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.
Ship it!
@popey do you need anything further from me? |
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.
LGTM!
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.
LGTM!
Also created a launcher script since the command wasn't working due to formatting.
Fixes #29
Fixes #32