radareorg / acr

autoconf replacement
GNU General Public License v2.0
34 stars 10 forks source link

Add WSL detection #20

Closed GustavoLCR closed 5 years ago

radare commented 5 years ago

Wtf??? You have to edit the configure.acr not this script

On 15 Feb 2019, at 04:13, GustavoLCR notifications@github.com wrote:

You can view, comment on, or merge this pull request online at:

https://github.com/radare/acr/pull/20

Commit Summary

Add WSL detection File Changes

M src/acr-sh (3) Patch Links:

https://github.com/radare/acr/pull/20.patch https://github.com/radare/acr/pull/20.diff — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or mute the thread.

GustavoLCR commented 5 years ago

Yeah, but I didn't see a way to detect wsl and disable ptrace-wrap automatically. With this I can add in the configure.acr: IFEQ USEROSTYPE wsl ; { HAVE_PTRACE = 1 ; USE_PTRACE_WRAP = 0 ; }

radare commented 5 years ago

Can you do a pr?

On 15 Feb 2019, at 15:17, GustavoLCR notifications@github.com wrote:

Yeah, but I didn't see a way to detect wsl and disable ptrace-wrap automatically. With this I can add in the configure.acr: IFEQ USEROSTYPE wsl ; { HAVE_PTRACE = 1 ; USE_PTRACE_WRAP = 0 ; }

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

GustavoLCR commented 5 years ago

@radare can you check this one again?