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

Switch to default-jre-headless on server for yui-compressor #251

Merged
merged 2 commits into from
May 15, 2024

Conversation

tom93
Copy link
Contributor

@tom93 tom93 commented Jan 5, 2024

Was previously openjdk-8-jdk.

  • yui-compressor only requires the JRE, not the full JDK

  • GUI libraries aren't required, headless version is enough

  • the JRE version doesn't really matter, use the default so it gets
    automatically updated

Also remove unrelated commented-out lines from JRE install script.

@coveralls
Copy link

Coverage Status

coverage: 37.131%. remained the same
when pulling 2be75ea on yui-switch-to-jre
into b26315e on master.

bagedevimo
bagedevimo previously approved these changes Jan 5, 2024
@tom93 tom93 force-pushed the yui-switch-to-jre branch from 1f72b98 to 927a99e Compare May 15, 2024 13:05
tom93 added 2 commits May 16, 2024 01:08
Was previously openjdk-8-jdk.

 - yui-compressor only requires the JRE, not the full JDK

 - GUI libraries aren't required, headless version is enough

 - the JRE version doesn't really matter, use the default so it gets
   automatically updated
The lines were copied from script/install/imagemagick.bash and are
irrelevant.
@tom93 tom93 force-pushed the yui-switch-to-jre branch from 927a99e to 4c89ac1 Compare May 15, 2024 13:09
@tom93 tom93 merged commit b29d5f2 into master May 15, 2024
6 checks passed
@tom93 tom93 deleted the yui-switch-to-jre branch May 15, 2024 13:13
tom93 added a commit that referenced this pull request May 15, 2024
Was previously openjdk-8-jdk.

 - yui-compressor only requires the JRE, not the full JDK

 - GUI libraries aren't required, headless version is enough

 - the JRE version doesn't really matter, use the default so it gets
   automatically updated
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.

4 participants