samtools / htslib

C library for high-throughput sequencing data formats
Other
799 stars 445 forks source link

Make it easier to modify shared library permissions during install #1532

Closed jkbonfield closed 1 year ago

jkbonfield commented 1 year ago

The new LIB_PERM Makefile variable defaults to 644 (as before), but can now be amended to e.g. 755. Updated the INSTALL file with a section for HP-UX explaining how to do this.

Fixes #1525

jmarshall commented 1 year ago

It can already be equivalently overridden with make install INSTALL_LIB='$(INSTALL)', but I guess this avoids the need for quoting there.

jkbonfield commented 1 year ago

Indeed, and I think I noted that in the issue. The comment was to make it "easier", which this most definitely does. Both easier for users, and inevitably easier for us as (assuming anyone actually uses it) we should get fewer issues caused by incorrect punctuation.