Addition of GTEST_INCLUDE_DIR requires dependencies
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Unity |
Fix Released
|
Critical
|
Unassigned | ||
unity (Ubuntu) |
Fix Released
|
Undecided
|
Unassigned |
Bug Description
A change to lp:unity in rev 2884 is causing build failures in ppa:unity-
https:/
or a backup is here:
http://
and also attached.
The change to CMakeLists.txt appears to enable variables that are only valid when libgtest-dev and google-mock are installed.
[Test Case]
- build in a pure pbuilder chroot with no additional depenencies
Note: this made it through the auto-merger because libgtest-dev and google-mock (and others) are added to the pbuilder prior to executing the build. This is needed to execute the unit tests. However, in launchpad, this is not done and therefore the build fails.
Related branches
- PS Jenkins bot (community): Needs Fixing (continuous-integration)
- Francis Ginther: Approve
-
Diff: 60 lines (+15/-11)3 files modifiedCMakeLists.txt (+7/-11)
debian/changelog (+6/-0)
debian/control (+2/-0)
Changed in unity: | |
status: | New → Fix Committed |
Changed in unity: | |
status: | Fix Committed → Fix Released |
A quick fix was done under: /code.launchpad .net/~fginther/ unity/unity- revert- gtest/+ merge/133576
https:/
A proper fix is needed.