[Bro-Dev] #622: Installation issues

[Bro Tracker]

#622: Installation issues
----------------------+------------------------
  Reporter:  robin    |      Owner:
      Type:  Problem  |     Status:  new
  Priority:  Normal   |  Milestone:  Bro1.6
 Component:  Bro      |    Version:  git/master
Resolution:           |   Keywords:  beta
----------------------+------------------------
Changes (by robin):

 * milestone:  Bro1.6 Bro1.7 => Bro1.6


Old description:

> Sorry for going back to the installation process but a few thoughts
> about "make install" (feel free to remind me about past discussions, I
> don't remember these very well anymore):
>
> - It currently installs the following binaries:
>
>     adtrace         bdcat           bro             broccoli-config
>     broctl          capstats        ftwire2bro      nfcollector
>     rst             trace-summary
>
>   That's seem too much to me, I think we should skip the bro-aux
>   programs by default. That would then leave us with just
>
>     bro broctl capstats trace-summary broccoli-config
>
>   We could add target "make install-aux" to get the rest.
>
> - It creates "$prefix/logs". I don't remember whether we discussed
>   this already, but is that a good default location for logs?
>
> - I noticed that we still install scripts that are now obsolete:
>
>         bro-logchk.pl
>         hot-report
>         mon-report
>         mvlog
>
>   That leaves us in into $prefix/scripts with:
>
>         host-grep
>         ip-grep
>         host-to-addrs
>         lock_file
>
>   However, I'm almost thinking we should remove these as well; not
>   sure it's worth shipping them. Or, if we really want to keep them,
>   at least don't install by default (then perhaps as part of the
>   "install-aux" target from above).
>
> - Was there a reason we install the spool pieces directly into
>   $prefix/spool/ instead of into $prefix/spool/bro/?
>
> - test-all-policy.bro ends up in share/bro, which doesn't seem right.
>
> - Do we really need to install share/bro/site/local-proxy.bro? It's
>   empty and unlikley that many people will want to edit it.
>
> - Thoughts about local.bro:
>
>     * Why is "@load protocols/http/detect-webapps" commented out? We
>     should add a comment explaining when one would want to include it.
>
>     * "Requires that the Site::local_zones variable". We should add
>     where/how to do that. Perhaps an empty definition right in
>     local.bro?

New description:

 Sorry for going back to the installation process but a few thoughts
 about "make install" (feel free to remind me about past discussions, I
 don't remember these very well anymore):

 - It currently installs the following binaries:

     adtrace         bdcat           bro             broccoli-config
     broctl          capstats        ftwire2bro      nfcollector
     rst             trace-summary

   That's seem too much to me, I think we should skip the bro-aux
   programs by default. That would then leave us with just

     bro broctl capstats trace-summary broccoli-config

   We could add target "make install-aux" to get the rest.

 - It creates "$prefix/logs". I don't remember whether we discussed
   this already, but is that a good default location for logs?

 - Was there a reason we install the spool pieces directly into
   $prefix/spool/ instead of into $prefix/spool/bro/?

 - test-all-policy.bro ends up in share/bro, which doesn't seem right.

 - Do we really need to install share/bro/site/local-proxy.bro? It's
   empty and unlikley that many people will want to edit it.

 - Thoughts about local.bro:

     * Why is "@load protocols/http/detect-webapps" commented out? We
     should add a comment explaining when one would want to include it.

     * "Requires that the Site::local_zones variable". We should add
     where/how to do that. Perhaps an empty definition right in
     local.bro?

--

-- 
Ticket URL: <http://tracker.bro-ids.org/bro/ticket/622#comment:1>
Bro Tracker <http://tracker.bro-ids.org/bro>
Bro Issue Tracker