Project

General

Profile

Actions

Feature #1040

closed
KS KS

Suricata should compile with -Werror

Feature #1040: Suricata should compile with -Werror

Added by Ken Steele over 12 years ago. Updated about 12 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.

KS Updated by Ken Steele over 12 years ago Actions #1

Pull Request 636 fixed compiler warnings on Tile.

KS Updated by Ken Steele over 12 years ago Actions #2

  • % Done changed from 0 to 80

KS Updated by Ken Steele over 12 years ago Actions #3

  • Assignee set to Ken Steele

KS Updated by Ken Steele over 12 years ago Actions #4

  • % 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?)

VJ Updated by Victor Julien over 12 years ago Actions #5

  • Target version set to 2.0beta2

VJ Updated by Victor Julien over 12 years ago Actions #6

  • Target version changed from 2.0beta2 to 2.0rc1

VJ Updated by Victor Julien about 12 years ago Actions #7

  • 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: PDF Atom