-
-
Notifications
You must be signed in to change notification settings - Fork 3
Remove JSON #27
base: master
Are you sure you want to change the base?
Remove JSON #27
Conversation
Does rizin still work against this or does it need adjustments? |
@thestr4ng3r 2 things still need it:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just to wait for all the stuff to be done.
@ret2libc any idea why this error appeared?
|
I think 0.63.0 became stricter. The manual actually says that link_whole should be a sdb_whole_dep = declare_dependency(
- link_whole: libsdb,
+ link_whole: [libsdb],
include_directories: libsdb_inc,
) |
@ret2libc didn't help, looking at the error message, the culprit seems in the value itself |
Right... my bad. Just wrap it in a |
Remove all JSON-related features since it's not used by Rizin.
A first step to address rizinorg/rizin#274 (comment)
cc @thestr4ng3r @08a