Page MenuHomeFreeBSD

snort2lua, installed by security/snort3, needs to be available as astandalone binary alongside snort2.
Needs ReviewPublic

Authored by dvl on Dec 18 2020, 6:25 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Apr 16, 12:50 AM
Unknown Object (File)
Wed, Apr 10, 1:54 PM
Unknown Object (File)
Jan 1 2024, 1:55 AM
Unknown Object (File)
Dec 31 2023, 8:41 PM
Unknown Object (File)
Dec 31 2023, 7:08 PM
Unknown Object (File)
Nov 22 2023, 6:01 PM
Unknown Object (File)
Nov 22 2023, 5:52 PM
Unknown Object (File)
Nov 21 2023, 5:06 PM
Subscribers

Details

Reviewers
bofh
Summary

snort2lua, part of the conversion package from Snort 2 to Snort 3, needs to be installable with snort 2.

See also https://reviews.freebsd.org/D28027

This proof-of-concept, taken directly from security/snort3, is a starting
point.

Options I have considered are:

  • slave port
  • add an OPTION to security/snort3
  • add FLAVORS to security/snort3

Suggestions please

Diff Detail

Repository
rP FreeBSD ports repository
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 35999
Build 32888: arc lint + arc unit

Event Timeline

dvl requested review of this revision.Dec 18 2020, 6:25 PM
dvl retitled this revision from snort2lua, installed by security/snort3, needs to be available as a standalone binary alongside snort2. to snort2lua, installed by security/snort3, needs to be available as astandalone binary alongside snort2..Dec 18 2020, 6:40 PM
dvl edited the summary of this revision. (Show Details)
security/snort2lua/Makefile
2โ€“3

Just this one question why do we need version specific to 3.0.3-1 whereas we have 3.0.3-4?

dvl marked an inline comment as done.Dec 18 2020, 11:33 PM
dvl added inline comments.
security/snort2lua/Makefile
2โ€“3

We don't need that version.

I think I was working on 2020Q4 and copied it to head.

dvl marked an inline comment as done.Jan 7 2021, 4:10 PM
$ sudo pkg install snort3-snort2lua
Updating vrt repository catalogue...
vrt repository is up to date.
All repositories are up to date.
The following 4 package(s) will be affected (of 0 checked):

New packages to be INSTALLED:
	hwloc: 1.11.11
	hyperscan: 5.2.1
	libdaq: 3.0.0.b1
	snort3-snort2lua: 3.0.3.1,1

Number of packages to be installed: 4

The process will require 25 MiB more space.
5 MiB to be downloaded.

Proceed with this action? [y/N]: y
[1/4] Fetching snort3-snort2lua-3.0.3.1,1.txz: 100%  239 KiB 244.8kB/s    00:01    
[2/4] Fetching libdaq-3.0.0.b1.txz: 100%  112 KiB 114.6kB/s    00:01    
[3/4] Fetching hyperscan-5.2.1.txz: 100%    3 MiB   2.7MB/s    00:01    
[4/4] Fetching hwloc-1.11.11.txz: 100%    2 MiB   2.2MB/s    00:01    
Checking integrity... done (1 conflicting)
  - libdaq-3.0.0.b1 conflicts with daq-2.2.2 on /usr/local/include/daq.h
Checking integrity... done (0 conflicting)
Conflicts with the existing packages have been found.
One more solver iteration is needed to resolve them.
The following 7 package(s) will be affected (of 0 checked):

New packages to be INSTALLED:
	hwloc: 1.11.11
	hyperscan: 5.2.1
	libdaq: 3.0.0.b1
	snort3-snort2lua: 3.0.3.1,1

Installed packages to be REINSTALLED:
	pkg-1.15.10

Number of packages to be installed: 4
Number of packages to be reinstalled: 1

The process will require 25 MiB more space.
7 MiB to be downloaded.

Proceed with this action? [y/N]: y
[1/1] Fetching pkg-1.15.10.txz: 100%    7 MiB   6.9MB/s    00:01    
Fetching daq-2.2.2.txz: 100%  160 KiB 163.9kB/s    00:01    
[1/7] Deinstalling daq-2.2.2...
[1/7] Deleting files for daq-2.2.2: 100%
[2/7] Installing libdaq-3.0.0.b1...
[2/7] Extracting libdaq-3.0.0.b1: 100%
[3/7] Installing hyperscan-5.2.1...
[3/7] Extracting hyperscan-5.2.1: 100%
[4/7] Installing hwloc-1.11.11...
[4/7] Extracting hwloc-1.11.11: 100%
[4/7] Installing daq-2.2.2...
pkg: daq-2.2.2 conflicts with libdaq-3.0.0.b1 (installs files into the same place).  Problematic file: /usr/local/include/daq.h
$ 
```One problem, unexpected installed, but not required, dependencies.

Locally, I have changed LIB_DEPENDS into BUILD_DEPENDS. Patches soon. My solution is unmaintainable though.

Change LIB_DEPENDS to BUILD_DEPENDS to avoid unnecessary dependencies

If left as-is, this would conflict with security/snort

This is still an unmaintainable solution.

That latest update is the wrong content.