search

rizinorg / rizin-pm

1 stars 3 forks source link

Add rz-svd - the SVD parser plugin #7

Closed officialcjunior closed 3 years ago

officialcjunior commented 3 years ago

Add rz-svd - the SVD parser plugin to the DB.

Followed the keystone plugin as an example while making this: https://github.com/rizinorg/rizin-pm/blob/master/db/keystone

ret2libc commented 3 years ago

I assume you have tried to install the plugin and it worked, right?

ret2libc commented 3 years ago

rz-pm will be the focus of next iteration anyway... it needs a lot of love to make it distro friendly.

officialcjunior commented 3 years ago 
$ rz-pm -i rz-svd
Install Done For rz-svd
The Meson build system
Version: 0.59.0
Source dir: /home/cjunior/.local/share/rizin/rz-pm/git/rizin-extras/rz-svd
Build dir: /home/cjunior/.local/share/rizin/rz-pm/git/rizin-extras/rz-svd/build
Build type: native build
Project name: rz-svd
Project version: undefined
C compiler for the host machine: cc (gcc 11.1.0 "cc (GCC) 11.1.0")
C linker for the host machine: cc ld.bfd 2.36.1
Host machine cpu family: x86_64
Host machine cpu: x86_64
Program python3 found: YES (/usr/bin/python)

Executing subproject yxml

yxml| Project name: yxml
yxml| Project version: 66507906673bc6159d5d620414479954c9c21c24
yxml| C compiler for the host machine: cc (gcc 11.1.0 "cc (GCC) 11.1.0")
yxml| C linker for the host machine: cc ld.bfd 2.36.1
yxml| Build targets in project: 1
yxml| Subproject yxml finished.

Found pkg-config: /usr/bin/pkg-config (1.7.3)
Run-time dependency rz_core found: YES 0.3.0-git
Build targets in project: 2

rz-svd undefined

  Subprojects
    yxml: YES

Found ninja-1.10.2 at /usr/bin/ninja
ninja: Entering directory `build'
[4/4] Linking target librz-svd.so
ninja: Entering directory `build'
[0/1] Installing files.
Installing librz-svd.so to /home/cjunior/.local/share/rizin/share/rizin/plugins

Don't worry about the place to which its installed on my machine. I've used the same location that is used by Jaguar GPU plugin and other plugins (https://github.com/rizinorg/rizin-extras/pull/3#discussion_r594579834). Also the CI succeeds on https://github.com/rizinorg/rizin-extras/runs/3291878636

So, I think we can go when https://github.com/rizinorg/rizin-extras/pull/11 goes in.

officialcjunior commented 3 years ago

Now, this can definitely go in. See: https://github.com/rizinorg/rizin-extras/pull/11