search

uglyoldbob / decompiler

A decompiler targeting c and similar languages.
MIT License
41 stars 9 forks source link

Include information that udis is required #5

Closed rfalke closed 1 year ago

rfalke commented 6 years ago

Because of different aclocal version I had to execute "autoreconf -vfi". This failed because udis is required. A " tar xfvz ../Download/udis86-1.7.2.tar.gz" solved this. But it should be included in the README.

uglyoldbob commented 6 years ago

Good point. I think the configure.ac should also have some lines to detect if this condition is satisfied.

uglyoldbob commented 4 years ago

I have added udis86 as a git submodule. Need to put together some quick instructions for initial setup.

RandomInsano commented 1 year ago

It doesn't seem to have helped? I'm on macOS over here and tried installing udis86 from Homebrew. That seems to work as I can use udcli just fine but it's throwing this during the configure step:

configure: creating ./config.status
config.status: creating Makefile
config.status: creating ac_config.h
config.status: executing depfiles commands
=== configuring in udis86 (/Users/edwinamsler/Documents/Code/rust/other/decompiler/build/udis86)
configure: WARNING: no configuration information is in udis86
uglyoldbob commented 1 year ago

I believe this is because the build system assumes that udis86 will be built from source, rather than it being provided as an external precompiled library.

uglyoldbob commented 1 year ago

The udis86 library is a git submodule now. The readme defines how to checkout the repo and get the submodule as well.