3.2.0
ClosedPublic
Actions

Authored by admin_hackacad.net on Sep 5 2025, 7:14 AM.

Details

Reviewers

admin_hackacad.net

Summary

textproc/opensearch: Update to 3.2.0

Changelogs:

https://github.com/opensearch-project/OpenSearch/releases/tag/3.2.0

Add opensearch_flags for temporary startup parameters
Add opensearch_env for setting environment variables
Add proper check for aarch64
Allow only amd64 and aarch64
Remove AVX2 check

Diff Detail

Repository

R11 FreeBSD ports repository

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

admin_hackacad.net requested review of this revision.Sep 5 2025, 7:14 AM

admin_hackacad.net created this revision.

Bump OpenJDK to upstream supported version 24

Is this the minimum upstream supported version of Java?

approved by opensearch@

This revision is now accepted and ready to land.Thu, Sep 18, 7:35 AM

In D52392#1201249, @vvd wrote:

Bump OpenJDK to upstream supported version 24

Is this the minimum upstream supported version of Java?

Supported JAVA is 11+ (https://docs.opensearch.org/latest/install-and-configure/install-opensearch/index/)
The bundled version is 24.
I decided to bump Java to the upstream 24(+) as there are major improvements on machines with large amount of memory.
https://reviews.freebsd.org/D52354

Version 21 is LTS, version 24 is not.
The recently released version 25 is also LTS, but it's not yet available in ports.
I wouldn't specify a non-LTS version unless absolutely necessary.

The bad thing is that the ports framework does not support version prioritization, but only a set that will be sorted and the minimum possible version will be selected for binary packages.

In D52392#1201468, @vvd wrote:

The bad thing is that the ports framework does not support version prioritization, but only a set that will be sorted and the minimum possible version will be selected for binary packages.

LTS is a very good point.

I'd prefere:
SUPPORTED_JAVA=11≤x≤24
DEFAULT_JAVA=24

rollback
JAVA_VERSION=21

This revision now requires review to proceed.Fri, Sep 19, 7:10 AM

I think we need something like this:
USES=java:min=11,max=24,recommended=21
min and max same as for USES=llvm, but recommended not implemented anywhere.
Instead of current:
USES=java
JAVA_VERSION=21+
But someone needs to implement this.

Approved by opensearch@

This revision is now accepted and ready to land.Mon, Sep 29, 2:19 PM

Where is OPENSEARCH_ARCH variable used?

Suggestion:
OPENSEARCH_ARCH= ${ARCH:S|aarch64|arm64|:S|amd64|x64|}

admin_hackacad.net updated this revision to Diff 163066.Tue, Sep 30, 7:22 AM

This revision now requires review to proceed.Tue, Sep 30, 7:22 AM

Approved by opensearch@

In D52392#1205895, @vvd wrote:

Where is OPENSEARCH_ARCH variable used?

Suggestion:
OPENSEARCH_ARCH= ${ARCH:S|aarch64|arm64|:S|amd64|x64|}

Fixed this. Arch is used for downloading the corresponding tar ball.

This revision is now accepted and ready to land.Tue, Sep 30, 7:24 AM

In D52392#1206146, @admin_hackacad.net wrote:

Approved by opensearch@

In D52392#1205895, @vvd wrote:

Where is OPENSEARCH_ARCH variable used?

Suggestion:
OPENSEARCH_ARCH= ${ARCH:S|aarch64|arm64|:S|amd64|x64|}

Fixed this. Arch is used for downloading the corresponding tar ball.