Optimization #3039
closedconfigure: don't generate warnings on missing features
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.
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?
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.
Updated by Victor Julien over 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.
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.
Updated by Jason Ish about 5 years ago
- Related to Bug #3113: python-yaml dependency is actually ptyhon3-yaml dependency added
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.
Updated by Victor Julien about 5 years ago
- Status changed from Assigned to Closed
Errors removed in https://github.com/OISF/suricata/pull/4295