Project

General

Profile

Actions

Optimization #3039

closed

configure: don't generate warnings on missing features

Added by Victor Julien over 5 years ago. Updated about 5 years ago.

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

Description

New users get confused by the warnings generated by configure about non-essential features.

Language should be updated to be informational, or perhaps not mention it at all beyond configures normal output. The summary at the end could be enough.


Related issues 1 (0 open1 closed)

Related to Suricata - Bug #3113: python-yaml dependency is actually ptyhon3-yaml dependency ClosedJason IshActions
Actions #1

Updated by Andreas Herz over 5 years ago

  • Assignee set to OISF Dev
  • Target version set to TBD

Do you have an example for how you would like it to look?

Actions #2

Updated by Victor Julien over 5 years ago

Not really. The table/listing at the end of configure we have now might be good enough for most cases, but it'd be nice to also output the package names for installing the dependency of a feature somewhere. Currently that is part of the warning/error messages.

Actions #3

Updated by Victor Julien about 5 years ago

  • Target version changed from TBD to 5.0.0

I think as a first step it would be good not to print the panic-y ERROR messages for components that are simply optional.

Actions #4

Updated by Andreas Herz about 5 years ago

With table/listing you mean the build-info like output?
So you would remove all warnings for optional parts and just have a list in the end with the info what is needed for those optional parts?

Would be a nice overview table/matrix in the docs useful as well or duplicated work?

I also saw that we're a bit inconsistent with the install recommendations, most are Ubuntu/Fedora/Centos but some just Ubuntu. Also mix between several methods (apt install and apt-get install). Could be moved to docs as well.

Actions #5

Updated by Jason Ish about 5 years ago

  • Related to Bug #3113: python-yaml dependency is actually ptyhon3-yaml dependency added
Actions #6

Updated by Victor Julien about 5 years ago

  • Status changed from New to Assigned
  • Assignee changed from OISF Dev to Victor Julien

For this ticket I will just remove the big ERROR parts of messages when we in fact don't error out.

Actions #7

Updated by Victor Julien about 5 years ago

  • Status changed from Assigned to Closed
Actions

Also available in: Atom PDF