wangp / bower

A curses terminal client for the Notmuch email system
Other
128 stars 11 forks source link

make_man: get path to awk from /usr/bin/env #27

Closed erictapen closed 7 years ago

erictapen commented 7 years ago

This should improve portability on systems which don't have their binaries in /usr/bin.

Example for real world usage: I'm currently packaging bower for Nix, the Purely Functional Package Manager. Nix doesn't store any binaries in /usr/bin/ besides env. With this patch, bower builds correctly.

On any other supported platform this shouldn't make a difference, afaik.

OT: thanks for this niece piece of software. Looks like this is going to be my new MUA.

erictapen commented 7 years ago

Oops, I may made a mistake. Will reopen it...

erictapen commented 7 years ago

Okay, I changed my mind about this. As far as I can see, #!/usr/bin/env awk -f has a problem with the -f flag. So I wouldn't reopen this, sorry for the noise.

For Nix, I will patch the file with the right path to awk.

wangp commented 7 years ago

The makefile now invokes awk instead of relying on #!

erictapen commented 7 years ago

Thanks, that does the job.