Project

General

Profile

Actions
Copy link

Feature #1040

closed

Suricata should compile with -Werror

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

Status:
Closed
Priority:
Normal
Assignee:
Ken Steele
Target version:
2.0rc1
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
Copy link
 #1

Updated by Ken Steele almost 11 years ago

Pull Request 636 fixed compiler warnings on Tile.

Actions
Copy link
 #2

Updated by Ken Steele almost 11 years ago

  • % Done changed from 0 to 80
Actions
Copy link
 #3

Updated by Ken Steele almost 11 years ago

  • Assignee set to Ken Steele
Actions
Copy link
 #4

Updated by Ken Steele almost 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
Copy link
 #5

Updated by Victor Julien almost 11 years ago

  • Target version set to 2.0beta2
Actions
Copy link
 #6

Updated by Victor Julien almost 11 years ago

  • Target version changed from 2.0beta2 to 2.0rc1
Actions
Copy link
 #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
Copy link

Also available in: Atom PDF