Skip to content
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

OpenJDK: Adding Java 11 into our package #176

Conversation

thopiekar
Copy link
Member

@thopiekar thopiekar commented Dec 12, 2020

Synology dropped any Java package from their software repository in DSM. It is unknown whether they will ever add Java at all.
As OpenHAB itself recommends specific version(s) of Java for installation, I'm adding this feature not only to get independent of Synology, but also to get back full control over our installation.

This work has not been tested. Some corrections might follow.

Signed-off-by: Thomas Karl Pietrowski [email protected] (github: thopiekar)


This change is Reviewable

Synology dropped any Java package from their software repository.
It is unknown whether they will ever add Java at all.
As OpenHAB itself recommends specific version(s) of Java for installation, I'm adding this feature not only to get independent from Synology, but also to get back full control over our installation.

This work has not been tested. Some corrections might follow.

Signed-off-by: Thomas Karl Pietrowski <[email protected]> (github: thopiekar)
@cniweb
Copy link
Member

cniweb commented Dec 14, 2020

Hi @thopiekar thank you, for your commit, but sign-off your commit, please:
https://github.com/openhab/openhab-syno-spk/pull/176/checks?check_run_id=1543277017

Copy link
Member

@cniweb cniweb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:lgtm:

Reviewed 2 of 2 files at r1.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @thopiekar)

@thopiekar
Copy link
Member Author

Is included everywhere. Is DCO picky? I don't see a mistake.
Signed-off-by: Thomas Karl Pietrowski <[email protected]> (github: thopiekar)

@thopiekar
Copy link
Member Author

Adding this is a bad idea actually. I only download the x86_64 version and don't care about other architectures.
Will be improved in the scope of #178 .
Oh, that's another addition: There is no Java on DSM 7.

@thopiekar thopiekar closed this Dec 14, 2020
@thopiekar thopiekar deleted the master-adding-openjdk-download-and-install-into-package branch March 20, 2022 16:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants