radare2 @0.9 New submission

[g5pw (Aljaž Srebrnič)]

This is a basic portfile for the amazing reverse-engineering software made by pancake and his team! It does not include bindings, I will add them as soon as possible!

First portfile ever. Open for suggestions!

[pixilla (Bradley Giesbrecht)]

a2piratesoft: Your Portfile matches our guide very well. Thanks.

Does radare2 installed by your Portfile work for you? It seems to me that the compiled binaries have incorrect library paths.

$ otool -L ./work/destroot/opt/local/lib/radare2/0.8.6/bin_elf64.dylib 
./work/destroot/opt/local/lib/radare2/0.8.6/bin_elf64.dylib:
	bin_elf64.dylib.0.8.6 (compatibility version 0.0.0, current version 0.0.0)
	libr_util.dylib (compatibility version 0.0.0, current version 0.0.0)
	/usr/lib/libSystem.B.dylib (compatibility version 1.0.0, current version 125.2.11)

[g5pw (Aljaž Srebrnič)]

Hmm... yes, it does! However, this could be a false positive because I installed it previously, then uninstalled but maybe some libraries weren't removed. Will check now!

[g5pw (Aljaž Srebrnič)]

Removed all files, removed radare2 and reinstalled radare2 from macports. No problem whatsoever, radare2 seems to run just fine for me!

[pixilla (Bradley Giesbrecht)]

I do not know how to use radare2. Here is what I see:

$ radare2
dyld: Library not loaded: libr_core.dylib
  Referenced from: /opt/local/bin/radare2
  Reason: image not found
Trace/BPT trap

$ otool -L /opt/local/bin/radare2 
/opt/local/bin/radare2:
	libr_core.dylib (compatibility version 0.0.0, current version 0.0.0)
...
	/opt/local/lib/libssl.1.0.0.dylib (compatibility version 1.0.0, current version 1.0.0)
...

Usually it would be a full path to libr_core.dylib like the path /opt/local/lib/libssl.1.0.0.dylib

[g5pw (Aljaž Srebrnič)]

Hmm.. interesting! looking into it now...

[g5pw (Aljaž Srebrnič)]

Looks like some issues with the configure script, I contacted the developer, will keep you posted!

[ryandesign (Ryan Carsten Schmidt)]

revised portfile proposal

[ryandesign (Ryan Carsten Schmidt)]

I'm attaching a revised Portfile proposal with these changes from the original:

• Fixed whitespace to use only spaces not tabs and rearranged some lines
• Used modern checksum types
• Fixed license
• Adjusted descriptions
• Adjusted homepage and master_sites
• Added dependencies on libraries that get used
• Added args and a patchfile to ensure we're UsingTheRightCompiler and -arch flags
• Disabled universal variant and explained why; this could possibly be reinstated later using the muniversal portgroup

There still remain issues. The install_name of the libraries is wrong, as Bradley mentioned, because the build process only passes the library name in the -install_name argument, when it needs to be passing the entire absolute path the library will ultimately be installed at. The build process also is not passing the -compatibility_version and -current_version flags when building the libraries, which it should be doing. Finally, the library names have the version number at the end (i.e. bin_elf64.dylib.0.8.6) which might be how things are done on Linux but is not correct on OS X (we use e.g. bin_elf64.0.8.6.dylib).

[g5pw (Aljaž Srebrnič)]

g5pw == a2piratesoft, so I can now take care of this :)