Skip to content

Conversation

@wargio
Copy link
Owner

@wargio wargio commented Aug 3, 2025

Fix #367

@wargio
Copy link
Owner Author

wargio commented Aug 3, 2025

@trufae seems that the 6.0.0 is borked.

https://github.com/wargio/r2dec-js/actions/runs/16702937772/job/47276788070?pr=368#step:7:205
FAILED: core_pdd.dll.p/c_r2dec-plugin.c.obj 
"cl" "-Icore_pdd.dll.p" "-I." "-I.." "-I..\c" "-I..\radare2\include\libr" "-Isubprojects\libquickjs" "-I..\subprojects\libquickjs" "-ID:\a\r2dec-js\r2dec-js\radare2\include" "/MD" "/nologo" "/showIncludes" "/utf-8" "/W2" "/O2" "/Gw" "-fvisibility=hidden" "-flto" "/Fdcore_pdd.dll.p\c_r2dec-plugin.c.pdb" /Focore_pdd.dll.p/c_r2dec-plugin.c.obj "/c" ../c/r2dec-plugin.c
cl : Command line warning D9002 : ignoring unknown option '-fvisibility=hidden'
cl : Command line warning D9002 : ignoring unknown option '-flto'
..\radare2\include\libr\r_util.h(29): fatal error C1083: Cannot open include file: 'r_event.h': No such file or directory

@trufae
Copy link
Contributor

trufae commented Aug 4, 2025

Meson install issue fixed here radareorg/radare2@266319b

can be workarounded like this https://github.com/nowsecure/r2frida/blob/eca8de580c6a6227f45d2740456b0793763d8df4/.github/workflows/ci.yml#L145

The fix will be included in r2-6.0.2

@wargio wargio merged commit 1017765 into master Aug 4, 2025
6 checks passed
@wargio wargio deleted the bump-6.0.0 branch August 4, 2025 12:41
@trufae
Copy link
Contributor

trufae commented Aug 13, 2025

the cp -f

cp -f radare2/include/libr/r_util/r_event.h radare2/include/libr/r_event.h
can be removed if you update the r2 dependency to 6.0.2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update artifacts for 6.0.0

3 participants