Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Zeek renaming: update broker's bro.hh #359

Closed
jsiwek opened this issue May 8, 2019 · 2 comments

Comments

2 participants
@jsiwek
Copy link
Member

commented May 8, 2019

Didn't look closely for other Bro usages in Broker yet, but I noticed least broker/bro.hh and the bro namespace.

@jsiwek jsiwek added this to the 3.0.0 milestone May 8, 2019

@jsiwek jsiwek added this to Unassigned / To Do in Zeek Renaming Tasks via automation May 8, 2019

@dnthayer dnthayer self-assigned this May 13, 2019

@jsiwek jsiwek moved this from Unassigned / To Do to Assigned / In Progress in Zeek Renaming Tasks May 13, 2019

@jsiwek jsiwek removed the Task: Renaming label May 13, 2019

@jsiwek jsiwek added this to Unassigned / Todo in Release 3.0.0 via automation May 13, 2019

@jsiwek jsiwek moved this from Unassigned / Todo to Assigned / In Progress in Release 3.0.0 May 13, 2019

@dnthayer

This comment has been minimized.

Copy link
Contributor

commented May 15, 2019

The broker-related bro-to-zeek renaming is in branch "topic/dnthayer/gh-359" in the zeek and broker repos.

jsiwek added a commit that referenced this issue May 15, 2019

Merge remote-tracking branch 'origin/topic/dnthayer/gh-359'
* origin/topic/dnthayer/gh-359:
  Changes needed due to bro-to-zeek renaming in broker
@jsiwek

This comment has been minimized.

Copy link
Member Author

commented May 15, 2019

Thanks for helping with this one, Daniel.

@jsiwek jsiwek closed this May 15, 2019

Zeek Renaming Tasks automation moved this from Assigned / In Progress to Done May 15, 2019

Release 3.0.0 automation moved this from Assigned / In Progress to Done May 15, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.