Page MenuHomeFreeBSD

textproc/opensearch update 3.1.0 -> 3.2.0
ClosedPublic

Authored by admin_hackacad.net on Sep 5 2025, 7:14 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Oct 7, 12:21 PM
Unknown Object (File)
Mon, Oct 6, 6:06 PM
Unknown Object (File)
Mon, Sep 29, 1:43 PM
Unknown Object (File)
Thu, Sep 25, 9:31 PM
Unknown Object (File)
Wed, Sep 24, 2:03 AM
Unknown Object (File)
Wed, Sep 24, 1:09 AM
Unknown Object (File)
Tue, Sep 23, 6:29 PM
Unknown Object (File)
Fri, Sep 19, 7:07 AM
Subscribers

Details

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

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

admin_hackacad.net created this revision.

Bump OpenJDK to upstream supported version 24

Is this the minimum upstream supported version of Java?

admin_hackacad.net edited reviewers, added: admin_hackacad.net; removed: otis.

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

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.

admin_hackacad.net updated this revision to Diff 163020.
admin_hackacad.net edited the summary of this revision. (Show Details)

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|}

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

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.

Then distinfo should also contain opensearch-3.2.0-linux-arm64.tar.gz.

Also, why wouldn't you want to use the "short" version of the OPENSEARCH_ARCH definition?