search

radareorg / radare2-bindings

Bindings of the r2 api for Valabind and friends
GNU Lesser General Public License v3.0
130 stars 92 forks source link

Separate `libr/lang` plugins into another repository #186

Closed XVilka closed 6 years ago

XVilka commented 6 years ago

Technically those are not really related to these autogenerated bindings, can perfectly work with r2pipe, so I think it make sense to put them into the another repository. It will improve their visibility too - here they are "lost". @radare what do you think?

radare commented 6 years ago

Agree on that but they are here because its a pain to have more repositories than the ones we have already and deps are the same, originally r2pipe was living in here too.

Is there any willing to evolve the lang plugins?

On 22 Mar 2018, at 05:44, Anton Kochkov notifications@github.com wrote:

Technically those are not really related to these autogenerated bindings, can perfectly work with r2pipe, so I think it make sense to put them into the another repository. It will improve their visibility too - here they are "lost". @radare what do you think?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

XVilka commented 6 years ago

Yes, I slowly improve Python one, will probably check other languages too, for consistency.

radare commented 6 years ago

Agree on that. This will make it clear the difference.. but its a bit confusing because of the amount of bindings we have

On 22 Mar 2018, at 17:56, Anton Kochkov notifications@github.com wrote:

Yes, I slowly improve Python one, will probably check other languages too, for consistency.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

XVilka commented 6 years ago

@radare so, can you please do the repo for this when you have time?

radare commented 6 years ago

radare2-bindings-lang?

Any better name?

On 28 Mar 2018, at 06:26, Anton Kochkov notifications@github.com wrote:

@radare so, can you please do the repo for this when you have time?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

XVilka commented 6 years ago

maybe 'lang-plugins'?

radare commented 6 years ago

Why not just put them in extras? I dont expect too much wok to be done in those plugins in the near future

On 30 Mar 2018, at 11:58, Anton Kochkov notifications@github.com wrote:

maybe 'lang-plugins'?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

XVilka commented 6 years ago

We can try to put it in the separate repo, I will add more documentation on how to implement/improve them too. If there won't be much dynamics - you can put in radare2-extras then.

radare commented 6 years ago

maybe radare2-langs is the best name for it, but we need to copy the configure-langs logic, and i dont see much traction on this to have enough reasons to maintain another repo, which implies making releases more complex. if you want to contribute i would suggest to do it in this repo.