-
Notifications
You must be signed in to change notification settings - Fork 85
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
Windows support #21
Comments
Nice! I would go with msvc; think most of the other open source projects and libraries in this field (e.g. openrv; usd, opensubdiv...) support either msvc or both. |
Thanks, at the moment I'm modifying cmake config to make it work on windows with Visual Studio and msvc. I will publish changes to |
After a while, I think that it's better to leave the port to qualified teams with better understanding of CMake, as I'm used to bazel, meson and ninja alone. Here is what has to be done:
Not sure if it's exhaustive enough but should be helpful. |
Just wanted to poke this thread with PR #50 that is an in-progress PR getting past a lot of these issues. There's still plenty of work to go, but a lot of the cmake/dependency building is working. If you were still interested in contributing and found those to be your blocker, then now might be a good time to take a look. Cheers, |
Hey, thanks for taking care of it. Unfortunatelly I'm not able to continue using Qt on Windows from dev side, it is doing all sorts of weird things deeply in system's registey, that causes e.g. Qt creator to appear 12 times in "Open With" popup and even after reinstallation you have to dig it out in the registry... However I will somehow try to review tue changes instead and see if they're compatible with VFX Platform. |
Hey, I’ve tried several times to load and build the repo for windows and after failing with your tutorial, I’ve tried it with Jetbrains Clion Community and came up with this steps to load it (I’ve replaced identifiable files and users by “...”) : Presintalled Clion, Visual Studio, Qt and cloned repo xstudio and ffmpeg 1) Added these compiling options -DCMAKE_PREFIX_PATH=C:\Qt\5.15.2\msvc2019_64\lib\cmake 2) Modified .vcpkg-clion/vcpkg/buildtrees/versioning_/baselines/ ... /baseline.json cmake-build-debug/vcpkg/buildtrees/versioning_/baselines/ ... /baseline.json Change the version of opencolorio of 2.1.2 (nonexistent version) to 2.2.1 3) Modified .vcpkg-clion\vcpkg\buildtrees\versioning_\versions\openimageio\ ... \portfile.cmake cmake-build-debug\vcpkg\buildtrees\versioning_\versions\openimageio\ ... \portfile.cmake
4) Modified extern/quickpromise/CMakeLists.txt line 46 to 52
After these steps the project is loading just fine
Can someone help me? |
Hello, I noticed that you have experience with XStudio on Windows, and I'm currently facing an issue with it. I was hoping you could provide some insights or solutions. I tried the XStudio Windows build, but unfortunately, I encountered the following error: I followed the build guide provided in this document to build (https://github.com/mpkepic/xstudio/blob/windows/docs/build_guides/windows.md) XStudio on Windows. The build was successful, but I'm encountering an issue when using the application. I am unable to play any media in the XStudio player. Expected Behavior: Environment: Build Method: Followed the steps in this document. When I try to play media, the player does not function as expected. Impact: Request for Assistance: Have you come across this issue before, and do you know of any solutions that could help me resolve it? I would greatly appreciate any guidance or advice you can offer. Thank you in advance! |
@anandan06 |
Hi! I've done a significant overhaul on the Cmake/vcpkg integration and cmake install target support among other things. I recommend checking out the latest commit on the pull request. The issues referenced in here also point to plugin install issues which should now be resolved. Note it now supports Visual Studio 2022, comes with some useful presets to make configuring easier, and has a bunch of build reliability improvements. |
Hey ! Thank you for your reply ! Yes i'm currently testing a build on my side from your latest reply on #57 Thank you again for your work ! Edit: I can confirm that the build works well now ! 🚀 |
Hi @mpkepic ! I don't know if i am the only one but i can't read sequences of images |
Thank you, I can see the same regression; going to take a look! |
I don't have an answer yet, but it looks like we've got different behavior between drag and drop vs commandline; if you pass the folder in via position argument, the sequence will load. |
I've spun off that issue into #106; see there for updates on that issue. |
Thank you for your investigation ! |
@OlivierArgentieri I've closed out #106 with the latest push to the PR, feel free to check it out. |
Thank you @mpkepic ! I have removed my old D:/xstudio folder, repull the repo to be sure i'm not building with old stuff, also removed the Maybe i am doing something wrong, my frames are just .jpg format like this from I have also tried in cli and got the same behavior, am i doing something wrong ? |
Ah, that's a different issue entirely than what I resolved; sorry. Try adding a basename to the frame number, such as frame.0001.jpg Can you spin off a separate feature request for frame sequences with only number and extension? Thanks. |
Hey ! I can read sequences of images with the latest commit on windows branch and with this naming structure |
Hey @mpkepic 👋 I also notice that the drag and drop doesn't work with a folder of frames store on a UNC path I don't know if this it's a bug or a feature request. context:
But in cli it's totally fine. If you want i can create a specific issue for this, to stop flooding this big |
Hi, I've seen that there is Windows support mentioned in the README as part of the roadmap. I'd just started working on it and would like to know which compiler is preferred,
msvc
orclang-cl
?Could be nice to have
#xstudio
or similar on ASWF's slack for real-time discussions and support.P.S. While reading through CMake file, just noticed that xstudio hardcodes Intel's microachitecture which makes it impossible to compile on AMD if you don't remove
-march=nehalem
flag.The text was updated successfully, but these errors were encountered: