-
Notifications
You must be signed in to change notification settings - Fork 0
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
Add Metabase to packaging script #37
Comments
We also need to decide here if we include the JRE with app or require a separate installation. |
bah, I didn't notice you took this one in the morning and I quickly handled it just now. My implementation is pretty dumb and I've spent close to no time on it, but in case you've not tackled it just yet, you can hold off. |
No problem, I haven't started on it yet, so I'll let you handle it. |
re this, I'm now looking into. After first glance it seems like bundling JRE is preferred. I actually quite easily managed to install the wrong JRE version, when I wasn't paying enough attention. Even when I followed the recommended links from Metabase page, the correct version was some clicks away. I see that that JRE is distributed under GNU GPL, this might be an issue for us, no? https://adoptium.net/docs/faq |
OK, I'll cut a follow-up ticket for the JRE part and start working on it |
(lets take the licensing topic to the |
Split out of #5 , which only handles PostgreSQL
The text was updated successfully, but these errors were encountered: