Skip to content

Fix #710. #800

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

Merged
merged 3 commits into from
Apr 9, 2025
Merged

Fix #710. #800

merged 3 commits into from
Apr 9, 2025

Conversation

blueberry
Copy link
Contributor

@blueberry blueberry commented Apr 8, 2025

This PR fixes #710 .

With the release of MacOS 15.4, and the corresponding default C++ compiler upgrade, what was previously a warning, now is a compiler error, so this fixes a show stopper on new macos machine builds. On older systems, it removes the warnings, I suppose :)

Build passes on all OSs defined in GitHub actions.

@karllessard
Copy link

Hey @saudet , any plan for releasing this fix soon? Thanks

@saudet
Copy link
Member

saudet commented Apr 11, 2025

@karllessard That only affects compilation, not runtime

@karllessard
Copy link

Yes but we were in the process of upgrading to TensorFlow 2.18 and this is blocking us right now, since I need to compile locally once for getting the ops generated.

@saudet
Copy link
Member

saudet commented Apr 12, 2025

We can easily use the snapshots:
http://bytedeco.org/builds/

@jkhoogland
Copy link

Hi Samuel,
it would be great if we could a new release for this out.
Thanks,
Jiri

@saudet
Copy link
Member

saudet commented Jun 8, 2025

Yeah, well, NVIDIA seems to think otherwise, see bytedeco/javacpp-presets#1620 (comment)

@blueberry
Copy link
Contributor Author

Is it technically possible to release 1.5.12 for everything except Pytorch and OpenCV, and then release these two (as 1.5.12, or 1.5.13) as soon as the issue is fixed?

@saudet
Copy link
Member

saudet commented Jun 8, 2025

Sure, anything is possible, but it's not something I will personally take the time to deal with. It's not like NVIDIA doesn't have enough money to help us out now do they. 😮‍💨

@blueberry
Copy link
Contributor Author

Sure, but they don't care about us. Maybe they'd even care about this issue if we could reach someone in the trenches and let them know that the issue exists, but I unfortunately don't have any contact...

@blueberry
Copy link
Contributor Author

As for the help to this project, I feel your frustration. But I do think that you're too hesitant of explicitly asking for financial support by companies. Yes, GitHub offers easy way to sponsor a project on a small scale, but companies especially don't care about something lke that unless someone pushes the people in seeing this as an issue. Maybe there should be an explicit plea for funding, with explicit reasoning, ways to contribute, budget needed, expected amounts, etc. I'm sure that wouldn't send tons of money towards your efforts with JavaCPP, but I'm pretty sure there would be some funds, and, who knows, maybe a decent support after some time. I understand that you may be frustrated enough with the cruel corporate world that you don't want to go that route, and that you've tried many times in the past with insufficient success, but times have changed in the past decade I believe. Anyway, currently I don't see any prominent call to action related to that on the JavaCPP website, so even people who could help, just don't think about that and don't even know that you need it.

@saudet
Copy link
Member

saudet commented Jun 15, 2025

It's all politics at this point, and I don't do politics. If the Java community had a need of JavaCPP, there would be demand for that kind of tool out there, but there isn't, not anymore, Oracle made sure of that. Anyway, I've been able to fix the builds in a satisfactory fashion, so if there are no other major issues, we should be able to make a release by the end of the month.

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.

Warning with "JavaCPP_createStringFromUTF16" method on Mac clang 15
4 participants