Skip to content
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

Changes to support depthai in windows #1430

Closed
wants to merge 9 commits into from
Closed

Conversation

mksbabu
Copy link

@mksbabu mksbabu commented Oct 30, 2023

No description provided.

@mksbabu
Copy link
Author

mksbabu commented Oct 30, 2023

Hi @saudet

created pull request, could you please look into it, let me know please if any changes are required to support depthai in windows

pull request: #1430

Thanks.

@saudet
Copy link
Member

saudet commented Oct 30, 2023

You'll need to update the cppbuild.sh file as well:
https://github.com/bytedeco/javacpp-presets#build-instructions

@mksbabu
Copy link
Author

mksbabu commented Oct 30, 2023

@saudet updated version cppbuild.sh file. Is there anything need to change in that file since I am not aware on this project?

@saudet
Copy link
Member

saudet commented Oct 30, 2023

We'll need to do more than that: https://github.com/bytedeco/javacpp-presets/wiki/Create-New-Presets

@mksbabu
Copy link
Author

mksbabu commented Oct 30, 2023

Hi @saudet

I am facing few build issues with this project when I imported in my local, not able to proceed further. If you don't mind, could you please make depthai to work in windows. I am thankful for your help.

Thanks.

@mksbabu
Copy link
Author

mksbabu commented Oct 30, 2023

It will be great help for our project if DepthAI to work with windows. Please consider my request.

Thanks,
Mks.

@saudet
Copy link
Member

saudet commented Oct 31, 2023

Like I keep telling you, contributions are welcome! I'm happy to help you get this working, but I don't have the time to do everything for everyone for free. If you're facing any problems, please let me know

@mksbabu
Copy link
Author

mksbabu commented Oct 31, 2023

Thanks for help!!
I have done few changes in cppbuild.sh to support for windows, Let me know please what else need to add to that file

@saudet
Copy link
Member

saudet commented Oct 31, 2023

You probably want to build for windows-x86_64, not windows-x86

@mksbabu
Copy link
Author

mksbabu commented Oct 31, 2023

I have added both windows-x86_64 and windows-x86 like all other projects(https://github.com/mksbabu/javacpp-presets-1/blob/master/depthai/cppbuild.sh), Let me know please if there is anything need to change

@saudet
Copy link
Member

saudet commented Oct 31, 2023

2023-10-31T09:51:06.1048629Z [ERROR] [ERROR] Could not find the selected project in the reactor: depthai @
2023-10-31T09:51:06.1049914Z [ERROR] Could not find the selected project in the reactor: depthai -> [Help 1]

That means we'll need to add depthai to the list here:
https://github.com/bytedeco/javacpp-presets/blob/master/pom.xml#L1647

@mksbabu
Copy link
Author

mksbabu commented Oct 31, 2023

@saudet I have updated pom.xml as per above comment from you, facing below error could you please let me know what's the resolution for below error
Error: Failed to execute goal org.bytedeco:javacpp:1.5.9:build (javacpp-cppbuild-install) on project depthai: Execution javacpp-cppbuild-install of goal org.bytedeco:javacpp:1.5.9:build failed: Process exited with an error: 1 -> [Help 1]
org.apache.maven.lifecycle.LifecycleExecutionException: Failed to execute goal org.bytedeco:javacpp:1.5.9:build (javacpp-cppbuild-install) on project depthai: Execution javacpp-cppbuild-install of goal org.bytedeco:javacpp:1.5.9:build failed: Process exited with an error: 1

@mksbabu
Copy link
Author

mksbabu commented Nov 1, 2023

@saudet could you please help me to fix this pull request

@saudet
Copy link
Member

saudet commented Nov 1, 2023

The CMake build is failing on Windows, so you'll need to follow the instructions to build DepthAI on Windows, but I'm unable to find them. @themarpe Where can we find the build instructions for Windows?

@mksbabu
Copy link
Author

mksbabu commented Nov 1, 2023

@themarpe could you please help us on it

@mksbabu
Copy link
Author

mksbabu commented Nov 1, 2023

@saudet Is there any option to fix this issue. could you please help us on it.

@saudet
Copy link
Member

saudet commented Nov 2, 2023

Like I said before, I don't think DepthAI supports Windows, even though @themarpe says otherwise.

@saudet
Copy link
Member

saudet commented Nov 2, 2023

Have you considered using WSL2? It's usually possible to run any Linux applications on Windows that way.

@mksbabu
Copy link
Author

mksbabu commented Nov 2, 2023

I am not aware of it. could you please suggest what to do?

@saudet saudet removed the help wanted label Dec 2, 2023
@saudet
Copy link
Member

saudet commented Dec 2, 2023

Duplicate of #1441

@saudet saudet marked this as a duplicate of #1441 Dec 2, 2023
@saudet saudet closed this Dec 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants