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

[CI] Test - add tests in windows #6413

Closed
wants to merge 6 commits into from
Closed

Conversation

ddiezrod
Copy link
Contributor

No description provided.

@ddiezrod ddiezrod closed this Feb 13, 2020
@ddiezrod ddiezrod reopened this Feb 13, 2020
@philbucher
Copy link
Member

wow now it really slow 😅
@roigcarlo the issue with the cotire you mentioned in #6412 cannot be resolved or requires some effort?

@roigcarlo
Copy link
Member

No we cannot install with cotire, unless you manually navigate through all the folders an collet the DLL, python files and build the dirs manually.

But try to force the /O0 flag that may help with the compilation speed

@philbucher
Copy link
Member

hm is it already building for ~2,5hrs 😅

@philbucher
Copy link
Member

But try to force the /O0 flag that may help with the compilation speed

@oberbichler do you have an idea how to do this?

Trying Custom build config with enforced /Od
@roigcarlo
Copy link
Member

No space left on device.

Ouch.. that's new :_D

@philbucher
Copy link
Member

No space left on device.

Ouch.. that's new :_D

:/

1,5 hrs is anyway too long I would say...

@roigcarlo
Copy link
Member

Then no more ideas from my side sorry. We could problably install it manually but it will take some time

@roigcarlo
Copy link
Member

@philbucher We can do this: https://github.com/sakra/cotire/blob/master/MANUAL.md#installing-files-generated-by-unity-targets

But for tat we need to change all CMakes of all applications...

@philbucher
Copy link
Member

The modification is rather small right?
We could try with the core I guess
Don’t think we can get rid of Cotire after seeing the build times without

@ddiezrod ddiezrod closed this Mar 6, 2020
@ddiezrod ddiezrod deleted the ci/add-tests-in-windows branch March 6, 2020 12:46
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.

3 participants