Ubuntu
gtest package

TEST_P fails with new G++ because of Property_CopyBaseTypeConstructor_Test::gtest_registering_dummy_' defined but not used

Bug #1521177 reported by Marco Trevisan (Treviño) on 2015-11-30

This bug affects 1 person

Affects		Status	Importance	Assigned to	Milestone
	gtest (Ubuntu)	Fix Released	High	Marco Trevisan (Treviño)	Ubuntu ubuntu-16.04

Bug Description

This is the error we get when compiling unity tests with new gcc:
- '{anonymous}::Property_CopyBaseTypeConstructor_Test::gtest_registering_dummy_' defined but not used [-Werror=unused-variable]
TEST_P(/*TestDesktopApplicationSubject*/Property, CopyBaseTypeConstructor)

Tags:

Marco Trevisan (Treviño) (3v1n0) on 2015-11-30

affects:

ubuntu → gtest (Ubuntu)

Revision history for this message

Marco Trevisan (Treviño) (3v1n0) wrote on 2015-11-30:

gtest_1.7.0-4ubuntu1.dsc.diff Edit (5.4 KiB, text/plain)

Revision history for this message

Ubuntu Foundations Team Bug Bot (crichton) wrote on 2015-11-30:

The attachment "gtest_1.7.0-4ubuntu1.dsc.diff" seems to be a debdiff. The ubuntu-sponsors team has been subscribed to the bug report so that they can review and hopefully sponsor the debdiff. If the attachment isn't a patch, please remove the "patch" flag from the attachment, remove the "patch" tag, and if you are member of the ~ubuntu-sponsors, unsubscribe the team.

[This is an automated message performed by a Launchpad user owned by ~brian-murray, for any issue please contact him.]

tags:

added: patch

Revision history for this message

Sebastien Bacher (seb128) wrote on 2015-11-30:

thanks!

Changed in gtest (Ubuntu):
status:	In Progress → Fix Committed

Revision history for this message

Launchpad Janitor (janitor) wrote on 2015-11-30:

This bug was fixed in the package gtest - 1.7.0-4ubuntu1

---------------
gtest (1.7.0-4ubuntu1) xenial; urgency=medium

  * patches/test-case-p-unused-dummy-variable-fix.patch:
    - Add GTEST_ATTRIBUTE_UNUSED_ to the dummy variable generated
       in INSTANTIATE_TEST_CASE_P (LP: #1521177).

-- Marco Trevisan (Treviño) <email address hidden> Mon, 30 Nov 2015 13:44:53 +0100