search

Feh / nocache

minimize caching effects
BSD 2-Clause "Simplified" License
554 stars 53 forks source link

Makefile #14

Closed onlyjob closed 11 years ago

onlyjob commented 11 years ago

Usable "Makefile" as per GNU Make best practice.

Feh commented 11 years ago

This is really good, thank you! I’d be in favor of adding another target fullinstall that also installs the cachedel and cachestats binaries. What do you think?

noushi commented 11 years ago

install target should install nocache plus cachestats and cachedel. The other tools are equally important. Thanks, Reda

onlyjob commented 11 years ago

Ops, I somehow forgotten about it, thanks for noticing. cache* executables are important and they shall be installed by "install" target. I added commit to fix that. I feel that "fullinstall" is unnecessary, uncommon and a bit confusing...

Feh commented 11 years ago

Okay, I was not sure about your intentions. I took the liberty of squashing your fixup commit with the previous one, rephrased my README update, and merged the changes to master. Thank you both!

onlyjob commented 11 years ago

Thank you for merging. By the way could you please stop advertising unstable "nocache" Debian packages? It would be better just to mention that "nocache" available from Debian or give a link to PTS page: 

http://packages.qa.debian.org/n/nocache.html

where users can choose binary packages from left hand side "versions" section. Thanks.

Feh commented 11 years ago

Okay with this?

onlyjob commented 11 years ago

Much better, thanks. :) It's just that binaries from "unstable" (a.k.a. "sid") may not be a good fit for other suites...