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

Port to vs2015 #307

Closed
wants to merge 10 commits into from
Closed

Conversation

JosefMeixner
Copy link
Contributor

I have ported opentoonz to VS 2015. It will probably not make sense at this time to move to it, but there are two commits, the first are only source changes which should be portable and the second one are all the binaries. I only built the 64bit variants of the files (release and debug).

The source changes are rather small, I had to remove cprintf in openblas, VS 2015 for some reason didn't link with it defined, so now printf is used instead. The other change was in toonz/sources/toonz/cleanuppopup.cpp, Microsoft for some reason decided to add a second set of count functions into containers so bind didn't resolve it. I substituted it with a small lambda.

Building QT is rather straight forward, only the OpenGL-setting has to be changed to desktop, I had no luck with the default setting of dynamic, it caused collisions between GLEW and QtGL. And you need ICU (forget the web pages saying differently), UIC doesn't compile without.

…ded a second function std::map::count and bind couldn't resolve the function. Substituted with a lambda-expression.
…ot finished, panel is wrong now (also uses CleanupSettings).
@janisozaur
Copy link
Contributor

How about #69 ?

@JosefMeixner
Copy link
Contributor Author

I am absolutely for it, it takes ages to upload the built binaries and having them everywhere is terrible. But I don't think I am able to do this currently. I would prefer to wait until it is fixed here and then also fix my branch. I will comment in #69.

…n resolutions. Used the same value (PERCENT * max dot size) as later in the code in compare_dots.
… fit and used that. The problem was, that the middle hole was used as center for auto center. For three peg holes that is correct, but does not work for more than three. Now the best fit is calculated and from there the offset of the middle of the three best fit holes is calculated and applied.
@ghost
Copy link

ghost commented Jun 2, 2016

Josef-
This is somewhat off topic- but do you know how to get a debug setup working for OpenToonz in VS? I have to build a release and try and crash it. I can't seem to get the debugger set up right in VS for OpenToonz.

@JosefMeixner
Copy link
Contributor Author

I use the target RelWithDebInfo, that is a release target with debugging info. I always start opentoonz from VS and have no problem this way to use it or debug it.

@skitaoka
Copy link
Member

skitaoka commented Jul 4, 2016

This PR was merged by #544.

@skitaoka skitaoka closed this Jul 4, 2016
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.

None yet

3 participants