GramThanos / WakeOnLAN

A simple C program that sends a magic packet
MIT License
100 stars 35 forks source link

Typo in Makefile and Linux include to heed warning #4

Closed dgoffredo closed 3 years ago

dgoffredo commented 3 years ago

I've got this thing working on my Ubuntu machine to wake up an old laptop over LAN. Thanks for writing this tool, it's handy.

I saw the Makefile in the repo, but it listed WakeOnLan.c as a dependency instead of WakeOnLAN.c (capital LAN), so I fixed that, and while I was there I used the special make variables $@ and $< to avoid restating the target and its dependency.

Then when I compiled the code, GCC warned that inet_addr was implicitly defined, so I included the header file from OpenGroup's documentation on inet_addr. You had already included it for Darwin.

I also added a .gitignore file to exclude the built binary.

GramThanos commented 3 years ago

Thank you!