Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

#60193 closed defect (fixed)

opencv should not be dependent on Java12

Reported by: WolfgangFahl (Wolfgang Fahl) Owned by: chrstphrchvz (Christopher Chavez)
Priority: Normal Milestone:
Component: ports Version:
Keywords: Cc: chrstphrchvz (Christopher Chavez)
Port: opencv

Description

The dependency makes opencv unusable in Java 8 environments. Alternatively provide a special port for Java8.

Change History (10)

comment:1 Changed 4 years ago by ryandesign (Ryan Carsten Schmidt)

opencv should only be using java 12 if you have requested the +java variant. If you don't want opencv to do that, don't request that variant.

comment:2 Changed 4 years ago by ryandesign (Ryan Carsten Schmidt)

Port: opencv added

comment:3 Changed 4 years ago by WolfgangFahl (Wolfgang Fahl)

I am talking about that variant I'd love to get a +java variant that does not depend on Java 12. E.g. either fix +java or create a +java8 so that Java 8 can still be used. I need to upgrade openCV 3.4.3 in an Java8 environment.

comment:4 Changed 4 years ago by WolfgangFahl (Wolfgang Fahl)

Is the java 12 dependency an upstream effector or introduced my macports?

comment:5 Changed 4 years ago by chrstphrchvz (Christopher Chavez)

Cc: chrstphrchvz added

comment:6 Changed 4 years ago by chrstphrchvz (Christopher Chavez)

I believe the choice of Java 12 was arbitrary to ensure a recent Java is installed; OpenCV 3.4 appears to still have support for older versions, so I propose lowering it to 1.6+ (or to no higher than 1.8+). I would prefer an LTS Java version be specified as the fallback so that the fallback doesn't have to be adjusted as often.

I have opened a PR to address this: https://github.com/macports/macports-ports/pull/6922

comment:7 Changed 4 years ago by WolfgangFahl (Wolfgang Fahl)

@Christopher Chavez Thank you so much - this would be just great!

comment:8 Changed 4 years ago by chrstphrchvz (Christopher Chavez)

Owner: set to chrstphrchvz
Resolution: fixed
Status: newclosed

In 720fa78ffc9d8944ebe4e3770ef2c92a3f928047/macports-ports (master):

opencv: lower required Java version for java variant (https://github.com/macports/macports-ports/pull/6922)

Fixes: #60193

Use latest LTS Java as fallback

comment:9 Changed 4 years ago by chrstphrchvz (Christopher Chavez)

I'm afraid this issue was reintroduced during the update to 3.4.10: [d98d7d7369/macports-ports]

comment:10 Changed 4 years ago by chrstphrchvz (Christopher Chavez)

In 19d5158851d512d5c7ff00d69c1577bb040272d7/macports-ports (master):

opencv +java: lower required Java version (again)

Unintentionally raised during update to 3.4.10 (d98d7d7369)
Use latest LTS Java as fallback
See: #60193

Note: See TracTickets for help on using tickets.