search

ckolivas / lrzip

Long Range Zip
http://lrzip.kolivas.org
GNU General Public License v2.0
618 stars 76 forks source link

configure: set nasm format correctly on Darwin #211

Closed alebcay closed 3 years ago

alebcay commented 3 years ago

I'm not sure if this is necessarily needed since linking and compiling with the elf64 object still succeeds on macOS, but I figure this is probably more proper, if nothing else.

pete4abw commented 3 years ago

IDK about this. I limit assembler to just x86_64 and configure nasm accordingly.. Your code does not account for 32 or 64 bit processors (although I am thinking of ditching 32 bit support anyway). See my implementation of lrzip-next and the 21.04beta branch. ASM support is not really required and only on decompression does it provide any really useful benefits, up to 40% speed improvement. The ASM match finder code (new) offers limited support. I no longer use the ASM CRC module, but rather use libgcrypt for all hashing and encryption.