Removed header from Makefile so princeprocessor compiles on Mac OS X

hashcat / princeprocessor

Standalone password candidate generator using the PRINCE algorithm

Other

425 stars 98 forks source link

Removed header from Makefile so princeprocessor compiles on Mac OS X #56

Closed brandonkovacs closed 6 years ago

brandonkovacs commented 6 years ago

Removed header file from Makefile so princeprocessor compiles on Mac OS X.

This was the advised method on Stackoverflow.

magnumripper commented 6 years ago

I believe this is better:

pp64.bin: pp.c mpz_int128.h
-   $(CC_LINUX64)   $(CFLAGS_LINUX64)   -o $@ $^
+   $(CC_LINUX64)   $(CFLAGS_LINUX64)   -o $@ $<

This change should be made for all targets

brandonkovacs commented 6 years ago

I believe this is better:
pp64.bin: pp.c mpz_int128.h
-     $(CC_LINUX64)   $(CFLAGS_LINUX64)   -o $@ $^
+     $(CC_LINUX64)   $(CFLAGS_LINUX64)   -o $@ $<
This change should be made for all targets

Yup - you're right. This worked too!

magnumripper commented 6 years ago

The difference is that the latter solution will correctly re-build the target if mpz_int128.h is changed whereas with your original PR it wouldn't. Makefiles are voodoo stuff.