Skip to content

Improve Qml error reporting, update doc after #85#86

Merged
rdumusc merged 1 commit intoBlueBrain:masterfrom
rdumusc:master
Sep 19, 2016
Merged

Improve Qml error reporting, update doc after #85#86
rdumusc merged 1 commit intoBlueBrain:masterfrom
rdumusc:master

Conversation

@rdumusc
Copy link

@rdumusc rdumusc commented Sep 19, 2016

No description provided.

doc/Building.md Outdated
* Boost 1.54 or later
* MPI with MPI_THREAD_MULTIPLE support (openmpi 1.6.5 or later recommended)
* Qt 5.4 or 5.5 (versions 5.6 and 5.7 currently have some
* Qt 5.4, 5.5 or 5.7 (version 5.6 currently has some
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

so 5.7 doesn't have the issues that 5.6 has? we could skip it then and tell people to use 5.7.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, I tested locally and on the displaywall and didn't have any crash (whereas the 5.6 crashes in less than a minute). This is very reassuring, but I am still going to see if something can be done about the 5.6 builds.

…n#85

Also:
- Try to build with Webengine support on Qt >= 5.6 (first version
  which works properly + has dropped support for Webkit), fallback
  to Webkit on older Qt versions.
- Fixed a crash if trying to open a webbrowser from the Launcher
  and Tide was compiled without any webbrowser support.
@rdumusc
Copy link
Author

rdumusc commented Sep 19, 2016

Update with also the following:

  • Try to build with Webengine support on Qt >= 5.6 (first version which works properly + has dropped support for Webkit), fallback to Webkit on older Qt versions.
  • Fixed a crash if trying to open a webbrowser from the Launcher and Tide was compiled without any webbrowser support.

@dnachbaur
Copy link
Contributor

+1

@rdumusc rdumusc merged commit b5b6118 into BlueBrain:master Sep 19, 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.

2 participants