Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix command path #44

Closed
wants to merge 3 commits into from
Closed

Fix command path #44

wants to merge 3 commits into from

Conversation

falsandtru
Copy link

Does not work on CentOS7.
Scripts installed in /usr/local/sbin/. But not in the path.

@dajhorn
Copy link
Member

dajhorn commented Apr 7, 2015

@falsandtru, thanks for the bug report and patch. This is an integration ticket because some distributions have PATH restrictions for things that run as root.

@falsandtru falsandtru changed the title Fix PATH variable value Fix command path May 4, 2015
@falsandtru
Copy link
Author

Fixed. But DESTDIR parameter does not apply. Is there a better solution?

https://github.com/zfsonlinux/zfs-auto-snapshot/blob/master/Makefile#L18-L19

vozhyk- added a commit to vozhyk-/zfs-auto-snapshot that referenced this pull request Feb 7, 2016
Change zfs-auto-snapshot to the absolute path when installing

Inspired by zfsonlinux#44.
vozhyk- added a commit to vozhyk-/zfs-auto-snapshot that referenced this pull request Feb 7, 2016
Change "zfs-auto-snapshot" to the absolute path when installing

Inspired by zfsonlinux#44.
vozhyk- added a commit to vozhyk-/zfs-auto-snapshot that referenced this pull request Feb 7, 2016
Change "zfs-auto-snapshot" to the absolute path when installing

Inspired by zfsonlinux#44

Issue zfsonlinux#47
@falsandtru falsandtru deleted the patch-1 branch April 12, 2019 20:56
@antoinetran
Copy link

Hi @highvoltage , it does not seem this is merged to zfsonlinux:master branch, despite the closed ticket.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants