Ticket #2932 (closed: fixed)
The Mac install is larger than it needs to be
| Reported by: | Russell Taylor | Owned by: | Russell Taylor |
|---|---|---|---|
| Priority: | minor | Milestone: | Release 2.3 |
| Component: | Infrastructure | Keywords: | |
| Cc: | martyn.gigg@… | Blocked By: | |
| Blocking: | Tester: | Michael Reuter |
Description
The cpack installer pulls in all the Qt libraries, even those we don't use (e.g. the 50Mb webkit library). Figure out how to only get the ones we want.
Change History
comment:2 Changed 9 years ago by Nick Draper
- Milestone changed from Iteration 30 to Iteration 31
Bulk move of tickets to iteration 31 at the iteration 30 code freeze
comment:3 Changed 9 years ago by Nick Draper
- Milestone changed from Iteration 32 to Iteration 33
Moved to iteration 33 at iteration 32 code freeze
comment:4 Changed 8 years ago by Nick Draper
- Milestone changed from Release 2.1 to Release 2.2
Moved at end of release 2.1
comment:6 Changed 8 years ago by Russell Taylor
Turns out that the problem is putting the whole of PyQt inside our installer, because each PyQt library pulls in its corresponding Qt library (when fixup_bundle looks at the dependencies). This WebKit etc is there even though we don't need it.
comment:7 Changed 8 years ago by Russell Taylor
Re #2932. Specify which PyQt libraries to include in Mac installer.
This stops Qt libraries such as WebKit being included unnecessarily in the install kit, which significantly increased its size. Actually including more than we currently use (which is only Qt, QtCore, QtGui & uic) because the others relate to Qt packages on which MantidPlot itself already depends so we wouldn't gain much by leaving them out.
Changeset: 67e95e9d534862a67d94198938ac80fef9ce63d9
comment:8 Changed 8 years ago by Russell Taylor
- Status changed from accepted to verify
- Resolution set to fixed
The size of the install kit had gone down from 91MB to 59MB. Everything still seems to work :)
Tester: Make sure that python things still work from MantidPlot, in particular the PyQt GUIs (ORNL SANS, REFM, REFL).
comment:9 Changed 8 years ago by Russell Taylor
- Status changed from verify to reopened
- Resolution fixed deleted
Webkit etc. is back in the installer again! Seems to have happened between 22nd & 24th May. Surely the Vates stuff doesn't require every single Qt library?!
comment:10 Changed 8 years ago by Russell Taylor
The issue is that the paraview on the ISIS buildserver is linked to these libraries, so thus the paraview plugins are and the libraries are then pulled in. On my machine I had to set VTK_QT_USE_WEBKIT (I think that's the one) to OFF to even get paraview to build. Then the dependency is not there, the libraries are not in the package and yet everything seems to work fine.
comment:11 Changed 8 years ago by Nick Draper
- Milestone changed from Release 2.2 to Release 2.3
comment:13 Changed 8 years ago by Russell Taylor
- Cc martyn.gigg@… added
- Status changed from accepted to verify
- Resolution set to fixed
Martyn has rebuilt Paraview on the ISIS Snow Leopard buildserver and now the installer on download.mantidproject.org is down to ~65 MB.
To test: Verify that MantidPlot still starts up correctly and that /Applications/MantidPlot.app/Contents/Frameworks does NOT contain QtDBus.framework, QtWebKit.framework & phonon.framework.
comment:14 Changed 8 years ago by Michael Reuter
- Status changed from verify to verifying
- Tester set to Michael Reuter
comment:15 Changed 8 years ago by Michael Reuter
- Status changed from verifying to closed
This seems to be working correctly.
comment:16 Changed 5 years ago by Stuart Campbell
This ticket has been transferred to github issue 3779
"New" tickets moved at the code freeze of iteration 29