Project

General

Profile

Actions

Bug #2473

closed

Please allow disabling every option (add --without-{option})

Added by Yuri Z over 6 years ago. Updated almost 6 years ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Target version:
-
Affected Versions:
Effort:
Difficulty:
Label:

Description

For libjansson, you have these two options:
--with-libjansson-includes=DIR libjansson include directory
--with-libjansson-libraries=DIR libjansson library directory

However, if I have libjansson installed, but wouldn't like to use it, there is no way to disable it.

I found this in the context of the FreeBSD port. The JSON option is "off", but it still finds libjansson and links to it.

Please add --without-{option} for every option which has --with-{option}.

Thank you.


Related issues 3 (0 open3 closed)

Related to Suricata - Feature #1970: json: make libjansson mandatoryClosedJason IshActions
Related to Suricata - Bug #1835: AC_ARG_ENABLE usage is incorrectClosedActions
Related to Suricata - Bug #2468: The autoconf process fails when some options are disabledClosedActions
Actions #1

Updated by Jason Ish over 6 years ago

  • Project changed from Suricata-Update to Suricata
  • Assignee deleted (Jason Ish)
Actions #2

Updated by Victor Julien over 6 years ago

In general, I agree. Wrt jansson, we are considering to make it mandatory: #1970

Actions #3

Updated by Victor Julien over 6 years ago

  • Related to Feature #1970: json: make libjansson mandatory added
Actions #4

Updated by Victor Julien over 6 years ago

  • Related to Bug #1835: AC_ARG_ENABLE usage is incorrect added
Actions #5

Updated by Victor Julien over 6 years ago

  • Related to Bug #2468: The autoconf process fails when some options are disabled added
Actions #6

Updated by Andreas Herz over 6 years ago

  • Assignee set to Andreas Herz
  • Target version set to TBD

@Yuri do you see any issues with making it mandatory?

Actions #7

Updated by Victor Julien almost 6 years ago

  • Status changed from New to Closed
  • Assignee deleted (Andreas Herz)
  • Target version deleted (TBD)

Fixed in ticket #1835

Actions

Also available in: Atom PDF