posixvala icon indicating copy to clipboard operation
posixvala copied to clipboard
verified publisher icon radare

Merge xdave branch containing vala rewrite

Open radare opened this issue 11 years ago • 7 comments

This branch provides the following changes:

  • rewrite the shellscripts into Vala
  • breaks Genie support
  • Implementes 'posixvala' and 'posixvalac' programs
  • Uses fork()+sed() while it must be using the internal vala regex engine

radare avatar Oct 19 '14 08:10 radare

who has the most updated version of this this project you or xdave?

momento-potsi avatar Jun 01 '20 21:06 momento-potsi

There's no pullreq from @xdave seems like he have worked out some more stuff after me, so i assume his branch is more updated, didnt tested though.

@xdave can you please submit a PR? so i can review the changes and merge it here?

Thanks!!

to avoid confussion there must be only one place to look at.

  • https://github.com/xdave/posixvala/commits/master
  • https://github.com/radare/posixvala/commits/master

radare avatar Jun 02 '20 11:06 radare

It's diverged a bit... and I've kind of abandoned it.. :P

xdave avatar Jun 02 '20 20:06 xdave

FYI there was a PR for this already... https://github.com/radare/posixvala/pull/6

xdave avatar Jun 02 '20 21:06 xdave

New PR: https://github.com/radare/posixvala/pull/13

xdave avatar Jun 02 '20 22:06 xdave

Wow so many changes!

Do this work with the latest Vala? Maybe worth to add some tests and CI integration

On 3 Jun 2020, at 00:01, xdave [email protected] wrote:

 New PR: #13

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

radare avatar Jun 03 '20 05:06 radare

I have no idea, the branch is years behind current vala

xdave avatar Jun 03 '20 23:06 xdave