facebook / facebook/zstd

More clarity in docus about what "make install" will do, and where.

Open
#4,757 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
C
Stars
27.9k
Forks
2.6k
Avg merge
1d 3h
Merged PRs (30d)
8

Description

I built zstd for the first time last night, on Gnu/Linux, and I want to point something out.
**The docs, as many as there are, don't explain that on a supported platform, the binary products (lib and zstd) will go under `/usr/local`**. Yes, that's something many people will assume, and that's what it does. But I asked myself what could be done if I wanted to install zstd somewhere else. Or what if I wanted to stage the install (put the installables into a temporary directory for eventual installation somewhere else).

Let me note that I admire the Makefile. It's very unusual these days to find that the build infrastructure is just a Makefile, rather than one which uses those other approaches (you do provide). Years ago I wrote Makefiles like this. Attempts to use every feature of gmake ;-). But I grepped through the Makefile and failed to locate anything like PREFIX or DESTDIR. Thus leaving me with the conclusion that using gmake alone, there was no way to stage.

**How to address this**
Introduce the DESTDIR and PREFIX variables into the Makefile logic.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.