Ticket #7245 (closed: fixed)
Remove qtassistant from the "all" target
Reported by: | Peter Peterson | Owned by: | Peter Peterson |
---|---|---|---|
Priority: | major | Milestone: | Release 2.6 |
Component: | Tools | Keywords: | |
Cc: | martyn.gigg@…, campbellsi@… | Blocked By: | |
Blocking: | Tester: | Mathieu Doucet |
Description (last modified by Nick Draper) (diff)
There are two parts to this:
- Remove qtassistant from all
- Reconfigure build servers to "make qtassistant" before "make package" otherwise everything will be broken
Step 2. will have to be done when this is merged into develop, and again when merged into master.
Change History
comment:2 Changed 7 years ago by Peter Peterson
Merge branch 'feature/7245_remove_qtassistant_from_all' into develop
Conflicts:
Code/Mantid/docs/qtassistant/CMakeLists.txt
Re #7245.
Changeset: 2aeeb09b22357582b85edbdc93ace8d2ba4136e2
comment:3 Changed 7 years ago by Peter Peterson
- Status changed from new to accepted
- Owner set to Peter Peterson
comment:4 Changed 7 years ago by Peter Peterson
- Status changed from accepted to verify
- Resolution set to fixed
This is on feature/7245_remove_qtassistant_from_all.
To test (pedantic method):
- create a new build area
- cmake -DENABLE_QTASSISTANT=True -DENABLE_CPACK=True
- Confirm that make (with default target) doesn't create qtassistant files (in bin/qtassistant/html), "make package" should also fail
- "make qtassistant" should generate the qtassistant files and rerunning "make package" should also succeed.
comment:5 Changed 7 years ago by Mathieu Doucet
- Status changed from verify to verifying
- Tester set to Mathieu Doucet
Note: See
TracTickets for help on using
tickets.
Re #7245. Removing qtassistant from the all target.
Changeset: e06d7f8d9a62db8d217a57494efb2b9ef2272046