search

aperezdc / snabb

Snabb Switch: Fast open source packet processing
Apache License 2.0
4 stars 1 forks source link

Name for snabbwall executable #16

Closed takikawa closed 7 years ago

takikawa commented 7 years ago

Currently doing make dist for snabbwall sets up an executable named snabbwall in the resulting tar archive, but that executable prints an error like the following:

$ sudo ./snabbwall spy
unsupported program: snabbwall
Usage: ./snabbwall <program> ...

Because snabb uses the executable name to choose a sub-program. Should we just change the name to wall for distribution? (or maybe just snabb?)

aperezdc commented 7 years ago

It should be snabb-wall, IIRC the logic for chosing the first subcommand also checks for the snabb- prefix and picks the string after the dash.

takikawa commented 7 years ago

Thanks, I made that change.