Project

General

Profile

Actions

Feature #1040

closed

Suricata should compile with -Werror

Added by Ken Steele about 11 years ago. Updated almost 11 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Target version:
Effort:
Difficulty:
Label:

Description

I would like to see Suricata add -Werror for CFLAGS by default. It prevents masking bugs by ignoring compiler warnings.

Actions #1

Updated by Ken Steele about 11 years ago

Pull Request 636 fixed compiler warnings on Tile.

Actions #2

Updated by Ken Steele about 11 years ago

  • % Done changed from 0 to 80
Actions #3

Updated by Ken Steele about 11 years ago

  • Assignee set to Ken Steele
Actions #4

Updated by Ken Steele about 11 years ago

  • % Done changed from 80 to 90

Pull request 642 will fix warning with -Werror when configuring with libprelude.

After that change, CFLAGS="-Werror" ./configure will work. The only remaining step is to enable -Werror by default, or by default after a specific version of GCC (4.4?)

Actions #5

Updated by Victor Julien about 11 years ago

  • Target version set to 2.0beta2
Actions #6

Updated by Victor Julien about 11 years ago

  • Target version changed from 2.0beta2 to 2.0rc1
Actions #7

Updated by Victor Julien almost 11 years ago

  • Status changed from New to Closed
  • % Done changed from 90 to 100

I've setup my QA gear to use this. Don't think it's safe yet to force this through configure.

Actions

Also available in: Atom PDF