updates to BAP.2.0 cmdline interface (#42)

* updates to BAP.2.0 cmdline interface

Bap.2.0 has a new cmdline interface, and this PR makes `bap-ida-python`
even with it

* fixes readme

Author: gitoleg

README.md

@@ -111,6 +111,20 @@ run:
 ```
 opam install bap-ida-python
 ```
+After the installation you'll see few commands that you need to run, e.g.:
+```
+=> In order to install bap-ida-python plugin:
+rm -rf $IDA_PATH/plugins/bap/
+cp $(opam config var prefix)/share/bap-ida-python/plugin_loader_bap.py $IDA_PATH/plugins/
+cp -r $(opam config var prefix)/share/bap-ida-python/bap $IDA_PATH/plugins/
+cp $(opam config var prefix)/share/bap-ida-python/bap.cfg $IDA_PATH/cfg/
+...
+```
+where `IDA_PATH` denotes the root of IDA Pro installation.
+
+You need to run all the commands manually because of the sandboxing enabled in
+modern versions of opam, that doesn't allow to install files outside
+the opam directory.
 
 Debugging
 ---------

plugins/bap/utils/run.py

@@ -70,8 +70,8 @@ def __init__(self, bap, input_file):
         if self.DEBUG:
             self.env['BAP_DEBUG'] = 'yes'
         if not Bap.plugins:
-            with os.popen(bap + ' --list-plugins') as out:
-                Bap.plugins = [e.split()[1] for e in out]
+            with os.popen(bap + ' list plugins') as out:
+                Bap.plugins = [e.split()[0] for e in out]
 
     def run(self):
         "starts BAP process"
@@ -201,8 +201,6 @@ def _setup_symbols(self):
             ida.output_symbols(out)
             self.args += [
                 "--read-symbols-from", out.name,
-                "--symbolizer=file",
-                "--rooter=file"
             ]
 
     def _setup_headers(self, bap):

tests/test_run.py

@@ -24,7 +24,6 @@ def test_run_without_args(bapida):
     assert args[0] == backend.path
     assert '--no-ida' in args
     assert '--read-symbols-from' in args
-    assert '--symbolizer=file' in args
 
 
 def test_disable_symbols(bapida):
@@ -38,7 +37,6 @@ def test_disable_symbols(bapida):
     assert args[0] == backend.path
     assert '--no-ida' in args
     assert '--read-symbols-from' not in args
-    assert '--symbolizer=file' not in args
 
 
 def test_event_handlers(bapida):