[Bro-Dev] #513: topic/jsiwek/update-restdoc-target

[Bro Tracker]

#513: topic/jsiwek/update-restdoc-target
----------------------------+------------------------
  Reporter:  jsiwek         |      Owner:  jsiwek
      Type:  Merge Request  |     Status:  assigned
  Priority:  Normal         |  Milestone:  Bro1.6
 Component:  Bro            |    Version:  git/master
Resolution:                 |   Keywords:
----------------------------+------------------------
Changes (by robin):

 * owner:   => jsiwek
 * status:  new => assigned


Comment:

 Questions about some of the script changes in here:

 - you added many @loads. Are they necessary only so that the doc
 generation works and not otherwise because normally the scripts would be
 pulled in implicitly? If indeed so, this is not so great: we will probably
 run into the problem all the time because it will only show up when docs
 are generated. However, I don't have a good idea right now how to address
 that.

 - The change from ProtocolFound to Protocol_Found: is that according to
 our new naming conventions?

 - One chunk adds a commit `#This should be optional`:  why isn't it?

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