Skip to content
This repository was archived by the owner on Mar 24, 2026. It is now read-only.

cutelyst: Update to 1.5.0#2591

Merged
NateBrady23 merged 2 commits intoTechEmpower:masterfrom
cutelyst:update_cutelyst
Mar 14, 2017
Merged

cutelyst: Update to 1.5.0#2591
NateBrady23 merged 2 commits intoTechEmpower:masterfrom
cutelyst:update_cutelyst

Conversation

@dantti
Copy link
Copy Markdown
Contributor

@dantti dantti commented Mar 13, 2017

No description provided.

@NateBrady23
Copy link
Copy Markdown
Member

hey @dantti !

So this failed here, and it also failed on the preview run we just did (as did a few other C/C++ fw's)

@dantti
Copy link
Copy Markdown
Contributor Author

dantti commented Mar 13, 2017

Did the previous version ran fine on the preview or you tested only this one?

@NateBrady23
Copy link
Copy Markdown
Member

It was the previous version that failed. It's also failed on a full travis run recently here: https://travis-ci.org/TechEmpower/FrameworkBenchmarks/builds/209848247

@dantti
Copy link
Copy Markdown
Contributor Author

dantti commented Mar 14, 2017

@nbrady-techempower Fixed the issue, and the link you pointed out failed due out of memory to build cutelyst, since the build script uses the number of cores to build maybe it would be a good idea to fix CPU cores to 1 when running on Travis.

@NateBrady23
Copy link
Copy Markdown
Member

Hmm, been trying my best to not have any Travis specific code polluting the toolset. Will see if there's another way to resolve. Thanks for all your support!

@NateBrady23 NateBrady23 merged commit 323f174 into TechEmpower:master Mar 14, 2017
@dantti
Copy link
Copy Markdown
Contributor Author

dantti commented Mar 14, 2017

@nbrady-techempower I can also change the make line to not use all cores if this keeps happening, the ideal solution would be to have a PPA with cutelyst so it would only build the test app, but current Ubuntu LTS doesn't even has the minimum Qt required and the toolset is using even older LTS...

zloster pushed a commit to zloster/FrameworkBenchmarks that referenced this pull request Mar 21, 2017
* cutelyst: Update to 1.5.0

* Cutelyst: Fix update to 1.5.0 as using v tags results in a different output directory
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants