Makefile: fix install path error - Githubissues

wslutilities / wslu

A collection of utilities for Windows Subsystem for Linux

https://wslutiliti.es/wslu

GNU General Public License v3.0

3.44k stars 155 forks source link

Makefile: fix install path error #261

Closed JeunghunKim closed 1 year ago

JeunghunKim commented 1 year ago

Add $(PREFIX) for conf_install path

Command

$ make PREFIX=/local/directory/ install

Expected message

install -Dm 644 src/etc/user/conf -t /local/directory/etc/wslu

Error

install: cannot create directory '/etc/wslu': Permission denied

Fix conf_install path in Makefile: add $(PREFIX)

Description

Types of changes

Breaking change (fix or feature that would cause existing functionality to not work as expected) are no longer accepted

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)

Checklist:

[x] I have read Code of Conduct and Contributing documentations.
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

patrick330602 commented 1 year ago

The behavior is intended. PREFIX is designed for installation for binary and support file locations. Configuration is intended to ignore PREFIX to be installed in /etc/. for installation the entire package to a custom directory, please use DESTDIR.