Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

fix(android): Kotlin 2.0 internal error for unsafe coercions#26354

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged

Conversation

@onuralpszr
Copy link
Contributor

@onuralpszronuralpszr commentedOct 22, 2024
edited
Loading

Pull Request Readiness Checklist

See details athttps://github.com/opencv/opencv/wiki/How_to_contribute#making-a-good-pull-request

  • I agree to contribute to the project under Apache 2 License.
  • To the best of my knowledge, the proposed patch is not based on a code under GPL or another license that is incompatible with OpenCV
  • The PR is proposed to the proper branch
  • There is a reference to the original bug report and related work
  • There is accuracy test, performance test and test data in opencv_extra repository, if applicable
    Patch to opencv_extra has the same branch name.
  • The feature is well documented and sample code can be built with the project CMake

This PR related to issue#26129 to attempting to fix.

I would like to say that this issue not happen when we use maven package with same configuration stated in the issue with same JDK 17 and SDK 34 it just works. This is only hppens when we download opencv android from website use as a module and add into gradle

kmy-n, Razorquake, Junacik99, dmitriy-chernysh, blujay77, and KodingWithKram reacted with heart emoji
@onuralpszr
Copy link
ContributorAuthor

@asmorkalov 👋 Hello, Is android test CI stuck ? It is started 8h ago and still in queue.

@Razorquake
Copy link

I hope the issue with Kotlin 2.0 will be resolved soon

Signed-off-by: Onuralp SEZER <thunderbirdtr@gmail.com>
@onuralpszronuralpszrforce-pushed thefix/kotlin-internal-error branch fromb4e2669 toce13988CompareMarch 22, 2025 15:34
@onuralpszr
Copy link
ContributorAuthor

@asmorkalov can you re-trigger android CI please.

@fzhinkin
Copy link

Seems like it should be enough to patch only a few places in code to workaround the compiler bug:https://gist.github.com/fzhinkin/e0a5dc7dd2df933408c0bcdb2be9ecd5

@Neightro
Copy link

I've been using this branch successfully with Kotlin 2. The only issue is that it triggers the linter warning "This declaration needs opt-in. Its usage should be marked with '@kotlin.ExperimentalUnsignedTypes' or '@OptIn(kotlin.ExperimentalUnsignedTypes::class)' for the following positions:

6:39, 6:73, 6:78, 7:38, 7:71, 7:76, 8:39, 8:73, 8:78, 9:38, 9:71, 9:76, 11:39, 11:74, 11:79, 12:38, 12:72, 12:77, 13:39, 13:74, 13:79, 14:38, 14:72, 14:77.

asmorkalov and Dev-Czle reacted with thumbs up emoji

@cshrpy
Copy link

What's holding up the progress?

@asmorkalovasmorkalov self-requested a reviewJune 26, 2025 07:23
@asmorkalovasmorkalov self-assigned thisJun 26, 2025
@asmorkalovasmorkalov added this to the4.12.0 milestoneJun 26, 2025
@asmorkalovasmorkalov merged commitd2d90dd intoopencv:4.xJun 26, 2025
51 of 55 checks passed
This was referencedJul 17, 2025
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@asmorkalovasmorkalovasmorkalov approved these changes

Assignees

@asmorkalovasmorkalov

Projects

None yet

Milestone

4.12.0

Development

Successfully merging this pull request may close these issues.

6 participants

@onuralpszr@Razorquake@fzhinkin@Neightro@cshrpy@asmorkalov

[8]ページ先頭

©2009-2025 Movatter.jp