installer icon indicating copy to clipboard operation
installer copied to clipboard

Generated MSIs lack end-user license agreement

Open jmjaffe37 opened this issue 1 year ago • 10 comments

Not sure if this was a desired change, but we recently noticed that the generated MSIs (windows installers) will no-longer show the End-User License Agreement page (see example below of what the page looks like). I looked for a while, but I could not find the code change that lead to this

image

jmjaffe37 avatar Apr 29 '24 22:04 jmjaffe37

@jmjaffe37 I noticed this when testing, it looks like we need to add this back as part of the flow.

gdams avatar Apr 30 '24 08:04 gdams

Looks like this was implemented as a feature request in https://github.com/adoptium/installer/issues/162 (since agreeing to the GPL2 license is optional for software users)

jmjaffe37 avatar Apr 30 '24 19:04 jmjaffe37

I guess we need to provide an override capability.

karianna avatar Apr 30 '24 21:04 karianna

I guess we need to provide an override capability.

The override capability was already implemented by @douph1. It depends on the JVM:

  • For Openj9, the license-OpenJ9.en-us.rtf license is shown
  • For hotspot, the license-GPLv2+CE.en-us.rtf is used (and skipped since it is optional)
  • Otherwise it says that there is no license defined for other custom JVMs

jmjaffe37 avatar Apr 30 '24 21:04 jmjaffe37

I should be able to add in the capability for the user specify that they want to show the optional GPL license. Would that be preferable?

jmjaffe37 avatar Apr 30 '24 21:04 jmjaffe37

I think we need to alter it so it's not vm based but "vm and vendor" based. I know that other non Adoptium vendors also have EULA's (on a hotspot base).

karianna avatar Apr 30 '24 21:04 karianna

I think we need to alter it so it's not vm based but "vm and vendor" based. I know that other non Adoptium vendors also have EULA's (on a hotspot base).

Ok, that can be arranged

jmjaffe37 avatar Apr 30 '24 21:04 jmjaffe37

I think we need to alter it so it's not vm based but "vm and vendor" based. I know that other non Adoptium vendors also have EULA's (on a hotspot base).

Feature added in https://github.com/adoptium/installer/pull/851. Also added sample code and an explanation of how to implement custom end-user license behavior in the readme

jmjaffe37 avatar Apr 30 '24 23:04 jmjaffe37

The vendor_based license logic has been removed (as was decided recently), but the README now has documentation on license behavior. If similar logic is desired in the future, it will be quick and easy to implement

jmjaffe37 avatar May 01 '24 16:05 jmjaffe37

Thank you!

karianna avatar May 01 '24 20:05 karianna