Set variables in the Makefile conditionally.

[ghost]

It's the bare minimum that's needed in order to make this utility more easily packageable by package managers.

[professorjamesmoriarty]

If this is merged, It would be great if a point release could be made.

[leahneukirchen]

I could do this, but make PREFIX=/usr works already... what's the benefit?

[jepler]

@chneukirchen when the PREFIX is provided via environment, it does not override a Makefile variable set with "=".

$ env PREFIX=y make -n install
mkdir -p /usr/local/bin /usr/local/share/man/man1
install -m0755 nq fq tq /usr/local/bin
install -m0644 nq.1 fq.1 tq.1 /usr/local/share/man/man1

With the proposed patch, it does.

[h3xx]

@jepler That's why you set the variables by appending them to the make(1) invocation:

$ make -n install PREFIX=y
...
mkdir -p y/bin y/share/man/man1
install -m0755 nq fq tq y/bin
install -m0644 nq.1 fq.1 tq.1 y/share/man/man1

Or you can use make -e:

$ env PREFIX=y make -e -n install 
...
mkdir -p y/bin y/share/man/man1
install -m0755 nq fq tq y/bin
install -m0644 nq.1 fq.1 tq.1 y/share/man/man1

I think this is flexible enough as it is.