Addition of GTEST_INCLUDE_DIR requires dependencies

Bug #1076763 reported by Francis Ginther on 2012-11-08
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Unity
Fix Released
Critical
Unassigned
unity (Ubuntu)
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

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) on 2013-02-17
Changed in unity:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers