search

angr / tracer

Utilities for generating dynamic traces
BSD 2-Clause "Simplified" License
88 stars 28 forks source link

Adding argv option and default. #20

Closed bannsec closed 7 years ago

bannsec commented 7 years ago

Adding option to specify argv and have it default to using the supplied binary name as the file name.

bannsec commented 7 years ago

not sure why that colorguard check is failing..

zardus commented 7 years ago

Travis should really have an option to auto-retry failed builds (or we should look into the flaky plugin for nosetests). Some of these tests fail intermittently. Let's see if this was such a case.

bannsec commented 7 years ago

Hah. Yeah, passed this time.

ltfish commented 7 years ago

@zardus IMO we should fix our tests...

zardus commented 7 years ago

Rather than use flaky? I'm all for it, but launching binaries and such (which we do in some of our tests) always has inherent race conditions, so it'll be hard to eliminate all race conditions. IMO, flaky is a good bandaid as well.

salls commented 7 years ago

Thanks!