FTBFS with new GCC 6 -Wmisleading-indentation warning

Bug #1613482 reported by Rik Mills
12
This bug affects 2 people
Affects Status Importance Assigned to Milestone
ubiquity (Ubuntu)
Undecided
Unassigned

Bug Description

ubiquity 16.10.7
ubuntu 16.10 (yakkety)

New GCC 6 -Wmisleading-indentation warning causes ubiquity build to fail on the file wpa.c

What should happen?

Should build from source.

What does happen?

FTBFS as below:

wpa.c: In function ‘poll_wpa_supplicant’:
wpa.c:260:10: error: this ‘if’ clause does not guard... [-Werror=misleading-indentation]
          if (debconf_progress_info(client, "netcfg/wpa_progress_note") ==
          ^~
wpa.c:264:14: note: ...this statement, but the latter is misleadingly indented as if it is guarded by the ‘if’
              if (debconf_progress_step(client, 1) == CMD_PROGRESSCANCELLED)
              ^~
cc1: all warnings being treated as errors
Makefile:60: recipe for target 'wpa.o' failed

A diff allowing the package to build is attached

Revision history for this message
Rik Mills (rikmills) wrote :
Revision history for this message
Ubuntu Foundations Team Bug Bot (crichton) wrote :

The attachment "fix-identation.diff" seems to be a patch. If it isn't, please remove the "patch" flag from the attachment, remove the "patch" tag, and if you are a member of the ~ubuntu-reviewers, unsubscribe the team.

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

tags: added: patch
tags: added: yakkety
tags: added: ftbfs
Revision history for this message
Launchpad Janitor (janitor) wrote :

Status changed to 'Confirmed' because the bug affects multiple users.

Changed in ubiquity (Ubuntu):
status: New → Confirmed
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers