Addition of GTEST_INCLUDE_DIR requires dependencies

Bug #1076763 reported by Francis Ginther
6
This bug affects 1 person
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-team/staging:
https://launchpadlibrarian.net/122383545/buildlog_ubuntu-quantal-amd64.unity_6.10.0bzr2884pkg0quantal0_FAILEDTOBUILD.txt.gz

or a backup is here:
http://paste.ubuntu.com/1343782/

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

Revision history for this message
Francis Ginther (fginther) wrote :
Revision history for this message
Francis Ginther (fginther) wrote :

A quick fix was done under:
https://code.launchpad.net/~fginther/unity/unity-revert-gtest/+merge/133576

A proper fix is needed.

Changed in unity:
status: New → Fix Committed
Stephen M. Webb (bregma)
Changed in unity:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.